Issues in index.php - Fixed Issues - Inspection of "3.10. FINAL" - XoopsModules25x/xfguestbook - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#10)

by Michael

created 2017-01-23 19:19 UTC

index.php (2 issues)

Labels

Severity

Minor 2

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
// $Id: index.php,v 1.3 2006/03/12 C. Felix AKA the Cat
//  ------------------------------------------------------------------------ //
//             XF Guestbook                                                  //
// ------------------------------------------------------------------------- //
//  This program is free software; you can redistribute it and/or modify     //
//  it under the terms of the GNU General Public License as published by     //
//  the Free Software Foundation; either version 2 of the License, or        //
//  (at your option) any later version.                                      //
//                                                                           //
//  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.                                      //
//                                                                           //
//  This program is distributed in the hope that it will be useful,          //
//  but WITHOUT ANY WARRANTY; without even the implied warranty of           //
//  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the            //
//  GNU General Public License for more details.                             //
//                                                                           //
//  You should have received a copy of the GNU General Public License        //
//  along with this program; if not, write metalslugto the Free Software              //
//  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA //
//  ------------------------------------------------------------------------ //

include dirname(dirname(__DIR__)) . '/mainfile.php';
//include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->dirname()."/class/msg.php");
include_once(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->dirname() . '/include/functions.php');
if (isset($_GET['msg_id'])) {
    $msg_id = (int)$_GET['msg_id'];
} elseif (isset($_POST['msg_id'])) {
    $msg_id = (int)$_POST['msg_id'];
} else {
    $msg_id = 0;
}

if (isset($_GET['op'])) {
    $op = $_GET['op'];
} elseif (isset($_POST['op'])) {
    $op = $_POST['op'];
} else {
    $op = 'show_all';
}

$msg_handler = xoops_getModuleHandler('msg');

//Admin or not
$xoopsUser ? $adminview = $xoopsUser->isAdmin() : $adminview = 0;

/**
 * @param $msg_id
 */
function delete($msg_id)
{
    global $msg_handler, $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    $ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
    if ($ok == 1) {
        $msg        = &$msg_handler->get($msg_id);
        $del_msg_ok = $msg_handler->delete($msg);
        $filename   = $msg->getVar('photo');
        if ($filename !== '') {
            $filename = XOOPS_UPLOAD_PATH . '/' . $xoopsModule->getVar('dirname') . '/' . $filename;
            unlink($filename);
        }
        if ($del_msg_ok) {
            $messagesent = _MD_XFGB_MSGDELETED;
        } else {
            $messagesent = _MD_XFGB_ERRORDEL;
        }
        redirect_header('index.php', 2, $messagesent);
    } else {
        xoops_confirm(array('op' => 'delete', 'msg_id' => $msg_id, 'ok' => 1), 'index.php', _DELETE);
    }
}

/**
 * @param $msg_id
 */
function approve($msg_id)
{
    global $msg_handler;
function myFunction($a, $b) {
    // Do something
}

    $msg = &$msg_handler->get($msg_id);
    $msg->setVar('moderate', 0);
    if (!$msg_handler->insert($msg)) {
        $messagesent = _MD_XFGB_ERRORVALID;
    } else {
        $messagesent = _MD_XFGB_VALIDATE;
    }
    redirect_header('index.php?op=show_waiting', 2, $messagesent);
}

/**
 * @param $msg
 */
function xfgb_getmsg($msg)
{
    global $nbmsg, $xoopsModule, $xoopsUser, $xoopsModuleConfig, $xoopsTpl, $xoopsConfig, $options, $opt, $xoopsDB;

    $arr_country = xfgb_getAllCountry();
    $xoopsTpl->assign('display_msg', true);
    foreach ($msg as $onemsg) {
        if ($poster = xfgb_get_user_data($onemsg->getVar('user_id'))) {
            $a_msg = &$poster;
        } else {
            $a_msg             = array();
            $a_msg['poster']   = $onemsg->getVar('uname');
            $a_msg['rank']     = '';
            $a_msg['rank_img'] = '';
            $a_msg['avatar']   = '';
        }
        $member_handler = xoops_getHandler('member');
        $user           = $member_handler->getUser($onemsg->getVar('user_id'));
        // email
        if ($xoopsModuleConfig['showemail'] || ($onemsg->getVar('email') && (($user->getVar('user_viewemail') == 1 || $onemsg->getVar('user_id') == 0) && is_object($xoopsUser)))) {
            $a_msg['email'] =
                "<a href=\"javascript:openWithSelfMain('" . XOOPS_URL . '/modules/xfguestbook/contact.php?msg_id=' . $onemsg->getVar('msg_id') . "', 'contact', 600, 450);\"><img src=\"" . XOOPS_URL
                . "/images/icons/email.gif\" alt=\"" . _SENDEMAILTO . "\" /></a>";
        }
        // url
        if ($onemsg->getVar('url')) {
            $a_msg['url'] = '<a href="' . $onemsg->getVar('url') . '" target="_blank"><img src="' . XOOPS_URL . '/images/icons/www.gif" alt="' . _VISITWEBSITE . '"></a>';
        }
        // gender
        if ($onemsg->getVar('gender') !== '') {
            $a_msg['gender'] = '<a href="index.php?op=show_gender&param=' . $onemsg->getVar('gender') . '"><img src="assets/images/' . $onemsg->getVar('gender') . '.gif"</a>';
        }
        // flag
        if ($onemsg->getVar('country') !== '') {
            if ($onemsg->getVar('country') != 'other') {
                $flag = XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->dirname() . '/assets/images/flags/' . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . '.gif';
                if (array_key_exists($onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country'), $arr_country)) {
                    $country_name = $arr_country[$onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country')];
                } else {
                    $country_name = '';
                }
                if (file_exists($flag)) {
                    $a_msg['country'] =
                        "<img src=\"" . XOOPS_URL . '/modules/xfguestbook/assets/images/flags/' . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . ".gif\" alt=\"" . $country_name
                        . "\">";
                } else {
                    $a_msg['country'] = $country_name;
                }
                $a_msg['country'] = "<a href=\"index.php?op=show_country&param=" . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . "\">" . $a_msg['country'] . '</a>';
            } else {
                $a_msg['country'] = $onemsg->getVar('other');
            }
        }
        $a_msg['msg_id']  = $onemsg->getVar('msg_id');
        $a_msg['i']       = $nbmsg;
        $a_msg['title']   = $onemsg->getVar('title');
        $a_msg['date']    = formatTimestamp($onemsg->getVar('post_time'), 's');
        $a_msg['message'] = $onemsg->getVar('message');
        if ($options['opt_url'] == 1) {
            $a_msg['message'] = str_replace('target="_blank"', 'target="_blank" rel="nofollow"', $a_msg['message']);
        }
        $a_msg['note_msg']  = $onemsg->getVar('note');
        $a_msg['poster_ip'] = $onemsg->getVar('poster_ip');
        $a_msg['moderate']  = $onemsg->getVar('moderate');
        if (isset($country_name)) {
            $a_msg['local'] = "<a href=\"index.php?op=show_country&param=" . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . "\">" . $country_name . '</a>';
        }
        $a_msg['photo'] = $onemsg->getVar('photo');
        $xoopsTpl->append('msg', $a_msg);
        $nbmsg--;
    }
}

function xfgb_genderlist()
{
    global $options, $xoopsTpl, $xoopsModuleConfig, $xoopsModule, $msg_handler;
    $criteria = new Criteria('moderate', 0);
    $arr_msg  = $msg_handler->countMsgByGender($criteria);
    $i        = 0;
    foreach ($arr_msg as $k => $v) {
        if ($k === 'M') {
            $gender[$i] = _MD_XFGB_MALES . '<br>';
            $gender[$i] .= '<img src="assets/images/M.gif" alt="' . _MD_XFGB_MALES . '"><br><br>';
            $gender[$i] .= '<a href="index.php?op=show_gender&param=M">' . $v . _MD_XFGB_MESSAGES . '</a>';
        } elseif ($k === 'F') {
            $gender[$i] = _MD_XFGB_FEMALES . '<br>';
            $gender[$i] .= '<img src="assets/images/F.gif" alt="' . _MD_XFGB_FEMALES . '"><br><br>';
            $gender[$i] .= '<a href="index.php?op=show_gender&param=F">' . $v . _MD_XFGB_MESSAGES . '</a>';
        } else {
            $gender[$i] = _MD_XFGB_UNKNOW2 . '<br>';
            $gender[$i] .= '<img src="assets/images/U.gif"><br><br>';
            $gender[$i] .= $v . _MD_XFGB_MESSAGES;
        }
        $i++;
    }
    $xoopsTpl->assign('gender', $gender);
    $xoopsTpl->assign('display_gender', $options['opt_gender']);
}

// end functions

// if op = show_***, functions needed
//if (substr($op, 0, 4) == 'show') {
if (0 === strpos($op, 'show')) {
    $debut = isset($_GET['debut']) ? (int)$_GET['debut'] : 0;
    $param = isset($_GET['param']) ? $_GET['param'] : '';

    include_once('include/functions.php');
    $xoopsOption['template_main'] = 'xfguestbook_index.tpl';
    include_once(XOOPS_ROOT_PATH . '/header.php');
    include_once(XOOPS_ROOT_PATH . '/class/pagenav.php');
    include_once(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->dirname() . '/include/config.inc.php');
    $options = getOptions();

    $criteria = new Criteria('moderate', 0);
    $nbmsg    = $msg_handler->countMsg($criteria);

    $xoopsTpl->assign('msg_message_count', sprintf(_MD_XFGB_THEREIS, '<b>' . $nbmsg . '</b>'));
    $xoopsTpl->assign('msg_moderated', $xoopsModuleConfig['moderate']);
    $xoopsTpl->assign('msg_lang_name', $xoopsConfig['language']);
    $xoopsTpl->assign('xoops_pagetitle', $xoopsModule->name() . ' -messages');
    if ($adminview) {
        $nbwait = $msg_handler->countMsg(new Criteria('moderate', '1'));
        $xoopsTpl->assign('msg_moderate_text', sprintf(_MD_XFGB_MODERATING, "<font class='fg2'><a href='" . XOOPS_URL . "/modules/xfguestbook/index.php?op=show_waiting'>" . $nbwait . '</a></font>'));
    }
}

switch ($op) {
    case 'delete':
        if ($adminview) {
            include_once(XOOPS_ROOT_PATH . '/header.php');
            delete($msg_id);
        } else {
            redirect_header('index.php', 1, '');
        }
        break;

    case 'approve':
        if ($adminview) {
            include_once(XOOPS_ROOT_PATH . '/header.php');
            approve($msg_id);
        } else {
            redirect_header('index.php', 1, '');
        }
        break;

    case 'show_stat':
        if ($options['opt_gender'] > 0) {
            xfgb_genderlist();
        }
        break;

    case 'show_waiting':
        $pagenav = new XoopsPageNav($nbwait, $xoopsModuleConfig['perpage'], $debut, 'debut', 'op=show_waiting');
        $xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
        $criteria = new Criteria('moderate', 1);
        $criteria->setOrder('DESC');
        $criteria->setLimit($xoopsModuleConfig['perpage']);
        $criteria->setStart($debut);
        $msg =& $msg_handler->getObjects($criteria);
        $nbwait -= $debut;
        $nbmsg = $nbwait;
        xfgb_getmsg($msg);
        break;

    case 'show_one':
        if ($adminview) {
            $criteria = new Criteria('msg_id', $msg_id);
        } else {
            $criteria = new CriteriaCompo(new Criteria('moderate', '0'));
            $criteria->add(new Criteria('msg_id', $msg_id));
        }
        $msg =& $msg_handler->getObjects($criteria);
        xfgb_getmsg($msg);
        if ($options['opt_gender'] > 0) {
            xfgb_genderlist();
        }
        break;

    case 'show_country':
        list($flagdir, $country) = explode('/', $param);
        $criteria = new CriteriaCompo(new Criteria('moderate', '0'));
        if ($flagdir == $xoopsModuleConfig['flagdir']) {
            $criteria->add(new Criteria('flagdir', $flagdir));
        }
        $criteria->add(new Criteria('country', $country));
        $nbmsg   = $msg_handler->countMsg($criteria);
        $pagenav = new XoopsPageNav($nbmsg, $xoopsModuleConfig['perpage'], $debut, 'debut', 'op=show_country&param=' . $param);
        $criteria->setOrder('DESC');
        $criteria->setLimit($xoopsModuleConfig['perpage']);
        $criteria->setStart($debut);
        $msg =& $msg_handler->getObjects($criteria);
        $nbmsg -= $debut;
        $xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
        xfgb_getmsg($msg);
        break;

    case 'show_gender':
        $criteria = new CriteriaCompo(new Criteria('moderate', '0'));
        $criteria->add(new Criteria('gender', $param));
        $nbmsg   = $msg_handler->countMsg($criteria);
        $pagenav = new XoopsPageNav($nbmsg, $xoopsModuleConfig['perpage'], $debut, 'debut', 'op=show_gender&param=' . $param);
        $criteria->setOrder('DESC');
        $criteria->setLimit($xoopsModuleConfig['perpage']);
        $criteria->setStart($debut);
        $msg =& $msg_handler->getObjects($criteria);
        $nbmsg -= $debut;
        $xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
        xfgb_getmsg($msg);
        if ($options['opt_gender'] > 0) {
            xfgb_genderlist();
        }
        break;

    case 'show_all':
    default:
        $pagenav = new XoopsPageNav($nbmsg, $xoopsModuleConfig['perpage'], $debut, 'debut', '');
        $xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
        $criteria = new Criteria('moderate', 0);
        $criteria->setOrder('DESC');
        $criteria->setLimit($xoopsModuleConfig['perpage']);
        $criteria->setStart($debut);
        $msg =& $msg_handler->getObjects($criteria);
        $nbmsg -= $debut;
        xfgb_getmsg($msg);
        if ($options['opt_gender'] > 0) {
            xfgb_genderlist();
        }
        break;

    case 'cancel':
        $photos_dir     = XOOPS_UPLOAD_PATH . '/' . $xoopsModule->getVar('dirname');
        $nb_removed_tmp = xfgb_clear_tmp_files($photos_dir);
        redirect_header('index.php', 0);
        break;
}
$sql = $xoopsDB->query('SELECT * FROM ' . $xoopsDB->prefix('xfguestbook_country') . ' ORDER BY country_name ASC');

while ($coun = $xoopsDB->fetchArray($sql)) {
    $sql2 = $xoopsDB->query('SELECT COUNT(country) tot FROM ' . $xoopsDB->prefix('xfguestbook_msg') . " WHERE country='" . $coun['country_code'] . "'");
    list($tlocal) = $xoopsDB->fetchRow($sql2);
    $tlocal = $tlocal ?: '0';
    if ($tlocal > 0) {
        $opt["<a href=\"index.php?op=show_country&param=" . $xoopsModuleConfig['flagdir'] . '/' . $coun['country_code'] . "\">" . $coun['country_name'] . '</a>'] = $tlocal;
    } else {
        $opt[$coun['country_name']] = $tlocal;
    }
}
$xoopsTpl->assign('country_l', $opt);

include(XOOPS_ROOT_PATH . '/footer.php');


Pull Request — master (#10)

index.php (2 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

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		<?php
2		// $Id: index.php,v 1.3 2006/03/12 C. Felix AKA the Cat
3		// ------------------------------------------------------------------------ //
4		// XF Guestbook //
5		// ------------------------------------------------------------------------- //
6		// This program is free software; you can redistribute it and/or modify //
7		// it under the terms of the GNU General Public License as published by //
8		// the Free Software Foundation; either version 2 of the License, or //
9		// (at your option) any later version. //
10		// //
11		// You may not change or alter any portion of this comment or credits //
12		// of supporting developers from this source code or any supporting //
13		// source code which is considered copyrighted (c) material of the //
14		// original comment or credit authors. //
15		// //
16		// This program is distributed in the hope that it will be useful, //
17		// but WITHOUT ANY WARRANTY; without even the implied warranty of //
18		// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the //
19		// GNU General Public License for more details. //
20		// //
21		// You should have received a copy of the GNU General Public License //
22		// along with this program; if not, write metalslugto the Free Software //
23		// Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA //
24		// ------------------------------------------------------------------------ //
25
26		include dirname(dirname(__DIR__)) . '/mainfile.php';
27		//include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->dirname()."/class/msg.php");
28		include_once(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->dirname() . '/include/functions.php');
29	View Code Duplication	if (isset($_GET['msg_id'])) {
30		$msg_id = (int)$_GET['msg_id'];
31		} elseif (isset($_POST['msg_id'])) {
32		$msg_id = (int)$_POST['msg_id'];
33		} else {
34		$msg_id = 0;
35		}
36
37	View Code Duplication	if (isset($_GET['op'])) {
38		$op = $_GET['op'];
39		} elseif (isset($_POST['op'])) {
40		$op = $_POST['op'];
41		} else {
42		$op = 'show_all';
43		}
44
45		$msg_handler = xoops_getModuleHandler('msg');
46
47		//Admin or not
48		$xoopsUser ? $adminview = $xoopsUser->isAdmin() : $adminview = 0;
49
50		/**
51		* @param $msg_id
52		*/
53		function delete($msg_id)
54		{
55		global $msg_handler, $xoopsModule;
		0 ignored issues – show Compatibility Best Practice introduced 2016-06-14 00:00 UTC by Report Bug Copy Issue Report Show Similar Issues like this Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
56		$ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
57		if ($ok == 1) {
58		$msg = &$msg_handler->get($msg_id);
59		$del_msg_ok = $msg_handler->delete($msg);
60		$filename = $msg->getVar('photo');
61	View Code Duplication	if ($filename !== '') {
62		$filename = XOOPS_UPLOAD_PATH . '/' . $xoopsModule->getVar('dirname') . '/' . $filename;
63		unlink($filename);
64		}
65		if ($del_msg_ok) {
66		$messagesent = _MD_XFGB_MSGDELETED;
67		} else {
68		$messagesent = _MD_XFGB_ERRORDEL;
69		}
70		redirect_header('index.php', 2, $messagesent);
71		} else {
72		xoops_confirm(array('op' => 'delete', 'msg_id' => $msg_id, 'ok' => 1), 'index.php', _DELETE);
73		}
74		}
75
76		/**
77		* @param $msg_id
78		*/
79		function approve($msg_id)
80		{
81		global $msg_handler;
		0 ignored issues – show Compatibility Best Practice introduced 2016-06-14 00:00 UTC by Report Bug Copy Issue Report Show Similar Issues like this Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
82
83		$msg = &$msg_handler->get($msg_id);
84		$msg->setVar('moderate', 0);
85		if (!$msg_handler->insert($msg)) {
86		$messagesent = _MD_XFGB_ERRORVALID;
87		} else {
88		$messagesent = _MD_XFGB_VALIDATE;
89		}
90		redirect_header('index.php?op=show_waiting', 2, $messagesent);
91		}
92
93		/**
94		* @param $msg
95		*/
96		function xfgb_getmsg($msg)
97		{
98		global $nbmsg, $xoopsModule, $xoopsUser, $xoopsModuleConfig, $xoopsTpl, $xoopsConfig, $options, $opt, $xoopsDB;
99
100		$arr_country = xfgb_getAllCountry();
101		$xoopsTpl->assign('display_msg', true);
102		foreach ($msg as $onemsg) {
103		if ($poster = xfgb_get_user_data($onemsg->getVar('user_id'))) {
104		$a_msg = &$poster;
105		} else {
106		$a_msg = array();
107		$a_msg['poster'] = $onemsg->getVar('uname');
108		$a_msg['rank'] = '';
109		$a_msg['rank_img'] = '';
110		$a_msg['avatar'] = '';
111		}
112		$member_handler = xoops_getHandler('member');
113		$user = $member_handler->getUser($onemsg->getVar('user_id'));
114		// email
115		if ($xoopsModuleConfig['showemail'] \|\| ($onemsg->getVar('email') && (($user->getVar('user_viewemail') == 1 \|\| $onemsg->getVar('user_id') == 0) && is_object($xoopsUser)))) {
116		$a_msg['email'] =
117		"<a href=\"javascript:openWithSelfMain('" . XOOPS_URL . '/modules/xfguestbook/contact.php?msg_id=' . $onemsg->getVar('msg_id') . "', 'contact', 600, 450);\"><img src=\"" . XOOPS_URL
118		. "/images/icons/email.gif\" alt=\"" . _SENDEMAILTO . "\" /></a>";
119		}
120		// url
121		if ($onemsg->getVar('url')) {
122		$a_msg['url'] = '<a href="' . $onemsg->getVar('url') . '" target="_blank"><img src="' . XOOPS_URL . '/images/icons/www.gif" alt="' . _VISITWEBSITE . '"></a>';
123		}
124		// gender
125		if ($onemsg->getVar('gender') !== '') {
126		$a_msg['gender'] = '<a href="index.php?op=show_gender&param=' . $onemsg->getVar('gender') . '"><img src="assets/images/' . $onemsg->getVar('gender') . '.gif"</a>';
127		}
128		// flag
129		if ($onemsg->getVar('country') !== '') {
130		if ($onemsg->getVar('country') != 'other') {
131		$flag = XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->dirname() . '/assets/images/flags/' . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . '.gif';
132		if (array_key_exists($onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country'), $arr_country)) {
133		$country_name = $arr_country[$onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country')];
134		} else {
135		$country_name = '';
136		}
137	View Code Duplication	if (file_exists($flag)) {
138		$a_msg['country'] =
139		"<img src=\"" . XOOPS_URL . '/modules/xfguestbook/assets/images/flags/' . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . ".gif\" alt=\"" . $country_name
140		. "\">";
141		} else {
142		$a_msg['country'] = $country_name;
143		}
144		$a_msg['country'] = "<a href=\"index.php?op=show_country&param=" . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . "\">" . $a_msg['country'] . '</a>';
145		} else {
146		$a_msg['country'] = $onemsg->getVar('other');
147		}
148		}
149		$a_msg['msg_id'] = $onemsg->getVar('msg_id');
150		$a_msg['i'] = $nbmsg;
151		$a_msg['title'] = $onemsg->getVar('title');
152		$a_msg['date'] = formatTimestamp($onemsg->getVar('post_time'), 's');
153		$a_msg['message'] = $onemsg->getVar('message');
154		if ($options['opt_url'] == 1) {
155		$a_msg['message'] = str_replace('target="_blank"', 'target="_blank" rel="nofollow"', $a_msg['message']);
156		}
157		$a_msg['note_msg'] = $onemsg->getVar('note');
158		$a_msg['poster_ip'] = $onemsg->getVar('poster_ip');
159		$a_msg['moderate'] = $onemsg->getVar('moderate');
160		if (isset($country_name)) {
161		$a_msg['local'] = "<a href=\"index.php?op=show_country&param=" . $onemsg->getVar('flagdir') . '/' . $onemsg->getVar('country') . "\">" . $country_name . '</a>';
162		}
163		$a_msg['photo'] = $onemsg->getVar('photo');
164		$xoopsTpl->append('msg', $a_msg);
165		$nbmsg--;
166		}
167		}
168
169		function xfgb_genderlist()
170		{
171		global $options, $xoopsTpl, $xoopsModuleConfig, $xoopsModule, $msg_handler;
172		$criteria = new Criteria('moderate', 0);
173		$arr_msg = $msg_handler->countMsgByGender($criteria);
174		$i = 0;
175		foreach ($arr_msg as $k => $v) {
176		if ($k === 'M') {
177		$gender[$i] = _MD_XFGB_MALES . '<br>';
178		$gender[$i] .= '<img src="assets/images/M.gif" alt="' . _MD_XFGB_MALES . '"><br><br>';
179		$gender[$i] .= '<a href="index.php?op=show_gender&param=M">' . $v . _MD_XFGB_MESSAGES . '</a>';
180		} elseif ($k === 'F') {
181		$gender[$i] = _MD_XFGB_FEMALES . '<br>';
182		$gender[$i] .= '<img src="assets/images/F.gif" alt="' . _MD_XFGB_FEMALES . '"><br><br>';
183		$gender[$i] .= '<a href="index.php?op=show_gender&param=F">' . $v . _MD_XFGB_MESSAGES . '</a>';
184		} else {
185		$gender[$i] = _MD_XFGB_UNKNOW2 . '<br>';
186		$gender[$i] .= '<img src="assets/images/U.gif"><br><br>';
187		$gender[$i] .= $v . _MD_XFGB_MESSAGES;
188		}
189		$i++;
190		}
191		$xoopsTpl->assign('gender', $gender);
192		$xoopsTpl->assign('display_gender', $options['opt_gender']);
193		}
194
195		// end functions
196
197		// if op = show_***, functions needed
198		//if (substr($op, 0, 4) == 'show') {
199		if (0 === strpos($op, 'show')) {
200		$debut = isset($_GET['debut']) ? (int)$_GET['debut'] : 0;
201		$param = isset($_GET['param']) ? $_GET['param'] : '';
202
203		include_once('include/functions.php');
204		$xoopsOption['template_main'] = 'xfguestbook_index.tpl';
205		include_once(XOOPS_ROOT_PATH . '/header.php');
206		include_once(XOOPS_ROOT_PATH . '/class/pagenav.php');
207		include_once(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->dirname() . '/include/config.inc.php');
208		$options = getOptions();
209
210		$criteria = new Criteria('moderate', 0);
211		$nbmsg = $msg_handler->countMsg($criteria);
212
213		$xoopsTpl->assign('msg_message_count', sprintf(_MD_XFGB_THEREIS, '<b>' . $nbmsg . '</b>'));
214		$xoopsTpl->assign('msg_moderated', $xoopsModuleConfig['moderate']);
215		$xoopsTpl->assign('msg_lang_name', $xoopsConfig['language']);
216		$xoopsTpl->assign('xoops_pagetitle', $xoopsModule->name() . ' -messages');
217		if ($adminview) {
218		$nbwait = $msg_handler->countMsg(new Criteria('moderate', '1'));
219		$xoopsTpl->assign('msg_moderate_text', sprintf(_MD_XFGB_MODERATING, "<font class='fg2'><a href='" . XOOPS_URL . "/modules/xfguestbook/index.php?op=show_waiting'>" . $nbwait . '</a></font>'));
220		}
221		}
222
223		switch ($op) {
224	View Code Duplication	case 'delete':
225		if ($adminview) {
226		include_once(XOOPS_ROOT_PATH . '/header.php');
227		delete($msg_id);
228		} else {
229		redirect_header('index.php', 1, '');
230		}
231		break;
232
233	View Code Duplication	case 'approve':
234		if ($adminview) {
235		include_once(XOOPS_ROOT_PATH . '/header.php');
236		approve($msg_id);
237		} else {
238		redirect_header('index.php', 1, '');
239		}
240		break;
241
242		case 'show_stat':
243		if ($options['opt_gender'] > 0) {
244		xfgb_genderlist();
245		}
246		break;
247
248	View Code Duplication	case 'show_waiting':
249		$pagenav = new XoopsPageNav($nbwait, $xoopsModuleConfig['perpage'], $debut, 'debut', 'op=show_waiting');
250		$xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
251		$criteria = new Criteria('moderate', 1);
252		$criteria->setOrder('DESC');
253		$criteria->setLimit($xoopsModuleConfig['perpage']);
254		$criteria->setStart($debut);
255		$msg =& $msg_handler->getObjects($criteria);
256		$nbwait -= $debut;
257		$nbmsg = $nbwait;
258		xfgb_getmsg($msg);
259		break;
260
261		case 'show_one':
262		if ($adminview) {
263		$criteria = new Criteria('msg_id', $msg_id);
264		} else {
265		$criteria = new CriteriaCompo(new Criteria('moderate', '0'));
266		$criteria->add(new Criteria('msg_id', $msg_id));
267		}
268		$msg =& $msg_handler->getObjects($criteria);
269		xfgb_getmsg($msg);
270		if ($options['opt_gender'] > 0) {
271		xfgb_genderlist();
272		}
273		break;
274
275		case 'show_country':
276		list($flagdir, $country) = explode('/', $param);
277		$criteria = new CriteriaCompo(new Criteria('moderate', '0'));
278		if ($flagdir == $xoopsModuleConfig['flagdir']) {
279		$criteria->add(new Criteria('flagdir', $flagdir));
280		}
281		$criteria->add(new Criteria('country', $country));
282		$nbmsg = $msg_handler->countMsg($criteria);
283		$pagenav = new XoopsPageNav($nbmsg, $xoopsModuleConfig['perpage'], $debut, 'debut', 'op=show_country&param=' . $param);
284		$criteria->setOrder('DESC');
285		$criteria->setLimit($xoopsModuleConfig['perpage']);
286		$criteria->setStart($debut);
287		$msg =& $msg_handler->getObjects($criteria);
288		$nbmsg -= $debut;
289		$xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
290		xfgb_getmsg($msg);
291		break;
292
293		case 'show_gender':
294		$criteria = new CriteriaCompo(new Criteria('moderate', '0'));
295		$criteria->add(new Criteria('gender', $param));
296		$nbmsg = $msg_handler->countMsg($criteria);
297		$pagenav = new XoopsPageNav($nbmsg, $xoopsModuleConfig['perpage'], $debut, 'debut', 'op=show_gender&param=' . $param);
298		$criteria->setOrder('DESC');
299		$criteria->setLimit($xoopsModuleConfig['perpage']);
300		$criteria->setStart($debut);
301		$msg =& $msg_handler->getObjects($criteria);
302		$nbmsg -= $debut;
303		$xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
304		xfgb_getmsg($msg);
305		if ($options['opt_gender'] > 0) {
306		xfgb_genderlist();
307		}
308		break;
309
310		case 'show_all':
311	View Code Duplication	default:
312		$pagenav = new XoopsPageNav($nbmsg, $xoopsModuleConfig['perpage'], $debut, 'debut', '');
313		$xoopsTpl->assign('msg_page_nav', $pagenav->renderNav());
314		$criteria = new Criteria('moderate', 0);
315		$criteria->setOrder('DESC');
316		$criteria->setLimit($xoopsModuleConfig['perpage']);
317		$criteria->setStart($debut);
318		$msg =& $msg_handler->getObjects($criteria);
319		$nbmsg -= $debut;
320		xfgb_getmsg($msg);
321		if ($options['opt_gender'] > 0) {
322		xfgb_genderlist();
323		}
324		break;
325
326	View Code Duplication	case 'cancel':
327		$photos_dir = XOOPS_UPLOAD_PATH . '/' . $xoopsModule->getVar('dirname');
328		$nb_removed_tmp = xfgb_clear_tmp_files($photos_dir);
329		redirect_header('index.php', 0);
330		break;
331		}
332		$sql = $xoopsDB->query('SELECT * FROM ' . $xoopsDB->prefix('xfguestbook_country') . ' ORDER BY country_name ASC');
333
334		while ($coun = $xoopsDB->fetchArray($sql)) {
335		$sql2 = $xoopsDB->query('SELECT COUNT(country) tot FROM ' . $xoopsDB->prefix('xfguestbook_msg') . " WHERE country='" . $coun['country_code'] . "'");
336		list($tlocal) = $xoopsDB->fetchRow($sql2);
337		$tlocal = $tlocal ?: '0';
338		if ($tlocal > 0) {
339		$opt["<a href=\"index.php?op=show_country&param=" . $xoopsModuleConfig['flagdir'] . '/' . $coun['country_code'] . "\">" . $coun['country_name'] . '</a>'] = $tlocal;
340		} else {
341		$opt[$coun['country_name']] = $tlocal;
342		}
343		}
344		$xoopsTpl->assign('country_l', $opt);
345
346		include(XOOPS_ROOT_PATH . '/footer.php');
347

XoopsModules25x / xfguestbook

Pull Request — master (#10)

index.php (2 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

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