DoliModules\BookCal\Controller\BookCalAvailabilitiesContactController::index()

Complexity

Conditions	16
Paths	256

Size

Total Lines	99
Code Lines	55

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function index(bool $executeActions = true): bool
    {
        global $conf;
        global $db;
        global $user;
        global $hookmanager;
        global $user;
        global $menumanager;
        global $langs;


// Load translation files required by the page
        $langs->loadLangs(["agenda", "companies", "other", "mails"]);

        $id = (GETPOST('id') ? GETPOSTINT('id') : GETPOSTINT('facid')); // For backward compatibility
        $ref = GETPOST('ref', 'alpha');
        $lineid = GETPOSTINT('lineid');
        $socid = GETPOSTINT('socid');
        $action = GETPOST('action', 'aZ09');

// Initialize technical objects
        $object = new Availabilities($db);
        $extrafields = new ExtraFields($db);
        $diroutputmassaction = $conf->bookcal->dir_output . '/temp/massgeneration/' . $user->id;
        $hookmanager->initHooks(['availabilitiescontact', 'globalcard']); // Note that conf->hooks_modules contains array
// Fetch optionals attributes and labels
        $extrafields->fetch_name_optionals_label($object->table_element);

// Load object
        include DOL_DOCUMENT_ROOT . '/core/actions_fetchobject.inc.php'; // Must be include, not include_once  // Must be include, not include_once. Include fetch and fetch_thirdparty but not fetch_optionals

// There is several ways to check permission.
// Set $enablepermissioncheck to 1 to enable a minimum low level of checks
        $enablepermissioncheck = 0;
        if ($enablepermissioncheck) {
            $permissiontoread = $user->hasRight('bookcal', 'availabilities', 'read');
            $permission = $user->hasRight('bookcal', 'availabilities', 'write');
        } else {
            $permissiontoread = 1;
            $permission = 1;
        }

// Security check (enable the most restrictive one)
//if ($user->socid > 0) accessforbidden();
//if ($user->socid > 0) $socid = $user->socid;
//$isdraft = (($object->status == $object::STATUS_DRAFT) ? 1 : 0);
//restrictedArea($user, $object->element, $object->id, $object->table_element, '', 'fk_soc', 'rowid', $isdraft);
        if (!isModEnabled('bookcal')) {
            accessforbidden();
        }
        if (!$permissiontoread) {
            accessforbidden();
        }


        /*
         * Add a new contact
         */

        if ($action == 'addcontact' && $permission) {
            $contactid = (GETPOST('userid') ? GETPOSTINT('userid') : GETPOSTINT('contactid'));
            $typeid = (GETPOST('typecontact') ? GETPOST('typecontact') : GETPOST('type'));
            $result = $object->add_contact($contactid, $typeid, GETPOST("source", 'aZ09'));

            if ($result >= 0) {
                header("Location: " . $_SERVER['PHP_SELF'] . "?id=" . $object->id);
                exit;

In this branch, the function will implicitly return null which is incompatible with the type-hinted return boolean. Consider adding a return statement or allowing null as return value.

            } else {
                if ($object->error == 'DB_ERROR_RECORD_ALREADY_EXISTS') {
                    $langs->load("errors");
                    setEventMessages($langs->trans("ErrorThisContactIsAlreadyDefinedAsThisType"), null, 'errors');
                } else {
                    setEventMessages($object->error, $object->errors, 'errors');
                }
            }
        } elseif ($action == 'swapstatut' && $permission) {
            // Toggle the status of a contact
            $result = $object->swapContactStatus(GETPOSTINT('ligne'));
        } elseif ($action == 'deletecontact' && $permission) {
            // Deletes a contact
            $result = $object->delete_contact($lineid);

            if ($result >= 0) {
                header("Location: " . $_SERVER['PHP_SELF'] . "?id=" . $object->id);
                exit;

In this branch, the function will implicitly return null which is incompatible with the type-hinted return boolean. Consider adding a return statement or allowing null as return value.

            } else {
                dol_print_error($db);
            }
        }


        /*
         * View
         */
        require_once realpath(BASE_PATH . '/../Dolibarr/Modules/BookCal/Views/availabilities_contact.php');

        $db->close();

        return true;
    }