Issues in print.php - All Issues - Inspection of "Update main.yaml" - mambax7/adslight - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b97f58...ccd955 )

by Michael

created 2023-01-11 05:01 UTC

print.php (1 issue)

Labels

Unused Code 1

Severity

Minor 1

<?php

declare(strict_types=1);

/*
 * 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.
 */

/**
 * @copyright    XOOPS Project (https://xoops.org)
 * @license      GNU GPL 2 or later (https://www.gnu.org/licenses/gpl-2.0.html)
 * @author       XOOPS Development Team
 * @author       Pascal Le Boustouller: original author ([email protected])
 * @author       Luc Bizet (www.frxoops.org)
 * @author       jlm69 (www.jlmzone.com)
 * @author       mamba (www.xoops.org)
 */

use Xmf\Request;
use XoopsModules\Adslight\{
    Helper,
    Utility
};

/** @var Helper $helper */

require_once __DIR__ . '/header.php';
//require_once XOOPS_ROOT_PATH . '/modules/adslight/include/gtickets.php';

/**
 * @param $lid
 */
function printAd($lid): void
{
    global $xoopsConfig, $xoopsDB, $useroffset, $myts;
    $helper = Helper::getInstance();
    $currenttheme = $xoopsConfig['theme_set'];
    $lid          = (int)$lid;

    $result = $xoopsDB->query(
        'SELECT l.lid, l.title, l.expire, l.type, l.desctext, l.tel, l.price, l.typeprice, l.date_created, l.email, l.submitter, l.town, l.country, l.photo, p.cod_img, p.lid, p.uid_owner, p.url FROM '
        . $xoopsDB->prefix('adslight_listing')
        . ' l LEFT JOIN '
        . $xoopsDB->prefix('adslight_pictures')
        . ' p ON l.lid=p.lid WHERE l.lid='
        . $xoopsDB->escape($lid)
    );
    [$lid, $title, $expire, $type, $desctext, $tel, $price, $typeprice, $date_created, $email, $submitter, $town, $country, $photo, $cod_img, $pic_lid, $uid_owner, $url] = $xoopsDB->fetchRow($result);

    $title     = \htmlspecialchars($title, ENT_QUOTES | ENT_HTML5);
    $expire    = \htmlspecialchars($expire, ENT_QUOTES | ENT_HTML5);
    $type      = Utility::getNameType(htmlspecialchars($type, ENT_QUOTES | ENT_HTML5));
    $desctext  = $myts->displayTarea($desctext, 1, 1, 1, 1, 1);
    $tel       = \htmlspecialchars($tel, ENT_QUOTES | ENT_HTML5);
    $price     = \htmlspecialchars($price, ENT_QUOTES | ENT_HTML5);
    $typeprice = \htmlspecialchars($typeprice, ENT_QUOTES | ENT_HTML5);
    $submitter = \htmlspecialchars($submitter, ENT_QUOTES | ENT_HTML5);
    $town      = \htmlspecialchars($town, ENT_QUOTES | ENT_HTML5);
    $country   = \htmlspecialchars($country, ENT_QUOTES | ENT_HTML5);

    echo '
    <html>
    <head><title>' . $xoopsConfig['sitename'] . "</title>
    <meta http-equiv=\"Content-Type\" content=\"text/html; charset=utf-8\" >
    <meta http-equiv=\”robots\” content=\"noindex, nofollow, noarchive\" >
    <link rel=\"StyleSheet\" href=\"../../themes/" . $currenttheme . '/style/style.css" type="text/css">
    </head>
    <body bgcolor="#FFFFFF" text="#000000">
    <table border=0><tr><td>
    <table border=0 width=100% cellpadding=0 cellspacing=1 bgcolor="#000000"><tr><td>
    <table border=0 width=100% cellpadding=15 cellspacing=1 bgcolor="#FFFFFF"><tr><td>';

    $useroffset = 0;
    if ($GLOBALS['xoopsUser'] instanceof \XoopsUser) {
        $timezone   = $GLOBALS['xoopsUser']->timezone();
        $useroffset = empty($timezone) ? $xoopsConfig['default_TZ'] : $GLOBALS['xoopsUser']->timezone();
    }
    $date_created = ($useroffset * 3600) + $date_created;
    $date2        = $date_created + ($expire * 86400);
    $date1        = formatTimestamp($date_created, 's');

    $date2        = formatTimestamp($date2, 's');

    echo '<br><br><table width=99% border=0>
        <tr>
      <td>' . _ADSLIGHT_CLASSIFIED . " (No. ${lid} ) <br>" . _ADSLIGHT_FROM . " ${submitter} <br><br>";

    echo " <strong>${type} :</strong> <i>${title}</i><br>";
    if ($price > 0) {
        echo '<strong>' . _ADSLIGHT_PRICE2 . "</strong> ${price} " . $helper->getConfig('adslight_currency_symbol') . "  - ${typeprice}<br>";
    }
    if ($photo) {
        echo "<tr><td><div style='text-align:left'><img class=\"thumb\" src=\"" . XOOPS_URL . "/uploads/adslight/${url}\" width=\"130px\" border=0 ></div>";
    }
    echo '</td>
          </tr>
    <tr>
      <td><strong>' . _ADSLIGHT_DESC . "</strong><br><br><div style=\"text-align:justify;\">${desctext}</div><p>";
    if ('' !== $tel) {
        echo '<br><strong>' . _ADSLIGHT_TEL . "</strong> ${tel}";
    }
    if ('' !== $town) {
        echo '<br><strong>' . _ADSLIGHT_TOWN . "</strong> ${town}";
    }
    if ('' !== $country) {
        echo '<br><strong>' . _ADSLIGHT_COUNTRY . "</strong> ${country}";
    }
    echo '<hr>';
    echo '' . _ADSLIGHT_NOMAIL . ' <br>' . XOOPS_URL . '/modules/adslight/viewads.php?lid=' . $lid . '<br>';
    echo '<br><br>' . _ADSLIGHT_DATE2 . " ${date_created} " . _ADSLIGHT_AND . ' ' . _ADSLIGHT_DISPO . " ${date2}<br><br>";
    echo '</td>
    </tr>
    </table>';
    echo '<br><br></td></tr></table></td></tr></table>
    <br><br><div style="text-align:center">
    ' . _ADSLIGHT_EXTRANN . ' <strong>' . $xoopsConfig['sitename'] . '</strong></div><br>
    <a href="' . XOOPS_URL . '/modules/adslight/">' . XOOPS_URL . '/modules/adslight/</a>
    </td></tr></table>
    </body>
    </html>';
}

##############################################################

$lid = Request::getInt('lid', 0);
$op  = Request::getString('op', '');

switch ($op) {
    case 'PrintAd':
        printAd($lid);
        break;
    default:
        $helper->redirect('index.php', 3, ' ' . _RETURNANN . ' ');
        break;
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* You may not change or alter any portion of this comment or credits
7			* of supporting developers from this source code or any supporting source code
8			* which is considered copyrighted (c) material of the original comment or credit authors.
9			*
10			* This program is distributed in the hope that it will be useful,
11			* but WITHOUT ANY WARRANTY; without even the implied warranty of
12			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
13			*/
14
15			/**
16			* @copyright XOOPS Project (https://xoops.org)
17			* @license GNU GPL 2 or later (https://www.gnu.org/licenses/gpl-2.0.html)
18			* @author XOOPS Development Team
19			* @author Pascal Le Boustouller: original author ([email protected])
20			* @author Luc Bizet (www.frxoops.org)
21			* @author jlm69 (www.jlmzone.com)
22			* @author mamba (www.xoops.org)
23			*/
24
25			use Xmf\Request;
26			use XoopsModules\Adslight\{
27			Helper,
28			Utility
29			};
30
31			/** @var Helper $helper */
32
33			require_once __DIR__ . '/header.php';
34			//require_once XOOPS_ROOT_PATH . '/modules/adslight/include/gtickets.php';
35
36			/**
37			* @param $lid
38			*/
39			function printAd($lid): void
40			{
41			global $xoopsConfig, $xoopsDB, $useroffset, $myts;
42			$helper = Helper::getInstance();
43			$currenttheme = $xoopsConfig['theme_set'];
44			$lid = (int)$lid;
45
46			$result = $xoopsDB->query(
47			'SELECT l.lid, l.title, l.expire, l.type, l.desctext, l.tel, l.price, l.typeprice, l.date_created, l.email, l.submitter, l.town, l.country, l.photo, p.cod_img, p.lid, p.uid_owner, p.url FROM '
48			. $xoopsDB->prefix('adslight_listing')
49			. ' l LEFT JOIN '
50			. $xoopsDB->prefix('adslight_pictures')
51			. ' p ON l.lid=p.lid WHERE l.lid='
52			. $xoopsDB->escape($lid)
53			);
54			[$lid, $title, $expire, $type, $desctext, $tel, $price, $typeprice, $date_created, $email, $submitter, $town, $country, $photo, $cod_img, $pic_lid, $uid_owner, $url] = $xoopsDB->fetchRow($result);
55
56			$title = \htmlspecialchars($title, ENT_QUOTES \| ENT_HTML5);
57			$expire = \htmlspecialchars($expire, ENT_QUOTES \| ENT_HTML5);
58			$type = Utility::getNameType(htmlspecialchars($type, ENT_QUOTES \| ENT_HTML5));
59			$desctext = $myts->displayTarea($desctext, 1, 1, 1, 1, 1);
60			$tel = \htmlspecialchars($tel, ENT_QUOTES \| ENT_HTML5);
61			$price = \htmlspecialchars($price, ENT_QUOTES \| ENT_HTML5);
62			$typeprice = \htmlspecialchars($typeprice, ENT_QUOTES \| ENT_HTML5);
63			$submitter = \htmlspecialchars($submitter, ENT_QUOTES \| ENT_HTML5);
64			$town = \htmlspecialchars($town, ENT_QUOTES \| ENT_HTML5);
65			$country = \htmlspecialchars($country, ENT_QUOTES \| ENT_HTML5);
66
67			echo '
68			<html>
69			<head><title>' . $xoopsConfig['sitename'] . "</title>
70			<meta http-equiv=\"Content-Type\" content=\"text/html; charset=utf-8\" >
71			<meta http-equiv=\”robots\” content=\"noindex, nofollow, noarchive\" >
72			<link rel=\"StyleSheet\" href=\"../../themes/" . $currenttheme . '/style/style.css" type="text/css">
73			</head>
74			<body bgcolor="#FFFFFF" text="#000000">
75			<table border=0><tr><td>
76			<table border=0 width=100% cellpadding=0 cellspacing=1 bgcolor="#000000"><tr><td>
77			<table border=0 width=100% cellpadding=15 cellspacing=1 bgcolor="#FFFFFF"><tr><td>';
78
79			$useroffset = 0;
80			if ($GLOBALS['xoopsUser'] instanceof \XoopsUser) {
81			$timezone = $GLOBALS['xoopsUser']->timezone();
82			$useroffset = empty($timezone) ? $xoopsConfig['default_TZ'] : $GLOBALS['xoopsUser']->timezone();
83			}
84			$date_created = ($useroffset * 3600) + $date_created;
85			$date2 = $date_created + ($expire * 86400);
86			$date1 = formatTimestamp($date_created, 's');
			0 ignored issues – show Unused Code introduced 2021-08-14 04:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this The assignment to `$date1` is dead and can be removed. Loading history...
87			$date2 = formatTimestamp($date2, 's');
88
89			echo '<br><br><table width=99% border=0>
90			<tr>
91			<td>' . _ADSLIGHT_CLASSIFIED . " (No. ${lid} ) <br>" . _ADSLIGHT_FROM . " ${submitter} <br><br>";
92
93			echo " <strong>${type} :</strong> <i>${title}</i><br>";
94			if ($price > 0) {
95			echo '<strong>' . _ADSLIGHT_PRICE2 . "</strong> ${price} " . $helper->getConfig('adslight_currency_symbol') . " - ${typeprice}<br>";
96			}
97			if ($photo) {
98			echo "<tr><td><div style='text-align:left'><img class=\"thumb\" src=\"" . XOOPS_URL . "/uploads/adslight/${url}\" width=\"130px\" border=0 ></div>";
99			}
100			echo '</td>
101			</tr>
102			<tr>
103			<td><strong>' . _ADSLIGHT_DESC . "</strong><br><br><div style=\"text-align:justify;\">${desctext}</div><p>";
104			if ('' !== $tel) {
105			echo '<br><strong>' . _ADSLIGHT_TEL . "</strong> ${tel}";
106			}
107			if ('' !== $town) {
108			echo '<br><strong>' . _ADSLIGHT_TOWN . "</strong> ${town}";
109			}
110			if ('' !== $country) {
111			echo '<br><strong>' . _ADSLIGHT_COUNTRY . "</strong> ${country}";
112			}
113			echo '<hr>';
114			echo '' . _ADSLIGHT_NOMAIL . ' <br>' . XOOPS_URL . '/modules/adslight/viewads.php?lid=' . $lid . '<br>';
115			echo '<br><br>' . _ADSLIGHT_DATE2 . " ${date_created} " . _ADSLIGHT_AND . ' ' . _ADSLIGHT_DISPO . " ${date2}<br><br>";
116			echo '</td>
117			</tr>
118			</table>';
119			echo '<br><br></td></tr></table></td></tr></table>
120			<br><br><div style="text-align:center">
121			' . _ADSLIGHT_EXTRANN . ' <strong>' . $xoopsConfig['sitename'] . '</strong></div><br>
122			<a href="' . XOOPS_URL . '/modules/adslight/">' . XOOPS_URL . '/modules/adslight/</a>
123			</td></tr></table>
124			</body>
125			</html>';
126			}
127
128			##############################################################
129
130			$lid = Request::getInt('lid', 0);
131			$op = Request::getString('op', '');
132
133			switch ($op) {
134			case 'PrintAd':
135			printAd($lid);
136			break;
137			default:
138			$helper->redirect('index.php', 3, ' ' . _RETURNANN . ' ');
139			break;
140			}
141

mambax7 / adslight

Push — master ( b97f58...ccd955 )

print.php (1 issue)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like