main.php ➔ edit() - Code Metrics - Inspection of "remove travis" - XoopsModules25x/xoopstube - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 8ca430...3024c9 )

by Michael
created 2016-06-01 04:12 UTC
main.php ➔ edit() F

↳ Parent: Project
Complexity

Conditions	35
Paths	> 20000
Size

Total Lines	276
Code Lines	196
Duplication

Lines	25
Ratio	9.06 %
Importance

Changes
Metric	Value
cc	35
eloc	196
c	0
b	0
f	0
nc	3221225473
nop	1
dl	25
loc	276
rs	2
How to fix Long Method Complexity

<?php

/**
 * Module: XoopsTube
 *
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 *
 * PHP version 5
 *
 * @category        Module
 * @package         Xoopstube
 * @author          XOOPS Development Team
 * @copyright       2001-2013 The XOOPS Project
 * @link            http://sourceforge.net/projects/xoops/
 * @license         GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @since           1.0.6
 * @version         $Id: $
 */

include_once __DIR__ . '/admin_header.php';
global $xoopsModule, $xoopsDB;
function myFunction($a, $b) {
    // Do something
}

$mytree = new XoopstubeTree($xoopsDB->prefix('xoopstube_cat'), 'cid', 'pid');

$op  = xtubeCleanRequestVars($_REQUEST, 'op', '');
$lid = xtubeCleanRequestVars($_REQUEST, 'lid', 0);

/**
 * @param int $lid
 *
 * @return null

 */
function edit($lid = 0)
{
    global $xoopsDB, $xtubemyts, $mytree, $xtubeImageArray, $xoopsModuleConfig;
function myFunction($a, $b) {
    // Do something
}

    $sql = 'SELECT * FROM ' . $xoopsDB->prefix('xoopstube_videos') . ' WHERE lid=' . $lid;
    if (!$result = $xoopsDB->query($sql)) {
        XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

        return false;
    }
    $video_array  = $xoopsDB->fetchArray($xoopsDB->query($sql));
    $directory    = $xoopsModuleConfig['videoimgdir'];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    $lid          = $video_array['lid'] ? $video_array['lid'] : 0;
    $cid          = $video_array['cid'] ? $video_array['cid'] : 0;
    $title        = $video_array['title'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['title']) : '';
    $vidid        = $video_array['vidid'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['vidid']) : '';
    $picurl       = $video_array['picurl'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['picurl']) : 'http://';
    $publisher    = $video_array['publisher'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['publisher']) : '';
    $screenshot   = $video_array['screenshot'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['screenshot']) : '';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    $descriptionb = $video_array['description'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['description']) : '';
    $published    = $video_array['published'] ? $video_array['published'] : time();
    $expired      = $video_array['expired'] ? $video_array['expired'] : 0;
    $updated      = $video_array['updated'] ? $video_array['updated'] : 0;
    $offline      = $video_array['offline'] ? $video_array['offline'] : 0;
    $vidsource    = $video_array['vidsource'] ? $video_array['vidsource'] : 0;
    $ipaddress    = $video_array['ipaddress'] ? $video_array['ipaddress'] : 0;
    $notifypub    = $video_array['notifypub'] ? $video_array['notifypub'] : 0;
    $time         = $video_array['time'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['time']) : '0:00:00';
    $keywords     = $video_array['keywords'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['keywords']) : '';
    $item_tag     = $video_array['item_tag'] ? $xtubemyts->htmlSpecialCharsStrip($video_array['item_tag']) : '';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

    include_once __DIR__ . '/admin_header.php';
    xoops_cp_header();
    //xtubeRenderAdminMenu( _AM_XOOPSTUBE_MVIDEOS );


    if ($lid) {
        $_vote_data = xtubeGetVoteDetails($lid);
        $text_info
                    = '
            <table width="100%" style="font-size: 90%;">
             <tr>
              <td style="width: 25%; border-right: #E8E8E8 1px solid; vertical-align: top; padding-left: 10px;">
               <div><b>' . _AM_XOOPSTUBE_VIDEO_ID . ' </b>' . $lid . '</div>
               <div><b>' . _AM_XOOPSTUBE_MINDEX_SUBMITTED . ': </b>' . xtubeGetTimestamp(
                formatTimestamp($video_array['date'], $xoopsModuleConfig['dateformat'])
            ) . '</div>
               <div><b>' . _AM_XOOPSTUBE_MOD_MODIFYSUBMITTER . ' </b>' . xtubeGetLinkedUserNameFromId(
                $video_array['submitter']
            ) . '</div>
               <div><b>' . _AM_XOOPSTUBE_VIDEO_IP . ' </b>' . $ipaddress . '</div>
               <div><b>' . _AM_XOOPSTUBE_VIDEO_VIEWS . ' </b>' . $video_array['hits'] . '</div>
              </td>
              <td style="width: 25%; border-right: #E8E8E8 1px solid; vertical-align: top; padding-left: 10px;">
               <div><b>' . _AM_XOOPSTUBE_VOTE_TOTALRATE . ': </b>' . intval($_vote_data['rate']) . '</div>
               <div><b>' . _AM_XOOPSTUBE_VOTE_USERAVG . ': </b>' . intval(round($_vote_data['avg_rate'], 2)) . '</div>
               <div><b>' . _AM_XOOPSTUBE_VOTE_MAXRATE . ': </b>' . intval($_vote_data['min_rate']) . '</div>
               <div><b>' . _AM_XOOPSTUBE_VOTE_MINRATE . ': </b>' . intval($_vote_data['max_rate']) . '</div>
              </td>
              <td style="width: 25%; border-right: #E8E8E8 1px solid; vertical-align: top; padding-left: 10px;">
               <div><b>' . _AM_XOOPSTUBE_VOTE_MOSTVOTEDTITLE . ': </b>' . intval($_vote_data['max_title']) . '</div>
               <div><b>' . _AM_XOOPSTUBE_VOTE_LEASTVOTEDTITLE . ': </b>' . intval($_vote_data['min_title']) . '</div>
               <div><b>' . _AM_XOOPSTUBE_VOTE_REGISTERED . ': </b>' . (intval(
                $_vote_data['rate'] - $_vote_data['null_ratinguser']
            )) . '</div>
               <div><b>' . _AM_XOOPSTUBE_VOTE_NONREGISTERED . ': </b>' . intval($_vote_data['null_ratinguser']) . '</div>
              </td>
              <td style="width: 25%; vertical-align: top; padding-left: 10px;">
                <div>' . xtubeGetVideoThumb(
                $video_array['vidid'],
                $video_array['title'],
                $video_array['vidsource'],
                $video_array['picurl'],
                $video_array['screenshot']
            ) . '</div>
              </td>
             </tr>
            </table>';
        echo '
            <fieldset style="border: #E8E8E8 1px solid;"><legend style="display: inline; font-weight: bold; color: #0A3760;">' . _AM_XOOPSTUBE_INFORMATION . '</legend>
            <div style="padding: 8px;">' . $text_info . '</div>
        <!--	<div style="padding: 8px;"><li>' . $xtubeImageArray['deleteimg'] . ' ' . _AM_XOOPSTUBE_VOTE_DELETEDSC . '</li></div>\n    -->
            </fieldset>
            <br />';
    }
    unset($_vote_data);

    $caption = ($lid) ? _AM_XOOPSTUBE_VIDEO_MODIFYFILE : _AM_XOOPSTUBE_VIDEO_CREATENEWFILE;

    $sform = new XoopsThemeForm($caption, 'storyform', xoops_getenv('PHP_SELF'));
    $sform->setExtra('enctype="multipart / form - data"');

// Video title
    $sform->addElement(new XoopsFormText(_AM_XOOPSTUBE_VIDEO_TITLE, 'title', 70, 255, $title), true);

// Video source
    $vidsource_array  = array(
        0   => _AM_XOOPSTUBE_YOUTUBE,
        1   => _AM_XOOPSTUBE_METACAFE,
        2   => _AM_XOOPSTUBE_IFILM,
        3   => _AM_XOOPSTUBE_PHOTOBUCKET,
        4   => _AM_XOOPSTUBE_VIDDLER,
        100 => _AM_XOOPSTUBE_GOOGLEVIDEO,
        101 => _AM_XOOPSTUBE_MYSPAVETV,
        102 => _AM_XOOPSTUBE_DAILYMOTION,
        103 => _AM_XOOPSTUBE_BLIPTV,
        104 => _AM_XOOPSTUBE_CLIPFISH,
        105 => _AM_XOOPSTUBE_LIVELEAK,
        106 => _AM_XOOPSTUBE_MAKTOOB,
        107 => _AM_XOOPSTUBE_VEOH,
        108 => _AM_XOOPSTUBE_VIMEO,
        109 => _MD_XOOPSTUBE_MEGAVIDEO,
        200 => _MD_XOOPSTUBE_XOOPSTUBE
    ); // #200 is reserved for XoopsTube's internal FLV player
    $vidsource_select = new XoopsFormSelect(_AM_XOOPSTUBE_VIDSOURCE, 'vidsource', $vidsource);
    $vidsource_select->addOptionArray($vidsource_array);
    $sform->addElement($vidsource_select);

// Video code
    $videocode = new XoopsFormText(_AM_XOOPSTUBE_VIDEO_DLVIDID, 'vidid', 70, 512, $vidid);
    $videocode->setDescription('<br /><span style="font-size: small;">' . _AM_XOOPSTUBE_VIDEO_DLVIDIDDSC . '</span>');
    $sform->addElement($videocode, true);
    $note = _AM_XOOPSTUBE_VIDEO_DLVIDID_NOTE;
    $sform->addElement(new XoopsFormLabel('', $note));

// Picture url
    $picurl = new XoopsFormText(_AM_XOOPSTUBE_VIDEO_PICURL, 'picurl', 70, 255, $picurl);
    $picurl->setDescription(
        '<br /><span style="font-weight: normal;font-size: smaller;">' . _AM_XOOPSTUBE_VIDEO_PICURLNOTE . '</span>'
    );
    $sform->addElement($picurl, false);

// Video publisher
    $sform->addElement(new XoopsFormText(_AM_XOOPSTUBE_VIDEO_PUBLISHER, 'publisher', 70, 255, $publisher), true);

// Time form
    $timeform = new XoopsFormText(_AM_XOOPSTUBE_TIME, 'time', 7, 7, $time);
    $timeform->setDescription('<span style="font-size: small;">(h:mm:ss)</span>');
    $sform->addElement($timeform, false);

// Category menu
    ob_start();
    $mytree->makeMySelBox('title', 'title', $cid, 0);
    $sform->addElement(new XoopsFormLabel(_AM_XOOPSTUBE_VIDEO_CATEGORY, ob_get_contents()));
    ob_end_clean();

// Description form

//    $editor = xtube_getWysiwygForm( _AM_XOOPSTUBE_VIDEO_DESCRIPTION, 'descriptionb', $descriptionb );
//    $sform -> addElement( $editor, false );

    $optionsTrayNote = new XoopsFormElementTray(_AM_XOOPSTUBE_VIDEO_DESCRIPTION, '<br />');
    if (class_exists('XoopsFormEditor')) {

        $options['name']   = 'descriptionb';
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        $options['value']  = $descriptionb;
        $options['rows']   = 5;
        $options['cols']   = '100%';
        $options['width']  = '100%';
        $options['height'] = '200px';
        $descriptionb      = new XoopsFormEditor('', $xoopsModuleConfig['form_options'], $options, $nohtml = false, $onfailure = 'textarea');
        $optionsTrayNote->addElement($descriptionb);
    } else {
        $descriptionb = new XoopsFormDhtmlTextArea(
            '', 'descriptionb', $item->getVar(

                'descriptionb',
                'e'
            ), '100%', '100%'
        );
        $optionsTrayNote->addElement($descriptionb);
    }

    $sform->addElement($optionsTrayNote, false);

// Meta keywords form
    $keywords = new XoopsFormTextArea(_AM_XOOPSTUBE_KEYWORDS, 'keywords', $keywords, 7, 60, false);
    $keywords->setDescription(
        "<br /><br /><br /><br /><span style='font-size: smaller;'>" . _AM_XOOPSTUBE_KEYWORDS_NOTE . "</span>"
    );
    $sform->addElement($keywords);

// Insert tags if Tag-module is installed
    if (xtubeIsModuleTagInstalled()) {

        include_once XOOPS_ROOT_PATH . "/modules/tag/include/formtag.php";
        $text_tags = new XoopsFormTag("item_tag", 70, 255, $video_array['item_tag'], 0);
        $sform->addElement($text_tags);
    } else {
        $sform->addElement(new XoopsFormHidden('item_tag', $video_array['item_tag']));
    }

// Video Publish Date
    $sform->addElement(new XoopsFormDateTime(_AM_XOOPSTUBE_VIDEO_SETPUBLISHDATE, 'published', $size = 15, $published));

    if ($lid) {
        $sform->addElement(new XoopsFormHidden('was_published', $published));
        $sform->addElement(new XoopsFormHidden('was_expired', $expired));
    }

// Video Expire Date
    $isexpired           = ($expired > time()) ? 1 : 0;
    $expiredates         = ($expired > time()) ? _AM_XOOPSTUBE_VIDEO_EXPIREDATESET . xtubeGetTimestamp(
            formatTimestamp($expired, $xoopsModuleConfig['dateformat'])
        ) : _AM_XOOPSTUBE_VIDEO_SETDATETIMEEXPIRE;
    $warning             = ($published > $expired && $expired > time()) ? _AM_XOOPSTUBE_VIDEO_EXPIREWARNING : '';
    $expiredate_checkbox = new XoopsFormCheckBox('', 'expiredateactivate', $isexpired);
    $expiredate_checkbox->addOption(1, $expiredates . " <br /> <br /> ");

    $expiredate_tray = new XoopsFormElementTray(_AM_XOOPSTUBE_VIDEO_EXPIREDATE . $warning, '');
    $expiredate_tray->addElement($expiredate_checkbox);
    $expiredate_tray->addElement(
        new XoopsFormDateTime(_AM_XOOPSTUBE_VIDEO_SETEXPIREDATE . " <br /> ", 'expired', 15, $expired)
    );
    $expiredate_tray->addElement(
        new XoopsFormRadioYN(_AM_XOOPSTUBE_VIDEO_CLEAREXPIREDATE, 'clearexpire', 0, ' ' . _YES . '', ' ' . _NO . '')
    );
    $sform->addElement($expiredate_tray);

// Set video offline yes/no
    $videostatus_radio = new XoopsFormRadioYN(
        _AM_XOOPSTUBE_VIDEO_FILESSTATUS, 'offline', $offline, ' ' . _YES . '', ' ' . _NO . ''
    );
    $sform->addElement($videostatus_radio);

// Set video status as updated yes/no
    $up_dated            = ($updated == 0) ? 0 : 1;
    $video_updated_radio = new XoopsFormRadioYN(
        _AM_XOOPSTUBE_VIDEO_SETASUPDATED, 'up_dated', $up_dated, ' ' . _YES . '', ' ' . _NO . ''
    );
    $sform->addElement($video_updated_radio);

    $result = $xoopsDB->query(
        "SELECT COUNT( * ) FROM " . $xoopsDB->prefix('xoopstube_broken') . " WHERE lid = " . $lid
    );
    list ($broken_count) = $xoopsDB->fetchRow($result);
    if ($broken_count > 0) {
        $video_updated_radio = new XoopsFormRadioYN(
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            _AM_XOOPSTUBE_VIDEO_DELEDITMESS, 'delbroken', 1, ' ' . _YES . '', ' ' . _NO . ''
        );
        $sform->addElement($editmess_radio);

    }

    if ($lid && $published == 0) {
        $approved         = ($published == 0) ? 0 : 1;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $approve_checkbox = new XoopsFormCheckBox(_AM_XOOPSTUBE_VIDEO_EDITAPPROVE, "approved", 1);
        $approve_checkbox->addOption(1, " ");
        $sform->addElement($approve_checkbox);
    }

    if (!$lid) {
        $button_tray = new XoopsFormElementTray('', '');
        $button_tray->addElement(new XoopsFormHidden('status', 1));
        $button_tray->addElement(new XoopsFormHidden('notifypub', $notifypub));
        $button_tray->addElement(new XoopsFormHidden('op', 'save'));
        $button_tray->addElement(new XoopsFormButton('', '', _AM_XOOPSTUBE_BSAVE, 'submit'));
        $sform->addElement($button_tray);
    } else {
        $button_tray = new XoopsFormElementTray('', '');
        $button_tray->addElement(new XoopsFormHidden('lid', $lid));
        $button_tray->addElement(new XoopsFormHidden('status', 2));
        $hidden = new XoopsFormHidden('op', 'save');
        $button_tray->addElement($hidden);

        $butt_dup = new XoopsFormButton('', '', _AM_XOOPSTUBE_BMODIFY, 'submit');
        $butt_dup->setExtra('onclick="this . form . elements . op . value = \'save\'"');
        $button_tray->addElement($butt_dup);
        $butt_dupct = new XoopsFormButton('', '', _AM_XOOPSTUBE_BDELETE, 'submit');
        $butt_dupct->setExtra('onclick="this.form.elements.op.value=\'delete\'"');
        $button_tray->addElement($butt_dupct);
        $butt_dupct2 = new XoopsFormButton('', '', _AM_XOOPSTUBE_BCANCEL, 'submit');
        $butt_dupct2->setExtra('onclick="this.form.elements.op.value=\'videosConfigMenu\'"');
        $button_tray->addElement($butt_dupct2);
        $sform->addElement($button_tray);
    }
    $sform->display();
    unset($hidden);
    include_once __DIR__ . '/admin_footer.php';

    return null;
}

switch (strtolower($op)) {
    case 'edit':
        edit($lid);
        break;

    case 'save':
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;

        $groups    = isset($_POST['groups']) ? $_POST['groups'] : array();
        $lid       = (!empty($_POST['lid'])) ? $_POST['lid'] : 0;
        $cid       = (!empty($_POST['cid'])) ? $_POST['cid'] : 0;
        $vidrating = (!empty($_POST['vidrating'])) ? $_POST['vidrating'] : 6;
        $status    = (!empty($_POST['status'])) ? $_POST['status'] : 2;

// Get data from form
        $vidid        = $xtubemyts->addslashes(trim($_POST['vidid']));
        $picurl       = ($_POST['picurl'] != 'http://') ? $xtubemyts->addslashes($_POST['picurl']) : '';
        $title        = $xtubemyts->addslashes(trim($_POST['title']));
        $descriptionb = $xtubemyts->addslashes(trim($_POST['descriptionb']));
        $time         = $xtubemyts->addslashes(trim($_POST['time']));
        $keywords     = $xtubemyts->addslashes(trim($_POST['keywords']));
        $item_tag     = $xtubemyts->addslashes(trim($_POST['item_tag']));
        $submitter    = $xoopsUser->uid();
        $publisher    = $xtubemyts->addslashes(trim($_POST['publisher']));
        $vidsource    = (!empty($_POST['vidsource'])) ? $_POST['vidsource'] : 0;
        $updated      = (isset($_POST['was_published']) && $_POST['was_published'] == 0) ? 0 : time();
        $published    = strtotime($_POST['published']['date']) + $_POST['published']['time'];

        if ($_POST['up_dated'] == 0) {
            $updated = 0;
            $status  = 1;
        }

        $offline   = ($_POST['offline'] == 1) ? 1 : 0;
        $approved  = (isset($_POST['approved']) && $_POST['approved'] == 1) ? 1 : 0;
        $notifypub = (isset($_POST['notifypub']) && $_POST['notifypub'] == 1);

        if (!$lid) {
            $date        = time();
            $publishdate = time();
            $expiredate  = '0';
        } else {
            $publishdate = $_POST['was_published'];
            $expiredate  = $_POST['was_expired'];
        }
        if ($approved == 1 && empty($publishdate)) {
            $publishdate = time();
        }
        if (isset($_POST['expiredateactivate'])) {
            $expiredate = strtotime($_POST['expired']['date']) + $_POST['expired']['time'];
        }
        if ($_POST['clearexpire']) {
            $expiredate = '0';
        }

// Update or insert linkload data into database
        if (!$lid) {
            $date        = time();
            $publishdate = time();
            $ipaddress   = $_SERVER['REMOTE_ADDR'];
            $sql         = "INSERT INTO " . $xoopsDB->prefix('xoopstube_videos')
                . " (lid, cid, title, vidid, screenshot, submitter, publisher, status, date, hits, rating, votes, comments, vidsource, published, expired, updated, offline, description, ipaddress, notifypub, vidrating, time, keywords, item_tag, picurl )";
            $sql .= " VALUES 	(NULL, $cid, '$title', '$vidid', '', '$submitter', '$publisher', '$status', '$date', 0, 0, 0, 0, '$vidsource', '$published', '$expiredate', '$updated', '$offline', '$descriptionb', '$ipaddress', '0', '$vidrating', '$time', '$keywords', '$item_tag', '$picurl')";
            //    $newid = $xoopsDB -> getInsertId();

        } else {
            $sql = "UPDATE " . $xoopsDB->prefix('xoopstube_videos') . " SET cid = $cid, title='$title', vidid='$vidid', screenshot='', publisher='$publisher', status='$status', vidsource='$vidsource', published='$published', expired='$expiredate', updated='$updated', offline='$offline', description='$descriptionb', vidrating='$vidrating', time='$time', keywords='$keywords', item_tag='$item_tag', picurl='$picurl' WHERE lid="
                . $lid;
        }

        if (!$result = $xoopsDB->queryF($sql)) {
            XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

            return false;
        }

        $newid = mysql_insert_id();

// Add item_tag to Tag-module
        if (!$lid) {

            $tagupdate = xtubeUpdateTag($newid, $item_tag);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        } else {
            $tagupdate = xtubeUpdateTag($lid, $item_tag);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        }

// Send notifications
        if (!$lid) {
            $tags               = array();
            $tags['VIDEO_NAME'] = $title;
            $tags['VIDEO_URL']
                                   = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/singlevideo.php?cid=' . $cid . '&amp;lid=' . $newid;
            $sql                   = 'SELECT title FROM ' . $xoopsDB->prefix('xoopstube_cat') . ' WHERE cid=' . $cid;
            $result                = $xoopsDB->query($sql);
            $row                   = $xoopsDB->fetchArray($xoopsDB->query($sql));
            $tags['CATEGORY_NAME'] = $row['title'];
            $tags['CATEGORY_URL']
                                   = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewcat.php?cid=' . $cid;
            $notification_handler  = & xoops_gethandler('notification');
            $notification_handler->triggerEvent('global', 0, 'new_video', $tags);
            $notification_handler->triggerEvent('category', $cid, 'new_video', $tags);
        }
        if ($lid && $approved && $notifypub) {
            $tags               = array();
            $tags['VIDEO_NAME'] = $title;
            $tags['VIDEO_URL']
                                   = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/singlevideo.php?cid=' . $cid . '&amp;lid=' . $lid;
            $sql                   = 'SELECT title FROM ' . $xoopsDB->prefix('xoopstube_cat') . ' WHERE cid=' . $cid;
            $result                = $xoopsDB->query($sql);
            $row                   = $xoopsDB->fetchArray($result);
            $tags['CATEGORY_NAME'] = $row['title'];
            $tags['CATEGORY_URL']
                                   = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewcat.php?cid=' . $cid;
            $notification_handler  = & xoops_gethandler('notification');
            $notification_handler->triggerEvent('global', 0, 'new_video', $tags);
            $notification_handler->triggerEvent('category', $cid, 'new_video', $tags);
            $notification_handler->triggerEvent('video', $lid, 'approve', $tags);
        }
        $message = (!$lid) ? _AM_XOOPSTUBE_VIDEO_NEWFILEUPLOAD : _AM_XOOPSTUBE_VIDEO_FILEMODIFIEDUPDATE;
        $message = ($lid && !$_POST['was_published'] && $approved) ? _AM_XOOPSTUBE_VIDEO_FILEAPPROVED : $message;

        if (xtubeCleanRequestVars($_REQUEST, 'delbroken', 0)) {
            $sql = 'DELETE FROM ' . $xoopsDB->prefix('xoopstube_broken') . ' WHERE lid=' . $lid;
            if (!$result = $xoopsDB->queryF($sql)) {
                XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

                return false;
            }
        }

        redirect_header('main.php', 1, $message);

        break;

    case 'delete':
        if (xtubeCleanRequestVars($_REQUEST, 'confirm', 0)) {
            $title = xtubeCleanRequestVars($_REQUEST, 'title', 0);

            // delete video
            $sql = 'DELETE FROM ' . $xoopsDB->prefix('xoopstube_videos') . ' WHERE lid=' . $lid;
            if (!$result = $xoopsDB->query($sql)) {
                XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

                return false;
            }

            // delete altcat
            $sql = 'DELETE FROM ' . $xoopsDB->prefix('xoopstube_altcat') . ' WHERE lid=' . $lid;
            if (!$result = $xoopsDB->query($sql)) {
                XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

                return false;
            }

            // delete vote data
            $sql = 'DELETE FROM ' . $xoopsDB->prefix('xoopstube_votedata') . ' WHERE lid=' . $lid;
            if (!$result = $xoopsDB->query($sql)) {
                XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

                return false;
            }

            // delete comments
            xoops_comment_delete($xoopsModule->getVar('mid'), $lid);
            redirect_header('main.php', 1, sprintf(_AM_XOOPSTUBE_VIDEO_FILEWASDELETED, $title));

            exit();
        } else {
            $sql = 'SELECT lid, title, item_tag, vidid FROM ' . $xoopsDB->prefix('xoopstube_videos') . ' WHERE lid=' . $lid;
            if (!$result = $xoopsDB->query($sql)) {
                XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

                return false;
            }
            list($lid, $title) = $xoopsDB->fetchrow($result);
            $item_tag = $result['item_tag'];

            xoops_cp_header();
            //xtubeRenderAdminMenu( _AM_XOOPSTUBE_BINDEX );


            xoops_confirm(
                array(
                    'op'      => 'delete',
                    'lid'     => $lid,
                    'confirm' => 1,
                    'title'   => $title
                ),
                'main.php',
                _AM_XOOPSTUBE_VIDEO_REALLYDELETEDTHIS . '<br /><br />' . $title,
                _DELETE
            );

            // Remove item_tag from Tag-module
            $tagupdate = xtubeUpdateTag($lid, $item_tag);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

            include_once __DIR__ . '/admin_footer.php';
        }
        break;

    case 'toggle':
        if (isset($_REQUEST['lid'])) {
            $lid = intval($_REQUEST['lid']);
            if (isset($_REQUEST['offline'])) {
                $offline = intval($_REQUEST['offline']);
                xtubeToggleOffline($lid, $offline);
            }
        }
        break;

    case 'delvote':
        $rid = xtubeCleanRequestVars($_REQUEST, 'rid', 0);
        $sql = 'DELETE FROM ' . $xoopsDB->prefix('xoopstube_votedata') . ' WHERE ratingid=' . $rid;
        if (!$result = $xoopsDB->queryF($sql)) {
            XoopsErrorHandler_HandleError(E_USER_WARNING, $sql, __FILE__, __LINE__);

            return false;
        }
        xtubeUpdateRating($rid);
        redirect_header('main.php', 1, _AM_XOOPSTUBE_VOTE_VOTEDELETED);
        break;

    case 'main':
    default:
switch ($expr) {
    default:
        doSomething(); //right
        break;
}


switch ($expr) {
    default:

        doSomething(); //wrong
        break;
}

        $start     = xtubeCleanRequestVars($_REQUEST, 'start', 0);
        $start1    = xtubeCleanRequestVars($_REQUEST, 'start1', 0);
        $start2    = xtubeCleanRequestVars($_REQUEST, 'start2', 0);
        $start3    = xtubeCleanRequestVars($_REQUEST, 'start3', 0);
        $start4    = xtubeCleanRequestVars($_REQUEST, 'start4', 0);
        $start5    = xtubeCleanRequestVars($_REQUEST, 'start5', 0);
        $totalcats = xtubeGetTotalCategoryCount();

        $result = $xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('xoopstube_broken'));
        list($totalbrokenvideos) = $xoopsDB->fetchRow($result);
        $result2 = $xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('xoopstube_mod'));
        list($totalmodrequests) = $xoopsDB->fetchRow($result2);
        $result3 = $xoopsDB->query(
            'SELECT COUNT(*) FROM ' . $xoopsDB->prefix('xoopstube_videos') . ' WHERE published = 0'
        );
        list($totalnewvideos) = $xoopsDB->fetchRow($result3);
        $result4 = $xoopsDB->query(
            'SELECT COUNT(*) FROM ' . $xoopsDB->prefix('xoopstube_videos') . ' WHERE published > 0'
        );
        list($totalvideos) = $xoopsDB->fetchRow($result4);

        xoops_cp_header();

        $indexAdmin = new ModuleAdmin();
        echo $indexAdmin->addNavigation('main.php');
        $indexAdmin->addItemButton(_MI_XOOPSTUBE_ADD_VIDEO, 'main.php?op=edit', 'add', '');
        $indexAdmin->addItemButton(_MI_XOOPSTUBE_ADD_CATEGORY, 'category.php', 'add', '');
        echo $indexAdmin->renderButton('right', '');

        //xtubeRenderAdminMenu( _AM_XOOPSTUBE_BINDEX );
//    echo '
//			<fieldset style="border: #E8E8E8 1px solid;">
//			<legend style="display: inline; font-weight: bold; color: #0A3760;">' . _AM_XOOPSTUBE_MINDEX_VIDEOSUMMARY . '</legend>
//			<div style="padding: 8px;">
//			<span style="font-size: small;">
//			<a href="category.php">' . _AM_XOOPSTUBE_SCATEGORY . '</a><b>' . $totalcats . '</b> |
//			<a href="main.php">' . _AM_XOOPSTUBE_SFILES . '</a><b>' . $totalvideos . '</b> |
//			<a href="newvideos.php">' . _AM_XOOPSTUBE_SNEWFILESVAL . '</a><b>' . $totalnewvideos . '</b> |
//			<a href="modifications.php">' . _AM_XOOPSTUBE_SMODREQUEST . '</a><b>' . $totalmodrequests . '</b> |
//			<a href="brokenvideo.php">' . _AM_XOOPSTUBE_SBROKENSUBMIT . '</a><b>' . $totalbrokenvideos . '</b>
//			</span>
//			</div>
//			</fieldset>';

        if ($totalcats > 0) {
            $sform = new XoopsThemeForm(_AM_XOOPSTUBE_CCATEGORY_MODIFY, 'category', 'category.php');
            ob_start();
            $mytree->makeMySelBox('title', 'title');
            $sform->addElement(new XoopsFormLabel(_AM_XOOPSTUBE_CCATEGORY_MODIFY_TITLE, ob_get_contents()));
            ob_end_clean();
            $dup_tray = new XoopsFormElementTray('', '');
            $dup_tray->addElement(new XoopsFormHidden('op', 'modCat'));
            $butt_dup = new XoopsFormButton('', '', _AM_XOOPSTUBE_BMODIFY, 'submit');
            $butt_dup->setExtra('onclick="this.form.elements.op.value=\'modCat\'"');
            $dup_tray->addElement($butt_dup);
            $butt_dupct = new XoopsFormButton('', '', _AM_XOOPSTUBE_BDELETE, 'submit');
            $butt_dupct->setExtra('onclick="this.form.elements.op.value=\'del\'"');
            $dup_tray->addElement($butt_dupct);
            $sform->addElement($dup_tray);
            $sform->display();

//TODO add table with categories


//            $sql='SELECT * FROM ' . $xoopsDB->prefix('xoopstube_cat') . ' ORDER BY cid DESC';

//            $published_array       = $xoopsDB->query($sql, $xoopsModuleConfig['admin_perpage'], $start);
//            $published_array_count = $xoopsDB->getRowsNum($xoopsDB->query($sql));
//            xtubeRenderCategoryListHeader(_AM_XOOPSTUBE_MINDEX_PUBLISHEDVIDEO);
//            xtubeSetPageNavigationCategoryList($published_array_count, $start, 'art', '', 'left');
//            if ($published_array_count > 0) {
//                while ($published = $xoopsDB->fetchArray($published_array)) {
//                    xtubeRenderCategoryListBody($published);
//                }
//                echo '</table>';
//            } else {
//                xtubeRenderCategoryListFooter();
//            }
//            xtubeSetPageNavigationCategoryList($published_array_count, $start, 'art', '', 'right');


        }

        if ($totalvideos > 0) {
            $sql
                                   = 'SELECT * FROM ' . $xoopsDB->prefix('xoopstube_videos') . ' WHERE published > 0  ORDER BY lid DESC';
            $published_array       = $xoopsDB->query($sql, $xoopsModuleConfig['admin_perpage'], $start);
            $published_array_count = $xoopsDB->getRowsNum($xoopsDB->query($sql));
            xtubeRenderVideoListHeader(_AM_XOOPSTUBE_MINDEX_PUBLISHEDVIDEO);
            xtubeSetPageNavigationVideoList($published_array_count, $start, 'art', '', 'left');
            if ($published_array_count > 0) {
                while ($published = $xoopsDB->fetchArray($published_array)) {
                    xtubeRenderVideoListBody($published);
                }
                echo '</table>';
            } else {
                xtubeRenderVideoListFooter();
            }
            xtubeSetPageNavigationVideoList($published_array_count, $start, 'art', '', 'right');
        }
        include_once __DIR__ . '/admin_footer.php';
        break;
}
/**
 * @param $lid
 * @param $offline
 *
 * @return bool|null
 */
function xtubeToggleOffline($lid, $offline)
{
    global $xoopsDB;
function myFunction($a, $b) {
    // Do something
}
    $message = '';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    $offline = ($offline == 1) ? 0 : 1;

    if ($offline) {
        $message = _AM_XOOPSTUBE_TOGGLE_OFFLINE_SUCCESS;
    } else {
        $message = _AM_XOOPSTUBE_TOGGLE_ONLINE_SUCCESS;
    }

//    $this_handler   =& xoops_getModuleHandler('xoopstube_videos', 'xoopstube');

//    $obj            = $this_handler->get($lid);
//    $obj->setVar('offline', $offline);
//    if ($this_handler->insert($obj, true)) {
//        redirect_header('main.php', 1, _AM_XOOPSTUBE_TOGGLE_SUCCESS);
//    } else {
//        redirect_header('main.php', 1, _AM_XOOPSTUBE_TOGGLE_FAILED);
//    }

    $sql = "UPDATE " . $xoopsDB->prefix('xoopstube_videos') . " SET  offline='$offline' WHERE lid=" . $lid;

    if (!$result = $xoopsDB->queryF($sql)) {
        redirect_header('main.php', 1, _AM_XOOPSTUBE_TOGGLE_FAILED);

        return false;
    } else {
        redirect_header('main.php', 1, $message);
    }

    return null;
}

Push — master ( 8ca430...3024c9 )

main.php ➔ edit() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

XoopsModules25x / xoopstube

Push — master ( 8ca430...3024c9 )

main.php ➔ edit() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like
