SMDisplayAjaxMap

Complexity

Total Complexity

4

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	4
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getName()	0	3	1
B	getParameterInfo()	0	25	2
B	render()	0	32	1

<?php

/**
 * Class for the 'display_map_ajax' parser hooks.
 *
 * @since 0.7
 *
 * @licence GNU GPL v2+
 * @author Jeroen De Dauw < jeroendedauw@gmail.com >
 * @author Peter Grassberger < petertheone@gmail.com >
 */
class SMDisplayAjaxMap extends MapsDisplayMap {

    /**
     * Gets the name of the parser hook.
     * @see ParserHook::getName
     *
     * @since 0.7
     *
     * @return string
     */
    protected function getName() {
        return 'display_ajax_map';
    }

    /**
     * Returns an array containing the parameter info.
     * @see ParserHook::getParameterInfo
     *
     * @since 0.7
     *
     * @return array
     */
    protected function getParameterInfo( $type ) {
        $params = MapsMapper::getCommonParameters();

The method MapsMapper::getCommonParameters() has been deprecated.

        $params['mappingservice']['feature'] = 'display_ajax_map';

        $params['coordinates'] = array(
            //'type' => 'mapslocation',
            //'aliases' => array( 'coords', 'location', 'address', 'addresses', 'locations', 'points' ),
            'dependencies' => array( 'mappingservice', 'geoservice' ),
            'default' => '',
            'islist' => true,
            'delimiter' => $type === ParserHook::TYPE_FUNCTION ? ';' : "\n",
            'message' => 'maps-displaymap-par-coordinates',
        );

        $params['ajax'] = array(
            'type' => 'boolean',
            'default' => true,
            'message' => 'maps-displaymap-par-ajax',
        );

        $params = array_merge( $params, self::getCommonMapParams() );

The method MapsDisplayMap::getCommonMapParams() has been deprecated.

        return $params;
    }

    /**
     * Renders and returns the output.
     * @see ParserHook::render
     *
     * @since 0.7
     *
     * @param array $parameters
     *
     * @return string
     */
    public function render( array $parameters ) {
        // Get the instance of the service class.
        $service = MapsMappingServices::getServiceInstance( $parameters['mappingservice'], $this->getName() );

The call to MapsMappingServices::getServiceInstance() has too many arguments starting with $this->getName().

        $mapName = $service->getMapId();

        $fullParams = $this->validator->getParameters();

$fullParams is not used, you could remove the assignment.

The method ParamProcessor\Processor::getParameters() has been deprecated with message: since 1.0

        $configVars = Skin::makeVariablesScript( $service->getConfigVariables() );
        $service->addDependencies( $this->parser );
        $this->parser->getOutput()->addHeadItem( $configVars );

        /*if ( array_key_exists( 'zoom', $fullParams ) && $fullParams['zoom']->wasSetToDefault() && count( $parameters['coordinates'] ) > 1 ) {
            $parameters['zoom'] = false;
        }

        $this->parser->addTrackingCategory( 'maps-tracking-category' );*/

        return Html::rawElement(
            'div',
            array(
                'id' => $mapName,
                'style' => "width: {$parameters['width']}; height: {$parameters['height']}; background-color: #cccccc; overflow: hidden;",
                'class' => 'maps-map maps-' . $service->getName()
            ),
            wfMessage( 'maps-loading-map' )->inContentLanguage()->escaped() .
            Html::element(
                'div',
                array( 'style' => 'display:none', 'class' => 'mapdata' ),
                FormatJson::encode( $parameters )
            )
        );
    }

}