+global\AlumniCategoryForm::__construct()

Complexity

Conditions	7
Paths	32

Size

Total Lines	81
Code Lines	63

Duplication

Lines	6
Ratio	7.41 %

Importance

Changes

0

<?php
    public function __construct(AlumniCategory $obj)
    {
        $xoops         = Xoops::getInstance();
        $moduleDirName = basename(dirname(__DIR__));
        $admin_lang    = '_AM_' . strtoupper($moduleDirName);

$admin_lang does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$admin_lang is not used, you could remove the assignment.

        $title = sprintf($obj->isNew() ? AlumniLocale::ADD_CAT : AlumniLocale::EDIT_CAT);
        parent::__construct($title, 'form', false, 'post', true);

        include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';

        $this->setExtra('enctype="multipart/form-data"');

        include_once XOOPS_ROOT_PATH . '/class/tree.php';
        $categoryHandler = $xoops->getModuleHandler('category', 'alumni');
        $arr             = $categoryHandler->getAll();
        $mytree          = new XoopsObjectTree($arr, 'cid', 'pid');

        $this->addElement(new Xoops\Form\Label(AlumniLocale::CATEGORY_PID, $mytree->makeSelBox('pid', 'title', '-', $obj->getVar('pid'), true)));
        $this->addElement(new Xoops\Form\Text(AlumniLocale::CATEGORY_TITLE, 'title', 50, 255, $obj->getVar('title')), true);

        if ($obj->isNew()) {
            $default_img = 'default.gif';

$default_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        } else {
            $default_img = str_replace('alumni/', '', $obj->getVar('img', 'e'));

$default_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }

        $img         = $obj->getVar('img') ?: 'default.gif';
        $imgtray_img = new Xoops\Form\ElementTray(AlumniLocale::IMGCAT, '<br>');

$imgtray_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $img_path    = \XoopsBaseConfig::get('root-path') . '/modules/alumni/images/cat';

$img_path does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imgpath_img     = sprintf(AlumniLocale::FORMIMAGE_PATH, $img_path);

$imgpath_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$imgpath_img is not used, you could remove the assignment.

        $imageselect_img = new Xoops\Form\Select(sprintf(XoopsLocale::F_FILE_EXISTS_IN, $img_path . '/'), 'img', $img);

$imageselect_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $image_array_img = XoopsLists::getImgListAsArray($img_path);

$image_array_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imageselect_img->addOption("$default_img", $default_img);

$imageselect_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        foreach ($image_array_img as $image_img) {

$image_array_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $imageselect_img->addOption("$image_img", $image_img);

$imageselect_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }

        $alumni_upload_url = \XoopsBaseConfig::get('url') . '/modules/alumni/images/cat';

$alumni_upload_url does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imageselect_img->setExtra("onchange='showImgSelected(\"image_img\", \"img\", \"\", \"\", \"" . $alumni_upload_url . "\")'");

$imageselect_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imgtray_img->addElement($imageselect_img, false);

$imgtray_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imgtray_img->addElement(new Xoops\Form\Label('', "<br><img src='" . $alumni_upload_url . '/' . $img . "' name='image_img' id='image_img' alt=''>"));

$imgtray_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $fileseltray_category_img = new Xoops\Form\ElementTray('<br>', '<br>');

$fileseltray_category_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $fileseltray_category_img->addElement(new Xoops\Form\File(AlumniLocale::FORMUPLOAD, 'img'), false);

$fileseltray_category_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $fileseltray_category_img->addElement(new Xoops\Form\Label(''), false);

$fileseltray_category_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imgtray_img->addElement($fileseltray_category_img);

$imgtray_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $this->addElement($imgtray_img);

$imgtray_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $this->addElement(new Xoops\Form\Text(AlumniLocale::ORDER, 'ordre', 4, 4, $obj->getVar('ordre')), false);
        $this->addElement(new Xoops\Form\Label(AlumniLocale::IFSCHOOL, ''));

        $photo_old            = $obj->getVar('scphoto') ?: '';

$photo_old does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $uploadirectory_photo = XOOPS_ROOT_PATH . "/uploads/{$moduleDirName}/photos/school_photos";

$uploadirectory_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imgtray_photo        = new Xoops\Form\ElementTray(AlumniLocale::SCPHOTO, '<br>');

$imgtray_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $imgpath_photo        = sprintf(AlumniLocale::FORMIMAGE_PATH, $uploadirectory_photo);

$imgpath_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$imgpath_photo is not used, you could remove the assignment.

        $fileseltray_photo    = new Xoops\Form\ElementTray('', '<br>');

$fileseltray_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $fileseltray_photo->addElement(new XoopsFormFile(AlumniLocale::FORMUPLOAD, 'scphoto', $xoops->getModuleConfig('alumni_photomax')), false);

$fileseltray_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        if ($photo_old) {

$photo_old does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

This code seems to be duplicated across your project.

            $fileseltray_photo->addElement(new Xoops\Form\Label(AlumniLocale::SELECTED_PHOTO, '<a href="../photos/school_photos/' . $photo_old . '">' . $photo_old . '</a>', false));

$fileseltray_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $imgtray_checkbox = new Xoops\Form\Checkbox('', 'del_photo', 0);

$imgtray_checkbox does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $imgtray_checkbox->addOption(1, AlumniLocale::DELPICT);

$imgtray_checkbox does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $fileseltray_photo->addElement($imgtray_checkbox);

$fileseltray_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
        $imgtray_photo->addElement($fileseltray_photo);

$imgtray_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $this->addElement($imgtray_photo);

$imgtray_photo does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $this->addElement(new Xoops\Form\Hidden('photo_old', $photo_old));

$photo_old does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCADDRESS, 'scaddress', 50, 255, $obj->getVar('scaddress')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCADDRESS2, 'scaddress2', 50, 255, $obj->getVar('scaddress2')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCCITY, 'sccity', 50, 255, $obj->getVar('sccity')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCSTATE, 'scstate', 50, 255, $obj->getVar('scstate')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCZIP, 'sczip', 50, 255, $obj->getVar('sczip')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCPHONE, 'scphone', 50, 255, $obj->getVar('scphone')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCFAX, 'scfax', 50, 255, $obj->getVar('scfax')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCMOTTO, 'scmotto', 50, 255, $obj->getVar('scmotto')), false);
        $this->addElement(new Xoops\Form\Text(AlumniLocale::SCURL, 'scurl', 50, 255, $obj->getVar('scurl')), false);
        $this->addElement(new Xoops\Form\Hidden('op', 'save_category'));
        $this->addElement(new Xoops\Form\Button('', 'submit', XoopsLocale::A_SUBMIT, 'submit'));
    }
}