Give_Admin::admin_loading() - Code Metrics - Inspection of "chore: bumped version and added change log and gen..." - impress-org/give - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 76928d...4654fc )

by Devin

created 2019-03-21 04:36 UTC

Give_Admin::admin_loading() B

↳ Parent: Give_Admin

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	55
rs	8.9818
c	0
b	0
f	0

How to fix Long Method

<?php
/**
 * This class will handle file loading for the wp-admin interface.
 *
 * @package     Give
 * @subpackage  Admin
 * @copyright   Copyright (c) 2018, GiveWP
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       2.4.0
 */

/**
 * Class Give_Admin
 */
class Give_Admin {

	/**
	 * Instance.
	 *
	 * @since  2.4.0
	 * @access private
	 * @var
	 */
	static private $instance;

	/**
	 * Singleton pattern.
	 *
	 * @since  2.4.0
	 * @access private
	 */
	private function __construct() {
	}


	/**
	 * Get instance.
	 *
	 * @since  2.4.0
	 * @access public
	 * @return Give_Admin
	 */
	public static function get_instance() {

		if ( null === static::$instance ) {
			self::$instance = new static();
			self::$instance->setup();
		}

		return self::$instance;
	}

	/**
	 * Setup Admin
	 *
	 * @sinve  2.4.0
	 * @access private
	 */
	private function setup() {
		$this->admin_loading();
		$this->conditional_loading();
	}

	/**
	 *  Load core file
	 *
	 * @since  2.4.0
	 * @access private
	 */
	private function admin_loading() {
		require_once GIVE_PLUGIN_DIR . 'includes/admin/misc-functions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/setting-page-functions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/import-functions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/give-metabox-functions.php'; // @todo: [refactor] can be loaded only on the form edit screen. review possibilities

		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-footer.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/welcome.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-pages.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/class-i18n-module.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-actions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-filters.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/add-ons.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/plugins.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/dashboard-widgets.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/class-blank-slate.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/actions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/payments-history.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donors.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-functions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-actions.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/metabox.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/class-give-form-duplicator.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/class-metabox-form-data.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/dashboard-columns.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-functions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-export.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-actions.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/give-export-donations-functions.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/reports.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/class-give-graph.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/graphing.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/logs.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/tools-actions.php';

		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/abstract-shortcode-generator.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/class-shortcode-button.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-form.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-goal.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-login.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-register.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-profile-editor.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donation-grid.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donation-history.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-receipt.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-totals.php';
		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donor-wall.php';
	}

	/**
	 *  Load file conditionally
	 *
	 * @since  2.4.0
	 * @access private
	 */
	private function conditional_loading() {
		if ( $this->is_generate_pdf() ) {
			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/pdf-reports.php';
		}
	}

	/**
	 * Check if performing action 'generate_pdf'
	 *
	 * @since  2.4.0
	 * @access private
	 *
	 * @return bool
	 */
	private function is_generate_pdf() {
		return isset( $_GET['give-action'] ) && 'generate_pdf' === give_clean( $_GET['give-action'] );

	}
}

Give_Admin::get_instance();


1		<?php
2		/**
3		* This class will handle file loading for the wp-admin interface.
4		*
5		* @package Give
6		* @subpackage Admin
7		* @copyright Copyright (c) 2018, GiveWP
8		* @license https://opensource.org/licenses/gpl-license GNU Public License
9		* @since 2.4.0
10		*/
11
12		/**
13		* Class Give_Admin
14		*/
15		class Give_Admin {
		0 ignored issues – show Coding Style introduced 2019-01-17 04:28 UTC by Report Bug Copy Issue Report Since you have declared the constructor as private, maybe you should also declare the class as final. Loading history...
16		/**
17		* Instance.
18		*
19		* @since 2.4.0
20		* @access private
21		* @var
22		*/
23		static private $instance;
24
25		/**
26		* Singleton pattern.
27		*
28		* @since 2.4.0
29		* @access private
30		*/
31		private function __construct() {
32		}
33
34
35		/**
36		* Get instance.
37		*
38		* @since 2.4.0
39		* @access public
40		* @return Give_Admin
41		*/
42	View Code Duplication	public static function get_instance() {
		0 ignored issues – show Duplication introduced 2018-10-24 21:23 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
43		if ( null === static::$instance ) {
44		self::$instance = new static();
45		self::$instance->setup();
46		}
47
48		return self::$instance;
49		}
50
51		/**
52		* Setup Admin
53		*
54		* @sinve 2.4.0
55		* @access private
56		*/
57		private function setup() {
58		$this->admin_loading();
59		$this->conditional_loading();
60		}
61
62		/**
63		* Load core file
64		*
65		* @since 2.4.0
66		* @access private
67		*/
68		private function admin_loading() {
69		require_once GIVE_PLUGIN_DIR . 'includes/admin/misc-functions.php';
70		require_once GIVE_PLUGIN_DIR . 'includes/admin/setting-page-functions.php';
71		require_once GIVE_PLUGIN_DIR . 'includes/admin/import-functions.php';
72		require_once GIVE_PLUGIN_DIR . 'includes/admin/give-metabox-functions.php'; // @todo: [refactor] can be loaded only on the form edit screen. review possibilities
73
74		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-footer.php';
75		require_once GIVE_PLUGIN_DIR . 'includes/admin/welcome.php';
76		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-pages.php';
77		require_once GIVE_PLUGIN_DIR . 'includes/admin/class-i18n-module.php';
78		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-actions.php';
79		require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-filters.php';
80		require_once GIVE_PLUGIN_DIR . 'includes/admin/add-ons.php';
81		require_once GIVE_PLUGIN_DIR . 'includes/admin/plugins.php';
82		require_once GIVE_PLUGIN_DIR . 'includes/admin/dashboard-widgets.php';
83		require_once GIVE_PLUGIN_DIR . 'includes/admin/class-blank-slate.php';
84
85		require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/actions.php';
86		require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/payments-history.php';
87
88		require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donors.php';
89		require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-functions.php';
90		require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-actions.php';
91
92		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/metabox.php';
93		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/class-give-form-duplicator.php';
94		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/class-metabox-form-data.php';
95		require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/dashboard-columns.php';
96
97		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-functions.php';
98		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-export.php';
99		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-actions.php';
100		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/give-export-donations-functions.php';
101
102		require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/reports.php';
103		require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/class-give-graph.php';
104		require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/graphing.php';
105
106		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/logs.php';
107
108		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/tools-actions.php';
109
110		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/abstract-shortcode-generator.php';
111		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/class-shortcode-button.php';
112		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-form.php';
113		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-goal.php';
114		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-login.php';
115		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-register.php';
116		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-profile-editor.php';
117		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donation-grid.php';
118		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donation-history.php';
119		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-receipt.php';
120		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-totals.php';
121		require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donor-wall.php';
122		}
123
124		/**
125		* Load file conditionally
126		*
127		* @since 2.4.0
128		* @access private
129		*/
130		private function conditional_loading() {
131		if ( $this->is_generate_pdf() ) {
132		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/pdf-reports.php';
133		}
134		}
135
136		/**
137		* Check if performing action 'generate_pdf'
138		*
139		* @since 2.4.0
140		* @access private
141		*
142		* @return bool
143		*/
144		private function is_generate_pdf() {
145		return isset( $_GET['give-action'] ) && 'generate_pdf' === give_clean( $_GET['give-action'] );
		0 ignored issues – show introduced 2019-01-17 04:28 UTC by Report Bug Copy Issue Report Detected access of super global var $_GET, probably need manual inspection. Loading history... introduced 2019-01-17 04:28 UTC by Report Bug Copy Issue Report Detected usage of a non-sanitized input variable: $_GET Loading history...
146		}
147		}
148
149		Give_Admin::get_instance();
150

impress-org / give

Push — master ( 76928d...4654fc )

Give_Admin::admin_loading() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like