admin/main.php ➔ AddEditMarqueeForm()

Complexity

Conditions	10
Paths	512

Size

Total Lines	100
Code Lines	77

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function AddEditMarqueeForm(
    $marqueeid,
    $Action,
    $FormTitle,
    $contentvalue,
    $bgcolorvalue,
    $widthvalue,
    $heightvalue,
    $scrollamountvalue,
    $hspacevalue,
    $vspacevalue,
    $scrolldelayvalue,
    $directionvalue,
    $behaviourvalue,
    $alignvalue,
    $loopvalue,
    $stopvalue,
    $LabelSubmitButton,
    $sourcevalue = 'fixed'
) {
    require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';

The constant XOOPS_ROOT_PATH was not found. Maybe you did not declare it correctly or list all dependencies?

    global $xoopsModule;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

    $sform = new \XoopsThemeForm($FormTitle, 'marqueeform', XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/admin/main.php');

The type XoopsThemeForm was not found. Maybe you did not declare it correctly or list all dependencies?

The constant XOOPS_URL was not found. Maybe you did not declare it correctly or list all dependencies?

    $source = new \XoopsFormSelect(_AM_MARQUEE_SOURCE, 'source', $sourcevalue);

The type XoopsFormSelect was not found. Maybe you did not declare it correctly or list all dependencies?

    $source->addOption('fixed', _AM_MARQUEE_SOURCE_FIXED);
    $fileslst = myglob(XOOPS_ROOT_PATH . '/modules/marquee/plugins/', 'php');
    foreach ($fileslst as $onefile) {
        $onefile = basename($onefile, '.php');
        $source->addOption($onefile, $onefile);
    }
    $sform->addElement($source);

    /** @var \XoopsModules\Marquee\Utility $utility */
    $utility = new Marquee\Utility();

    $editor = $utility::getWysiwygForm(_AM_MARQUEE_CONTENT, 'content', $contentvalue, 15, 60, 'content_text_hidden');
    if ($editor) {
        $sform->addElement($editor, false);
    }

    if ('DHTML' !== $utility::getModuleOption('methodtouse')) {

The condition 'DHTML' !== $utility::getModuleOption('methodtouse') is always true.

        // $sform->addElement(new \XoopsFormText(_AM_MARQUEE_BGCOLOR, 'bgcolor', 7, 7, $bgcolorvalue), false);

65% of this comment could be valid code. Did you maybe forget this after debugging?

        $sform->addElement(new \XoopsFormColorPicker(_AM_MARQUEE_BGCOLOR, 'bgcolor', $bgcolorvalue), false);

The type XoopsFormColorPicker was not found. Maybe you did not declare it correctly or list all dependencies?

    }
    $sform->addElement(new \XoopsFormText(_AM_MARQUEE_WIDTH, 'width', 4, 4, $widthvalue), false);

The type XoopsFormText was not found. Maybe you did not declare it correctly or list all dependencies?

    $sform->addElement(new \XoopsFormText(_AM_MARQUEE_HEIGHT, 'height', 4, 4, $heightvalue), false);
    $sform->addElement(new \XoopsFormText(_AM_MARQUEE_SCRAMOUNT, 'scrollamount', 4, 4, $scrollamountvalue), false);
    if ('DHTML' !== $utility::getModuleOption('methodtouse')) {

The condition 'DHTML' !== $utility::getModuleOption('methodtouse') is always true.

        $sform->addElement(new \XoopsFormText(_AM_MARQUEE_HSPACE, 'hspace', 4, 4, $hspacevalue), false);
        $sform->addElement(new \XoopsFormText(_AM_MARQUEE_VSPACE, 'vspace', 4, 4, $vspacevalue), false);
    }

    $sform->addElement(new \XoopsFormText(_AM_MARQUEE_SCRDELAY, 'scrolldelay', 6, 6, $scrolldelayvalue), false);
    $direction = new \XoopsFormSelect(_AM_MARQUEE_DIRECTION, 'direction', $directionvalue);
    $direction->addOption('0', _AM_MARQUEE_DIRECTION1);
    $direction->addOption('1', _AM_MARQUEE_DIRECTION2);
    $direction->addOption('2', _AM_MARQUEE_DIRECTION3);
    $direction->addOption('3', _AM_MARQUEE_DIRECTION4);
    $sform->addElement($direction, true);

    $behaviour = new \XoopsFormSelect(_AM_MARQUEE_BEHAVIOUR, 'behaviour', $behaviourvalue);
    $behaviour->addOption('0', _AM_MARQUEE_BEHAVIOUR1);
    if ('DHTML' !== $utility::getModuleOption('methodtouse')) {

The condition 'DHTML' !== $utility::getModuleOption('methodtouse') is always true.

        $behaviour->addOption('1', _AM_MARQUEE_BEHAVIOUR2);
    }
    $behaviour->addOption('2', _AM_MARQUEE_BEHAVIOUR3);
    $sform->addElement($behaviour, true);

    if ('DHTML' !== $utility::getModuleOption('methodtouse')) {

The condition 'DHTML' !== $utility::getModuleOption('methodtouse') is always true.

        $align = new \XoopsFormSelect(_AM_MARQUEE_ALIGN, 'align', $alignvalue);
        $align->addOption('0', _AM_MARQUEE_ALIGN1);
        $align->addOption('1', _AM_MARQUEE_ALIGN2);
        $align->addOption('2', _AM_MARQUEE_ALIGN3);
        $sform->addElement($align, true);
    }

    $loop = new \XoopsFormSelect(_AM_MARQUEE_LOOP, 'loop', $loopvalue);
    $loop->addOption('0', _AM_MARQUEE_INFINITELOOP);
    for ($i = 1; $i <= 100; ++$i) {
        $loop->addOption($i, $i);
    }
    if ('DHTML' !== $utility::getModuleOption('methodtouse')) {

The condition 'DHTML' !== $utility::getModuleOption('methodtouse') is always true.

        $sform->addElement($loop, true);
        $sform->addElement(new \XoopsFormRadioYN(_AM_MARQUEE_STOP, 'stoponmouseover', $stopvalue, _YES, _NO));

The constant _NO was not found. Maybe you did not declare it correctly or list all dependencies?

The constant _YES was not found. Maybe you did not declare it correctly or list all dependencies?

The type XoopsFormRadioYN was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    $sform->addElement(new \XoopsFormHidden('op', $Action), false);

The type XoopsFormHidden was not found. Maybe you did not declare it correctly or list all dependencies?

    if (!empty($marqueeid)) {
        $sform->addElement(new \XoopsFormHidden('marqueeid', $marqueeid), false);
    }
    $button_tray = new \XoopsFormElementTray('', '');

The type XoopsFormElementTray was not found. Maybe you did not declare it correctly or list all dependencies?

    $submit_btn  = new \XoopsFormButton('', 'submit', $LabelSubmitButton, 'submit');

The type XoopsFormButton was not found. Maybe you did not declare it correctly or list all dependencies?

    $button_tray->addElement($submit_btn);
    $cancel_btn = new \XoopsFormButton('', 'reset', _AM_MARQUEE_RESETBUTTON, 'reset');
    $button_tray->addElement($cancel_btn);
    $sform->addElement($button_tray);
    $sform->display();
}