Issues in print.php - New Issues - Inspection of "updates" - mambax7/smartfaq - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 2744eb...81ba93 )

by Michael

created 2018-04-01 09:45 UTC

print.php (1 issue)

Labels

Bug 1

Severity

Major 1

<?php

/**
 * Module: SmartFAQ
 * Author: The SmartFactory <www.smartfactory.ca>
 * Licence: GNU
 */

use XoopsModules\Smartfaq;

require_once __DIR__ . '/header.php';

$faqid = \Xmf\Request::getInt('faqid', 0, 'GET');
filter:
    dependency_paths: ["lib/*"]

if (0 == $faqid) {
    redirect_header('javascript:history.go(-1)', 1, _MD_SF_NOFAQSELECTED);
}

// Creating the FAQ handler object
/** @var \XoopsModules\Smartfaq\FaqHandler $faqHandler */
$faqHandler = \XoopsModules\Smartfaq\Helper::getInstance()->getHandler('Faq');

// Creating the FAQ object for the selected FAQ
$faqObj = new Smartfaq\Faq($faqid);

// If the selected FAQ was not found, exit
if ($faqObj->notLoaded()) {
    redirect_header('javascript:history.go(-1)', 1, _MD_SF_NOFAQSELECTED);
}

// Creating the category object that holds the selected FAQ
$categoryObj = $faqObj->category();

// Creating the answer object
$answerObj = $faqObj->answer();

// Check user permissions to access that category of the selected FAQ
if (Smartfaq\Utility::faqAccessGranted($faqObj) < 0) {
    redirect_header('javascript:history.go(-1)', 1, _NOPERM);
}

global $xoopsConfig, $xoopsDB, $xoopsModule, $myts;

$who_where = $faqObj->getWhoAndWhen();
$comeFrom  = $faqObj->getComeFrom();

echo "<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN'>\n";
echo "<html>\n<head>\n";
echo '<title>' . _MD_SF_FAQCOMEFROM . ' ' . $xoopsConfig['sitename'] . "</title>\n";
echo "<meta http-equiv='Content-Type' content='text/html; charset=" . _CHARSET . "'>\n";
echo "<meta name='AUTHOR' content='" . $xoopsConfig['sitename'] . "'>\n";
echo "<meta name='COPYRIGHT' content='Copyright (c) 2001 by " . $xoopsConfig['sitename'] . "'>\n";
echo "<meta name='DESCRIPTION' content='" . $xoopsConfig['slogan'] . "'>\n";
echo "<meta name='GENERATOR' content='" . XOOPS_VERSION . "'>\n\n\n";

echo "<body bgcolor='#ffffff' text='#000000' onload='window.print()'>
     <div style='width: 650px; border: 1px solid #000; padding: 20px;'>
     <div style='text-align: center; display: block; margin: 0 0 6px 0;'><img src='" . XOOPS_URL . "/modules/smartfaq/assets/images/logo_module.png' border='0' alt=''><h2 style='margin: 0;'>" . $faqObj->question() . "</h2></div>
     <div align='center'>" . $who_where . "</div>
                <div style='text-align: center; display: block; padding-bottom: 12px; margin: 0 0 6px 0; border-bottom: 2px solid #ccc;'></div>
                <div></div>
                <b><p>" . $faqObj->question() . '</p></b>
                <p>' . $answerObj->answer() . "</p>
                <div style='padding-top: 12px; border-top: 2px solid #ccc;'></div>
                <p>" . $comeFrom . '</p>
            </div>
    <br>';

echo '<br>
          </body>
          </html>';


1			<?php
2
3			/**
4			* Module: SmartFAQ
5			* Author: The SmartFactory <www.smartfactory.ca>
6			* Licence: GNU
7			*/
8
9			use XoopsModules\Smartfaq;
10
11			require_once __DIR__ . '/header.php';
12
13			$faqid = \Xmf\Request::getInt('faqid', 0, 'GET');
			0 ignored issues – show Bug introduced 2018-04-01 09:47 UTC by Report Bug Copy Issue Report Show Similar Issues like this The type `Xmf\Request` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
14
15			if (0 == $faqid) {
16			redirect_header('javascript:history.go(-1)', 1, _MD_SF_NOFAQSELECTED);
17			}
18
19			// Creating the FAQ handler object
20			/** @var \XoopsModules\Smartfaq\FaqHandler $faqHandler */
21			$faqHandler = \XoopsModules\Smartfaq\Helper::getInstance()->getHandler('Faq');
22
23			// Creating the FAQ object for the selected FAQ
24			$faqObj = new Smartfaq\Faq($faqid);
25
26			// If the selected FAQ was not found, exit
27			if ($faqObj->notLoaded()) {
28			redirect_header('javascript:history.go(-1)', 1, _MD_SF_NOFAQSELECTED);
29			}
30
31			// Creating the category object that holds the selected FAQ
32			$categoryObj = $faqObj->category();
33
34			// Creating the answer object
35			$answerObj = $faqObj->answer();
36
37			// Check user permissions to access that category of the selected FAQ
38			if (Smartfaq\Utility::faqAccessGranted($faqObj) < 0) {
39			redirect_header('javascript:history.go(-1)', 1, _NOPERM);
40			}
41
42			global $xoopsConfig, $xoopsDB, $xoopsModule, $myts;
43
44			$who_where = $faqObj->getWhoAndWhen();
45			$comeFrom = $faqObj->getComeFrom();
46
47			echo "<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN'>\n";
48			echo "<html>\n<head>\n";
49			echo '<title>' . _MD_SF_FAQCOMEFROM . ' ' . $xoopsConfig['sitename'] . "</title>\n";
50			echo "<meta http-equiv='Content-Type' content='text/html; charset=" . _CHARSET . "'>\n";
51			echo "<meta name='AUTHOR' content='" . $xoopsConfig['sitename'] . "'>\n";
52			echo "<meta name='COPYRIGHT' content='Copyright (c) 2001 by " . $xoopsConfig['sitename'] . "'>\n";
53			echo "<meta name='DESCRIPTION' content='" . $xoopsConfig['slogan'] . "'>\n";
54			echo "<meta name='GENERATOR' content='" . XOOPS_VERSION . "'>\n\n\n";
55
56			echo "<body bgcolor='#ffffff' text='#000000' onload='window.print()'>
57			<div style='width: 650px; border: 1px solid #000; padding: 20px;'>
58			<div style='text-align: center; display: block; margin: 0 0 6px 0;'><img src='" . XOOPS_URL . "/modules/smartfaq/assets/images/logo_module.png' border='0' alt=''><h2 style='margin: 0;'>" . $faqObj->question() . "</h2></div>
59			<div align='center'>" . $who_where . "</div>
60			<div style='text-align: center; display: block; padding-bottom: 12px; margin: 0 0 6px 0; border-bottom: 2px solid #ccc;'></div>
61			<div></div>
62			<b><p>" . $faqObj->question() . '</p></b>
63			<p>' . $answerObj->answer() . "</p>
64			<div style='padding-top: 12px; border-top: 2px solid #ccc;'></div>
65			<p>" . $comeFrom . '</p>
66			</div>
67			<br>';
68
69			echo '<br>
70			</body>
71			</html>';
72

mambax7 / smartfaq

Push — master ( 2744eb...81ba93 )

print.php (1 issue)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like