MapsHooks::onExtensionFunction() - Code Metrics - Inspection of "[DO NOT MERGE] Add support for extension.json" - JeroenDeDauw/Maps - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#131)

by None

created 2017-06-02 15:54 UTC

MapsHooks::onExtensionFunction() D

↳ Parent: MapsHooks

Complexity

Conditions	10
Paths	27

Size

Total Lines	96
Code Lines	51

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	96
rs	4.9494
c	0
b	0
f	0
cc	10
eloc	51
nc	27
nop	0

How to fix Long Method Complexity

<?php

use DataValues\Geo\Parsers\GeoCoordinateParser;
use Maps\ServiceParam;
use Maps\LocationParser;
use Maps\LineParser;
use Maps\CircleParser;
use Maps\RectangleParser;
use Maps\PolygonParser;
use Maps\DistanceParser;
use Maps\WmsOverlayParser;
use Maps\ImageOverlayParser;

/**
 * Static class for hooks handled by the Maps extension.
 *
 * @since 0.7
 *
 * @licence GNU GPL v2+
 * @author Jeroen De Dauw < [email protected] >
 */
final class MapsHooks {

 	public static function onExtensionCallback() {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
		if ( defined( 'Maps_COORDS_FLOAT' ) ) {
			// Do not initialize more than once.
			return 1;
		}

		// The different coordinate notations.
		define( 'Maps_COORDS_FLOAT' , 'float' );
		define( 'Maps_COORDS_DMS' , 'dms' );
		define( 'Maps_COORDS_DM' , 'dm' );
		define( 'Maps_COORDS_DD' , 'dd' );

		require_once __DIR__ . '/Maps_Settings.php';

		define( 'Maps_VERSION' , '4.2.1' );
		define( 'SM_VERSION', Maps_VERSION );

		// Only initialize the extension when all dependencies are present.
		if ( !defined( 'Validator_VERSION' ) ) {
			throw new Exception( 'You need to have Validator installed in order to use Maps' );
		}

		$GLOBALS['egMapsStyleVersion'] = $GLOBALS['wgStyleVersion'] . '-' . Maps_VERSION;
	}

	public static function onExtensionFunction() {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
		if ( $GLOBALS['egMapsDisableExtension'] ) {
			return true;
		}
		if ( defined( 'Maps_VERSION' ) ) {
			// Do not initialize more than once.
			return true;
		}

		// Only initialize the extension when all dependencies are present.
		if ( !defined( 'Validator_VERSION' ) ) {
			throw new Exception( 'You need to have Validator installed in order to use Maps' );
		}

		define( 'Maps_VERSION' , '4.2.1' );
		define( 'SM_VERSION', Maps_VERSION );

		if ( $GLOBALS['egMapsGMaps3Language'] === '' ) {
			$GLOBALS['egMapsGMaps3Language'] = $GLOBALS['wgLang'];
		}

		MapsMappingServices::registerService( 'googlemaps3', MapsGoogleMaps3::class );

		$googleMaps = MapsMappingServices::getServiceInstance( 'googlemaps3' );
		$googleMaps->addFeature( 'display_map', MapsDisplayMapRenderer::class );

		MapsMappingServices::registerService(
			'openlayers',
			MapsOpenLayers::class,
			[ 'display_map' => MapsDisplayMapRenderer::class ]
		);

		MapsMappingServices::registerService( 'leaflet', MapsLeaflet::class );
		$leafletMaps = MapsMappingServices::getServiceInstance( 'leaflet' );
		$leafletMaps->addFeature( 'display_map', MapsDisplayMapRenderer::class );

		if ( in_array( 'googlemaps3', $GLOBALS['egMapsAvailableServices'] ) ) {
			$GLOBALS['wgSpecialPages']['MapEditor'] = 'SpecialMapEditor';
			$GLOBALS['wgSpecialPageGroups']['MapEditor'] = 'maps';
		}

		$GLOBALS['egMapsStyleVersion'] = $GLOBALS['wgStyleVersion'] . '-' . Maps_VERSION;

		// Users that can geocode. By default the same as those that can edit.
		foreach ( $GLOBALS['wgGroupPermissions'] as $group => $rights ) {
			if ( array_key_exists( 'edit' , $rights ) ) {
				$GLOBALS['wgGroupPermissions'][$group]['geocode'] = $GLOBALS['wgGroupPermissions'][$group]['edit'];
			}
		}

		$GLOBALS['wgParamDefinitions']['coordinate'] = [
			'string-parser' => GeoCoordinateParser::class,
		];

		$GLOBALS['wgParamDefinitions']['mappingservice'] = [
			'definition'=> ServiceParam::class,
		];

		$GLOBALS['wgParamDefinitions']['mapslocation'] = [
			'string-parser' => LocationParser::class,
		];

		$GLOBALS['wgParamDefinitions']['mapsline'] = [
			'string-parser' => LineParser::class,
		];

		$GLOBALS['wgParamDefinitions']['mapscircle'] = [
			'string-parser' => CircleParser::class,
		];

		$GLOBALS['wgParamDefinitions']['mapsrectangle'] = [
			'string-parser' => RectangleParser::class,
		];

		$GLOBALS['wgParamDefinitions']['mapspolygon'] = [
			'string-parser' => PolygonParser::class,
		];

		$GLOBALS['wgParamDefinitions']['distance'] = [
			'string-parser' => DistanceParser::class,
		];

		$GLOBALS['wgParamDefinitions']['wmsoverlay'] = [
			'string-parser' => WmsOverlayParser::class,
		];

		$GLOBALS['wgParamDefinitions']['mapsimageoverlay'] = [
			'string-parser' => ImageOverlayParser::class,
		];

		if ( !$GLOBALS['egMapsDisableSmwIntegration'] && defined( 'SMW_VERSION' ) ) {
			SemanticMaps::newFromMediaWikiGlobals( $GLOBALS )->initExtension();
		}

		return true;
	}

	public static function onParserFirstCallInit1( Parser &$parser ) {
		$instance = new MapsCoordinates();
		return $instance->init( $parser );
	}

	public static function onParserFirstCallInit2( Parser &$parser ) {
		$instance = new MapsDisplayMap();
		return $instance->init( $parser );
	}

	public static function onParserFirstCallInit3( Parser &$parser ) {
		$instance = new MapsDistance();
		return $instance->init( $parser );
	}

	public static function onParserFirstCallInit4( Parser &$parser ) {
		$instance = new MapsFinddestination();
		return $instance->init( $parser );
	}

	public static function onParserFirstCallInit5( Parser &$parser ) {
		$instance = new MapsGeocode();
		return $instance->init( $parser );
	}

	public static function onParserFirstCallInit6( Parser &$parser ) {
		$instance = new MapsGeodistance();
		return $instance->init( $parser );
	}

	public static function onParserFirstCallInit7( Parser &$parser ) {
		$instance = new MapsMapsDoc();
		return $instance->init( $parser );
	}

	/**
	 * Adds a link to Admin Links page.
	 *
	 * @since 0.7
	 *
	 * @param ALTree $admin_links_tree
	 *
	 * @return boolean
	 */
	public static function addToAdminLinks( ALTree &$admin_links_tree ) {
		$displaying_data_section = $admin_links_tree->getSection( wfMessage( 'smw_adminlinks_displayingdata' )->text() );

		// Escape if SMW hasn't added links.
		if ( is_null( $displaying_data_section ) ) {
			return true;
		}

		$smw_docu_row = $displaying_data_section->getRow( 'smw' );

		$maps_docu_label = wfMessage( 'adminlinks_documentation', 'Maps' )->text();
		$smw_docu_row->addItem( AlItem::newFromExternalLink( 'https://semantic-mediawiki.org/wiki/Maps', $maps_docu_label ) );

		return true;
	}

	/**
	 * Adds global JavaScript variables.
	 *
	 * @since 1.0
	 * @see http://www.mediawiki.org/wiki/Manual:Hooks/MakeGlobalVariablesScript
	 * @param array &$vars Variables to be added into the output
	 * @param OutputPage $outputPage OutputPage instance calling the hook
	 * @return boolean true in all cases
	 */
	public static function onMakeGlobalVariablesScript( array &$vars, OutputPage $outputPage ) {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
		global $egMapsGlobalJSVars;

		$vars['egMapsDebugJS'] = $GLOBALS['egMapsDebugJS'];
                $vars[ 'egMapsAvailableServices' ] = $GLOBALS['egMapsAvailableServices'];

		$vars += $egMapsGlobalJSVars;

		return true;
	}

}



1			<?php
2
3			use DataValues\Geo\Parsers\GeoCoordinateParser;
4			use Maps\ServiceParam;
5			use Maps\LocationParser;
6			use Maps\LineParser;
7			use Maps\CircleParser;
8			use Maps\RectangleParser;
9			use Maps\PolygonParser;
10			use Maps\DistanceParser;
11			use Maps\WmsOverlayParser;
12			use Maps\ImageOverlayParser;
13
14			/**
15			* Static class for hooks handled by the Maps extension.
16			*
17			* @since 0.7
18			*
19			* @licence GNU GPL v2+
20			* @author Jeroen De Dauw < [email protected] >
21			*/
22			final class MapsHooks {
23
24			public static function onExtensionCallback() {
			0 ignored issues – show Coding Style introduced 2016-04-06 21:20 UTC by Report Bug Copy Issue Report `onExtensionCallback` uses the super-global variable `$GLOBALS` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
25			if ( defined( 'Maps_COORDS_FLOAT' ) ) {
26			// Do not initialize more than once.
27			return 1;
28			}
29
30			// The different coordinate notations.
31			define( 'Maps_COORDS_FLOAT' , 'float' );
32			define( 'Maps_COORDS_DMS' , 'dms' );
33			define( 'Maps_COORDS_DM' , 'dm' );
34			define( 'Maps_COORDS_DD' , 'dd' );
35
36			require_once __DIR__ . '/Maps_Settings.php';
37
38			define( 'Maps_VERSION' , '4.2.1' );
39			define( 'SM_VERSION', Maps_VERSION );
40
41			// Only initialize the extension when all dependencies are present.
42			if ( !defined( 'Validator_VERSION' ) ) {
43			throw new Exception( 'You need to have Validator installed in order to use Maps' );
44			}
45
46			$GLOBALS['egMapsStyleVersion'] = $GLOBALS['wgStyleVersion'] . '-' . Maps_VERSION;
47			}
48
49			public static function onExtensionFunction() {
			0 ignored issues – show Coding Style introduced 2016-04-06 19:55 UTC by Report Bug Copy Issue Report `onExtensionFunction` uses the super-global variable `$GLOBALS` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
50			if ( $GLOBALS['egMapsDisableExtension'] ) {
51			return true;
52			}
53			if ( defined( 'Maps_VERSION' ) ) {
54			// Do not initialize more than once.
55			return true;
56			}
57
58			// Only initialize the extension when all dependencies are present.
59			if ( !defined( 'Validator_VERSION' ) ) {
60			throw new Exception( 'You need to have Validator installed in order to use Maps' );
61			}
62
63			define( 'Maps_VERSION' , '4.2.1' );
64			define( 'SM_VERSION', Maps_VERSION );
65
66			if ( $GLOBALS['egMapsGMaps3Language'] === '' ) {
67			$GLOBALS['egMapsGMaps3Language'] = $GLOBALS['wgLang'];
68			}
69
70			MapsMappingServices::registerService( 'googlemaps3', MapsGoogleMaps3::class );
71
72			$googleMaps = MapsMappingServices::getServiceInstance( 'googlemaps3' );
73			$googleMaps->addFeature( 'display_map', MapsDisplayMapRenderer::class );
74
75			MapsMappingServices::registerService(
76			'openlayers',
77			MapsOpenLayers::class,
78			[ 'display_map' => MapsDisplayMapRenderer::class ]
79			);
80
81			MapsMappingServices::registerService( 'leaflet', MapsLeaflet::class );
82			$leafletMaps = MapsMappingServices::getServiceInstance( 'leaflet' );
83			$leafletMaps->addFeature( 'display_map', MapsDisplayMapRenderer::class );
84
85			if ( in_array( 'googlemaps3', $GLOBALS['egMapsAvailableServices'] ) ) {
86			$GLOBALS['wgSpecialPages']['MapEditor'] = 'SpecialMapEditor';
87			$GLOBALS['wgSpecialPageGroups']['MapEditor'] = 'maps';
88			}
89
90			$GLOBALS['egMapsStyleVersion'] = $GLOBALS['wgStyleVersion'] . '-' . Maps_VERSION;
91
92			// Users that can geocode. By default the same as those that can edit.
93			foreach ( $GLOBALS['wgGroupPermissions'] as $group => $rights ) {
94			if ( array_key_exists( 'edit' , $rights ) ) {
95			$GLOBALS['wgGroupPermissions'][$group]['geocode'] = $GLOBALS['wgGroupPermissions'][$group]['edit'];
96			}
97			}
98
99			$GLOBALS['wgParamDefinitions']['coordinate'] = [
100			'string-parser' => GeoCoordinateParser::class,
101			];
102
103			$GLOBALS['wgParamDefinitions']['mappingservice'] = [
104			'definition'=> ServiceParam::class,
105			];
106
107			$GLOBALS['wgParamDefinitions']['mapslocation'] = [
108			'string-parser' => LocationParser::class,
109			];
110
111			$GLOBALS['wgParamDefinitions']['mapsline'] = [
112			'string-parser' => LineParser::class,
113			];
114
115			$GLOBALS['wgParamDefinitions']['mapscircle'] = [
116			'string-parser' => CircleParser::class,
117			];
118
119			$GLOBALS['wgParamDefinitions']['mapsrectangle'] = [
120			'string-parser' => RectangleParser::class,
121			];
122
123			$GLOBALS['wgParamDefinitions']['mapspolygon'] = [
124			'string-parser' => PolygonParser::class,
125			];
126
127			$GLOBALS['wgParamDefinitions']['distance'] = [
128			'string-parser' => DistanceParser::class,
129			];
130
131			$GLOBALS['wgParamDefinitions']['wmsoverlay'] = [
132			'string-parser' => WmsOverlayParser::class,
133			];
134
135			$GLOBALS['wgParamDefinitions']['mapsimageoverlay'] = [
136			'string-parser' => ImageOverlayParser::class,
137			];
138
139			if ( !$GLOBALS['egMapsDisableSmwIntegration'] && defined( 'SMW_VERSION' ) ) {
140			SemanticMaps::newFromMediaWikiGlobals( $GLOBALS )->initExtension();
141			}
142
143			return true;
144			}
145
146			public static function onParserFirstCallInit1( Parser &$parser ) {
147			$instance = new MapsCoordinates();
148			return $instance->init( $parser );
149			}
150
151			public static function onParserFirstCallInit2( Parser &$parser ) {
152			$instance = new MapsDisplayMap();
153			return $instance->init( $parser );
154			}
155
156			public static function onParserFirstCallInit3( Parser &$parser ) {
157			$instance = new MapsDistance();
158			return $instance->init( $parser );
159			}
160
161			public static function onParserFirstCallInit4( Parser &$parser ) {
162			$instance = new MapsFinddestination();
163			return $instance->init( $parser );
164			}
165
166			public static function onParserFirstCallInit5( Parser &$parser ) {
167			$instance = new MapsGeocode();
168			return $instance->init( $parser );
169			}
170
171			public static function onParserFirstCallInit6( Parser &$parser ) {
172			$instance = new MapsGeodistance();
173			return $instance->init( $parser );
174			}
175
176			public static function onParserFirstCallInit7( Parser &$parser ) {
177			$instance = new MapsMapsDoc();
178			return $instance->init( $parser );
179			}
180
181			/**
182			* Adds a link to Admin Links page.
183			*
184			* @since 0.7
185			*
186			* @param ALTree $admin_links_tree
187			*
188			* @return boolean
189			*/
190			public static function addToAdminLinks( ALTree &$admin_links_tree ) {
191			$displaying_data_section = $admin_links_tree->getSection( wfMessage( 'smw_adminlinks_displayingdata' )->text() );
192
193			// Escape if SMW hasn't added links.
194			if ( is_null( $displaying_data_section ) ) {
195			return true;
196			}
197
198			$smw_docu_row = $displaying_data_section->getRow( 'smw' );
199
200			$maps_docu_label = wfMessage( 'adminlinks_documentation', 'Maps' )->text();
201			$smw_docu_row->addItem( AlItem::newFromExternalLink( 'https://semantic-mediawiki.org/wiki/Maps', $maps_docu_label ) );
202
203			return true;
204			}
205
206			/**
207			* Adds global JavaScript variables.
208			*
209			* @since 1.0
210			* @see http://www.mediawiki.org/wiki/Manual:Hooks/MakeGlobalVariablesScript
211			* @param array &$vars Variables to be added into the output
212			* @param OutputPage $outputPage OutputPage instance calling the hook
213			* @return boolean true in all cases
214			*/
215			public static function onMakeGlobalVariablesScript( array &$vars, OutputPage $outputPage ) {
			0 ignored issues – show Unused Code introduced 2015-12-04 20:45 UTC by Report Bug Copy Issue Report The parameter `$outputPage` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Coding Style introduced 2015-12-04 20:45 UTC by Report Bug Copy Issue Report `onMakeGlobalVariablesScript` uses the super-global variable `$GLOBALS` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
216			global $egMapsGlobalJSVars;
217
218			$vars['egMapsDebugJS'] = $GLOBALS['egMapsDebugJS'];
219			$vars[ 'egMapsAvailableServices' ] = $GLOBALS['egMapsAvailableServices'];
220
221			$vars += $egMapsGlobalJSVars;
222
223			return true;
224			}
225
226			}
227
228

JeroenDeDauw / Maps

Pull Request — master (#131)

MapsHooks::onExtensionFunction() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like