Nexcessnet_Turpentine_Model_Observer_Varnish::fixCmRedisSessionLocks() - Code Metrics - Inspection of "Merge pull request #1334 from jeroenvermeulen/bugf..." - nexcess/magento-turpentine - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — devel ( 5c12d3...407f4a )

by Miguel

created 2016-12-27 13:46 UTC

fixCmRedisSessionLocks() B

↳ Parent: Nexcessnet_Turpentine_Model_Observer_Varnish

Complexity

Conditions	5
Paths	3

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	8
rs	8.8571
c	0
b	0
f	0
cc	5
eloc	5
nc	3
nop	1

<?php

/**
 * Nexcess.net Turpentine Extension for Magento
 * Copyright (C) 2012  Nexcess.net L.L.C.
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License along
 * with this program; if not, write to the Free Software Foundation, Inc.,
 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */

class Nexcessnet_Turpentine_Model_Observer_Varnish extends Varien_Event_Observer {
namespace YourVendor;

class YourClass { }
    /**
     * Check sentinel flags and set headers/cookies as needed
     *
     * Events: http_response_send_before
     *
     * @param  mixed $eventObject
     * @return null
     */
    public function setCacheFlagHeader($eventObject) {

        $response = $eventObject->getResponse();
        if (Mage::helper('turpentine/varnish')->shouldResponseUseVarnish()) {
            $response->setHeader('X-Turpentine-Cache',
                Mage::registry('turpentine_nocache_flag') ? '0' : '1');
            if (Mage::helper('turpentine/varnish')->getVarnishDebugEnabled()) {
                Mage::helper('turpentine/debug')->logDebug(
                    'Set Varnish cache flag header to: '.
                    (Mage::registry('turpentine_nocache_flag') ? '0' : '1') );
            }
        }
    }

    /**
     * Add a rewrite for catalog/product_list_toolbar if config option enabled
     *
     * @param Varien_Object $eventObject
     * @return null
     */
    public function addProductListToolbarRewrite($eventObject) {
        if (Mage::helper('turpentine/varnish')->shouldFixProductListToolbar()) {
            Mage::getSingleton('turpentine/shim_mage_core_app')
                ->shim_addClassRewrite('block', 'catalog', 'product_list_toolbar',
                    'Nexcessnet_Turpentine_Block_Catalog_Product_List_Toolbar');
        }
    }

    /**
     * Turpentine sets the fake cookie 'frontend=crawler-session' when a crawler is detected.
     * This causes lock problems with Cm_RedisSession, because all crawler hits are requesting the same session lock.
     * Cm_RedisSession provides the define CM_REDISSESSION_LOCKING_ENABLED to overrule if locking should be enabled.
     *
     * @param $eventObject
     * @return null
     */
    public function fixCmRedisSessionLocks($eventObject) {
// 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 (Mage::helper('core')->isModuleEnabled('Cm_RedisSession')) {
            if ( ! empty($_COOKIE['frontend']) && 'crawler-session' == $_COOKIE['frontend'] &&
                    !defined('CM_REDISSESSION_LOCKING_ENABLED') ) {
                define('CM_REDISSESSION_LOCKING_ENABLED', false);
            }
        }
    }

    /**
     * Re-apply and save Varnish configuration on config change
     *
     * @param  mixed $eventObject
     * @return null
     */
    public function adminSystemConfigChangedSection($eventObject) {
        if (Mage::helper('turpentine/varnish')->getVarnishEnabled() &&
                Mage::helper('turpentine/data')->getAutoApplyOnSave()) {
            $result = Mage::getModel('turpentine/varnish_admin')->applyConfig();
            $session = Mage::getSingleton('core/session');
            foreach ($result as $name => $value) {
                if ($value === true) {
                    $session->addSuccess(Mage::helper('turpentine/data')
                        ->__('VCL successfully applied to: '.$name));
                } else {
                    $session->addError(Mage::helper('turpentine/data')
                        ->__(sprintf('Failed to apply the VCL to %s: %s',
                            $name, $value)));
                }
            }
            $cfgr = Mage::getModel('turpentine/varnish_admin')->getConfigurator();
            if (is_null($cfgr)) {
                $session->addError(Mage::helper('turpentine/data')
                    ->__('Failed to load configurator'));
            } else {
                $result = $cfgr->save($cfgr->generate());
                if ($result[0]) {
                    $session->addSuccess(Mage::helper('turpentine/data')
                        ->__('The VCL file has been saved.'));
                } else {
                    $session->addError(Mage::helper('turpentine/data')
                        ->__('Failed to save the VCL file: '.$result[1]['message']));
                }
            }
        }
    }
}


1		<?php
2
3		/**
4		* Nexcess.net Turpentine Extension for Magento
5		* Copyright (C) 2012 Nexcess.net L.L.C.
6		*
7		* This program is free software; you can redistribute it and/or modify
8		* it under the terms of the GNU General Public License as published by
9		* the Free Software Foundation; either version 2 of the License, or
10		* (at your option) any later version.
11		*
12		* This program is distributed in the hope that it will be useful,
13		* but WITHOUT ANY WARRANTY; without even the implied warranty of
14		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15		* GNU General Public License for more details.
16		*
17		* You should have received a copy of the GNU General Public License along
18		* with this program; if not, write to the Free Software Foundation, Inc.,
19		* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
20		*/
21
22		class Nexcessnet_Turpentine_Model_Observer_Varnish extends Varien_Event_Observer {
		0 ignored issues – show Coding Style Compatibility introduced 2015-11-25 14:59 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
23		/**
24		* Check sentinel flags and set headers/cookies as needed
25		*
26		* Events: http_response_send_before
27		*
28		* @param mixed $eventObject
29		* @return null
30		*/
31	View Code Duplication	public function setCacheFlagHeader($eventObject) {
		0 ignored issues – show Duplication introduced 2015-11-25 14:59 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...
32		$response = $eventObject->getResponse();
33		if (Mage::helper('turpentine/varnish')->shouldResponseUseVarnish()) {
34		$response->setHeader('X-Turpentine-Cache',
35		Mage::registry('turpentine_nocache_flag') ? '0' : '1');
36		if (Mage::helper('turpentine/varnish')->getVarnishDebugEnabled()) {
37		Mage::helper('turpentine/debug')->logDebug(
38		'Set Varnish cache flag header to: '.
39		(Mage::registry('turpentine_nocache_flag') ? '0' : '1') );
40		}
41		}
42		}
43
44		/**
45		* Add a rewrite for catalog/product_list_toolbar if config option enabled
46		*
47		* @param Varien_Object $eventObject
48		* @return null
49		*/
50		public function addProductListToolbarRewrite($eventObject) {
51		if (Mage::helper('turpentine/varnish')->shouldFixProductListToolbar()) {
52		Mage::getSingleton('turpentine/shim_mage_core_app')
53		->shim_addClassRewrite('block', 'catalog', 'product_list_toolbar',
54		'Nexcessnet_Turpentine_Block_Catalog_Product_List_Toolbar');
55		}
56		}
57
58		/**
59		* Turpentine sets the fake cookie 'frontend=crawler-session' when a crawler is detected.
60		* This causes lock problems with Cm_RedisSession, because all crawler hits are requesting the same session lock.
61		* Cm_RedisSession provides the define CM_REDISSESSION_LOCKING_ENABLED to overrule if locking should be enabled.
62		*
63		* @param $eventObject
64		* @return null
65		*/
66		public function fixCmRedisSessionLocks($eventObject) {
		0 ignored issues – show Coding Style introduced 2016-01-14 13:42 UTC by Report Bug Copy Issue Report `fixCmRedisSessionLocks` uses the super-global variable `$_COOKIE` 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...
67		if (Mage::helper('core')->isModuleEnabled('Cm_RedisSession')) {
68		if ( ! empty($_COOKIE['frontend']) && 'crawler-session' == $_COOKIE['frontend'] &&
69		!defined('CM_REDISSESSION_LOCKING_ENABLED') ) {
70		define('CM_REDISSESSION_LOCKING_ENABLED', false);
71		}
72		}
73		}
74
75		/**
76		* Re-apply and save Varnish configuration on config change
77		*
78		* @param mixed $eventObject
79		* @return null
80		*/
81		public function adminSystemConfigChangedSection($eventObject) {
82		if (Mage::helper('turpentine/varnish')->getVarnishEnabled() &&
83		Mage::helper('turpentine/data')->getAutoApplyOnSave()) {
84		$result = Mage::getModel('turpentine/varnish_admin')->applyConfig();
85		$session = Mage::getSingleton('core/session');
86		foreach ($result as $name => $value) {
87		if ($value === true) {
88		$session->addSuccess(Mage::helper('turpentine/data')
89		->__('VCL successfully applied to: '.$name));
90		} else {
91		$session->addError(Mage::helper('turpentine/data')
92		->__(sprintf('Failed to apply the VCL to %s: %s',
93		$name, $value)));
94		}
95		}
96		$cfgr = Mage::getModel('turpentine/varnish_admin')->getConfigurator();
97		if (is_null($cfgr)) {
98		$session->addError(Mage::helper('turpentine/data')
99		->__('Failed to load configurator'));
100		} else {
101		$result = $cfgr->save($cfgr->generate());
102		if ($result[0]) {
103		$session->addSuccess(Mage::helper('turpentine/data')
104		->__('The VCL file has been saved.'));
105		} else {
106		$session->addError(Mage::helper('turpentine/data')
107		->__('Failed to save the VCL file: '.$result[1]['message']));
108		}
109		}
110		}
111		}
112		}
113

nexcess / magento-turpentine

Push — devel ( 5c12d3...407f4a )

fixCmRedisSessionLocks() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like