XoopsModules25x / xoopstube

newlist.php

<?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-2016 XOOPS Project (http://xoops.org)
 * @license         GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @since           1.0.6
 * @link            http://xoops.org/
 */

include __DIR__ . '/header.php';

$xoopsOption['template_main'] = 'xoopstube_newlistindex.tpl';
include XOOPS_ROOT_PATH . '/header.php';

global $xoopsModule;

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

$catarray['imageheader'] = XoopstubeUtilities::xtubeRenderImageHeader();
$xoopsTpl->assign('catarray', $catarray);

if (XoopsRequest::getInt('newvideoshowdays', '', 'GET')) {
    $newvideoshowdays = XoopsRequest::getInt('newvideoshowdays', 7, 'GET');
    if ($newvideoshowdays !== 7) {
        if ($newvideoshowdays !== 14) {
            if ($newvideoshowdays !== 30) {
                redirect_header('newlist.php?newvideoshowdays=7', 5,
                                _MD_XOOPSTUBE_STOPIT . '<br><img src="' . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/assets/images/icon/security.png" />');
            }
        }
    }
    $time_cur       = time();
    $duration       = ($time_cur - (86400 * 30));
    $duration_week  = ($time_cur - (86400 * 7));
    $allmonthvideos = 0;
    $allweekvideos  = 0;
    $result         =
        $GLOBALS['xoopsDB']->query('SELECT lid, cid, published, updated FROM ' . $GLOBALS['xoopsDB']->prefix('xoopstube_videos') . ' WHERE (published >= ' . $duration . ' AND published <= '
                                   . $time_cur . ') OR updated >= ' . $duration . ' AND (expired = 0 OR expired > ' . $time_cur . ') AND offline = 0');
    while (false !== ($myrow = $GLOBALS['xoopsDB']->fetcharray($result))) {
        $published = ($myrow['updated'] > 0) ? $myrow['updated'] : $myrow['published'];
        ++$allmonthvideos;
        if ($published > $duration_week) {
            ++$allweekvideos;
        }
    }
    $xoopsTpl->assign('allweekvideos', $allweekvideos);
    $xoopsTpl->assign('allmonthvideos', $allmonthvideos);

    // List Last VARIABLE Days of videos
    //  $newvideoshowdays = xtubeCleanRequestVars($_REQUEST, 'newvideoshowdays', 7 );

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

    $newvideoshowdays = (int)XoopsRequest::getInt('newvideoshowdays', 7, 'GET');
    $xoopsTpl->assign('newvideoshowdays', $newvideoshowdays);

    $dailyvideos = array();
    for ($i = 0; $i < $newvideoshowdays; ++$i) {
        $key                                 = $newvideoshowdays - $i - 1;
        $time                                = $time_cur - (86400 * $key);
        $dailyvideos[$key]['newvideodayRaw'] = $time;
        $dailyvideos[$key]['newvideoView']   = XoopstubeUtilities::xtubeGetTimestamp(formatTimestamp($time, $GLOBALS['xoopsModuleConfig']['dateformat']));
        $dailyvideos[$key]['totalvideos']    = 0;
    }
}

$duration = ($time_cur - (86400 * ($newvideoshowdays - 1)));
$result   =
    $GLOBALS['xoopsDB']->query('SELECT lid, cid, published, updated FROM ' . $GLOBALS['xoopsDB']->prefix('xoopstube_videos') . ' WHERE (published > ' . $duration . ' AND published <= ' . $time_cur
                               . ') OR (updated >= ' . $duration . ' AND updated <= ' . $time_cur . ') AND (expired = 0 OR expired > ' . $time_cur . ') AND offline = 0');
while (false !== ($myrow = $GLOBALS['xoopsDB']->fetcharray($result))) {
    $published = ($myrow['updated'] > 0) ? $myrow['updated'] : $myrow['published'];
    $d         = date('j', $published);
    $m         = date('m', $published);
    $y         = date('Y', $published);
    $key       = (int)($time_cur - mktime(0, 0, 0, $m, $d, $y) / 86400);
    $dailyvideos[$key]['totalvideos']++;
}
ksort($dailyvideos);
reset($dailyvideos);
$xoopsTpl->assign('dailyvideos', $dailyvideos);
unset($dailyvideos);

$mytree = new XoopstubeTree($GLOBALS['xoopsDB']->prefix('xoopstube_cat'), 'cid', 'pid');
$sql    = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('xoopstube_videos');
$sql .= 'WHERE (published > 0 AND published <= ' . $time_cur . ')
         OR (updated > 0 AND updated <= ' . $time_cur . ')
         AND (expired = 0 OR expired > ' . $time_cur . ')
         AND offline = 0
         ORDER BY ' . $GLOBALS['xoopsModuleConfig']['linkxorder'];
$result = $GLOBALS['xoopsDB']->query($sql, 10, 0);
while (false !== ($video_arr = $GLOBALS['xoopsDB']->fetchArray($result))) {
    include XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/videoloadinfo.php';
}

$xoopsTpl->assign('back', '<a href="javascript:history.go(-1)"><img src="' . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/assets/images/icon/back.png" /></a>');
$xoopsTpl->assign('module_dir', $xoopsModule->getVar('dirname'));
include XOOPS_ROOT_PATH . '/footer.php';