main_module::main() - Code Metrics - Inspection of "RSI v1.0.0" - Dark1z/reducesearchindex - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#1)

by Dark❶

created 2020-03-08 17:35 UTC

main_module::main() A

↳ Parent: main_module

Complexity

Conditions	1
Paths	1

Size

Total Lines	21
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	7
c	1
b	0
f	0
nc	1
nop	2
dl	0
loc	21
rs	10

<?php
/**
 *
 * Reduce Search Index [RSI]. An extension for the phpBB Forum Software package.
 *
 * @copyright (c) 2020, Dark❶, https://dark1.tech
 * @license GNU General Public License, version 2 (GPL-2.0)
 *
 */

namespace dark1\reducesearchindex\acp;

/**
 * Reduce Search Index [RSI] ACP module.
 */
class main_module
{
	public $page_title;
	public $tpl_name;
	public $u_action;

	/**
	 * Main ACP module
	 *
	 * @param int    $id   The module ID
	 * @param string $mode The module mode
	 * @throws \Exception
	 */
	public function main($id, $mode)
	{
		global $phpbb_container;

		/** @var \dark1\reducesearchindex\controller\acp_controller $acp_controller */
		$acp_controller = $phpbb_container->get('dark1.reducesearchindex.controller.acp');

		// Load the display handle in our ACP controller
		$acp_controller->set_data($id, $mode, $this->u_action);

		// Get data from our ACP controller
		$acp_get_data = $acp_controller->get_data();
class A
{
    function getObject()
    {
        return null;
    }

}

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

		// Load a template from adm/style for our ACP page
		$this->tpl_name = $acp_get_data['tpl_name'];

		// Set the page title for our ACP page
		$this->page_title = $acp_get_data['page_title'];

		// Load the display handle in our ACP controller
		$acp_controller->display();
	}
}


1			<?php
2			/**
3			*
4			* Reduce Search Index [RSI]. An extension for the phpBB Forum Software package.
5			*
6			* @copyright (c) 2020, Dark❶, https://dark1.tech
7			* @license GNU General Public License, version 2 (GPL-2.0)
8			*
9			*/
10
11			namespace dark1\reducesearchindex\acp;
12
13			/**
14			* Reduce Search Index [RSI] ACP module.
15			*/
16			class main_module
17			{
18			public $page_title;
19			public $tpl_name;
20			public $u_action;
21
22			/**
23			* Main ACP module
24			*
25			* @param int $id The module ID
26			* @param string $mode The module mode
27			* @throws \Exception
28			*/
29			public function main($id, $mode)
30			{
31			global $phpbb_container;
32
33			/** @var \dark1\reducesearchindex\controller\acp_controller $acp_controller */
34			$acp_controller = $phpbb_container->get('dark1.reducesearchindex.controller.acp');
35
36			// Load the display handle in our ACP controller
37			$acp_controller->set_data($id, $mode, $this->u_action);
38
39			// Get data from our ACP controller
40			$acp_get_data = $acp_controller->get_data();
			0 ignored issues – show Bug introduced 2020-03-08 17:38 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$acp_get_data` is correct as `$acp_controller->get_data()` targeting `dark1\reducesearchindex\..._controller::get_data()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
41
42			// Load a template from adm/style for our ACP page
43			$this->tpl_name = $acp_get_data['tpl_name'];
44
45			// Set the page title for our ACP page
46			$this->page_title = $acp_get_data['page_title'];
47
48			// Load the display handle in our ACP controller
49			$acp_controller->display();
50			}
51			}
52

Dark1z / reducesearchindex

Pull Request — master (#1)

main_module::main() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like