diff --git a/app/code/community/Nexcessnet/Turpentine/Block/Poll/ActivePoll.php b/app/code/community/Nexcessnet/Turpentine/Block/Poll/ActivePoll.php index 56343127f..665af029b 100644 --- a/app/code/community/Nexcessnet/Turpentine/Block/Poll/ActivePoll.php +++ b/app/code/community/Nexcessnet/Turpentine/Block/Poll/ActivePoll.php @@ -21,19 +21,19 @@ class Nexcessnet_Turpentine_Block_Poll_ActivePoll extends Mage_Poll_Block_ActivePoll { - public function setTemplate($template) - { - if ((Mage::getConfig()->getModuleConfig('Mage_Poll')->is('active', 'true')) && - (!Mage::getStoreConfig('advanced/modules_disable_output/Mage_Poll'))) - { - $this->_template = $template; - $this->setPollTemplate('turpentine/ajax.phtml', 'poll'); - $this->setPollTemplate('turpentine/ajax.phtml', 'results'); - } - else - { - // Mage_Poll is disabled, so do nothing - } - return $this; - } + public function setTemplate($template) + { + if ((Mage::getConfig()->getModuleConfig('Mage_Poll')->is('active', 'true')) && + (!Mage::getStoreConfig('advanced/modules_disable_output/Mage_Poll'))) + { + $this->_template = $template; + $this->setPollTemplate('turpentine/ajax.phtml', 'poll'); + $this->setPollTemplate('turpentine/ajax.phtml', 'results'); + } + else + { + // Mage_Poll is disabled, so do nothing + } + return $this; + } } diff --git a/app/code/community/Nexcessnet/Turpentine/Helper/Esi.php b/app/code/community/Nexcessnet/Turpentine/Helper/Esi.php index f7b263059..d9054c6fe 100644 --- a/app/code/community/Nexcessnet/Turpentine/Helper/Esi.php +++ b/app/code/community/Nexcessnet/Turpentine/Helper/Esi.php @@ -373,7 +373,7 @@ public function getFormKeyEsiUrl() { * @param string $blockName value of name= attribute in layout XML * @return Mage_Core_Model_Layout_Element */ - public function getEsiLayoutBlockNode($layout,$blockName) { + public function getEsiLayoutBlockNode($layout, $blockName) { // first try very specific by checking for action setEsiOptions inside block $blockNode = current($layout->getNode()->xpath( sprintf('//block[@name=\'%s\'][action[@method=\'setEsiOptions\']]', diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Core/Session.php b/app/code/community/Nexcessnet/Turpentine/Model/Core/Session.php index 4078fffb1..141c667b2 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Core/Session.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Core/Session.php @@ -56,7 +56,7 @@ public function real_getFormKey() */ public function renewFormKey() { - $this->setData('_form_key', Mage::helper('core')->getRandomString(16)); + $this->setData('_form_key', Mage::helper('core')->getRandomString(16)); } /** diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Observer/Esi.php b/app/code/community/Nexcessnet/Turpentine/Model/Observer/Esi.php index a79b7727d..bf06417a0 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Observer/Esi.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Observer/Esi.php @@ -227,9 +227,9 @@ public function injectEsi($eventObject) { 'Checking ESI block candidate: %s', $blockObject->getNameInLayout() ? $blockObject->getNameInLayout() : $blockObject->getModuleName() ); - $debugHelper->logInfo( "-- block testing: shouldResponseUseEsi = " . $esiHelper->shouldResponseUseEsi()); - $debugHelper->logInfo( "-- block testing: instanceof Mage_Core_Block_Template = " . $blockObject instanceof Mage_Core_Block_Template ); - $debugHelper->logInfo( "-- block testing: Esi Options = " . print_r($blockObject->getEsiOptions(), true) ); + $debugHelper->logInfo("-- block testing: shouldResponseUseEsi = ".$esiHelper->shouldResponseUseEsi()); + $debugHelper->logInfo("-- block testing: instanceof Mage_Core_Block_Template = ".$blockObject instanceof Mage_Core_Block_Template); + $debugHelper->logInfo("-- block testing: Esi Options = ".print_r($blockObject->getEsiOptions(), true)); } if ($esiHelper->shouldResponseUseEsi() && $blockObject instanceof Mage_Core_Block_Template && @@ -332,8 +332,8 @@ protected function _getEsiData($blockObject, $esiOptions) { $methodParam = $esiHelper->getEsiMethodParam(); $esiData = new Varien_Object(); $esiData->setStoreId(Mage::app()->getStore()->getId()); - $esiData->setDesignPackage( Mage::getDesign()->getPackageName() ); - $esiData->setDesignTheme( Mage::getDesign()->getTheme( 'layout' ) ); + $esiData->setDesignPackage(Mage::getDesign()->getPackageName()); + $esiData->setDesignTheme(Mage::getDesign()->getTheme('layout')); $esiData->setNameInLayout($blockObject->getNameInLayout()); $esiData->setBlockType(get_class($blockObject)); $esiData->setLayoutHandles($this->_getBlockLayoutHandles($blockObject)); diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Observer/Varnish.php b/app/code/community/Nexcessnet/Turpentine/Model/Observer/Varnish.php index bfab14f7d..1fcadda7c 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Observer/Varnish.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Observer/Varnish.php @@ -66,7 +66,7 @@ public function addProductListToolbarRewrite($eventObject) { public function fixCmRedisSessionLocks($eventObject) { if (Mage::helper('core')->isModuleEnabled('Cm_RedisSession')) { if ( ! empty($_COOKIE['frontend']) && 'crawler-session' == $_COOKIE['frontend'] && - !defined('CM_REDISSESSION_LOCKING_ENABLED') ) { + ! defined('CM_REDISSESSION_LOCKING_ENABLED')) { define('CM_REDISSESSION_LOCKING_ENABLED', false); } } diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin.php b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin.php index 096448da0..764f5dac5 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin.php @@ -138,7 +138,7 @@ protected function _testEsiSyntaxParam($socket) { $result = false; if ($helper->csrfFixupNeeded()) { - if ($socket->getVersion() === '4.0' || $socket->getVersion() === '4.1' ) { + if ($socket->getVersion() === '4.0' || $socket->getVersion() === '4.1') { $paramName = 'feature'; $value = $socket->param_show($paramName); $value = explode("\n", $value['text']); diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin/Socket.php b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin/Socket.php index fadcb2c38..ed4b92718 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin/Socket.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Admin/Socket.php @@ -500,7 +500,7 @@ protected function _command($verb, $okCode = 200) { $response['code'], $response['text'] )); } else { if (Mage::getStoreConfig('turpentine_varnish/general/varnish_log_commands')) { - Mage::helper('turpentine/debug')->logDebug('VARNISH command sent: ' . $data); + Mage::helper('turpentine/debug')->logDebug('VARNISH command sent: '.$data); } return $response; } diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php index a8c6149a7..d40efe3ea 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php @@ -111,8 +111,8 @@ public function save($generatedConfig) { * @return string */ protected function _getVclTemplateFilename($baseFilename) { - $extensionDir = Mage::getModuleDir('', 'Nexcessnet_Turpentine'); - return sprintf('%s/misc/%s', $extensionDir, $baseFilename); + $extensionDir = Mage::getModuleDir('', 'Nexcessnet_Turpentine'); + return sprintf('%s/misc/%s', $extensionDir, $baseFilename); } /** @@ -131,7 +131,7 @@ protected function _getVclFilename() { * * @return string */ - protected function _getCustomIncludeFilename($position='') { + protected function _getCustomIncludeFilename($position = '') { $key = 'custom_include_file'; $key .= ($position) ? '_'.$position : ''; return $this->_formatTemplate( @@ -151,8 +151,7 @@ protected function _getCustomTemplateFilename() { Mage::getStoreConfig('turpentine_varnish/servers/custom_vcl_template'), array('root_dir' => Mage::getBaseDir()) ); - if (is_file($filePath)) { return $filePath; } - else { return null; } + if (is_file($filePath)) { return $filePath; } else { return null; } } @@ -193,8 +192,8 @@ protected function _vcl_call($subroutine) { */ protected function _getAdminFrontname() { if (Mage::getStoreConfig('admin/url/use_custom_path')) { - if(Mage::getStoreConfig('web/url/use_store')) { - return Mage::getModel('core/store')->load(0)->getCode() . "/" . Mage::getStoreConfig('admin/url/custom_path'); + if (Mage::getStoreConfig('web/url/use_store')) { + return Mage::getModel('core/store')->load(0)->getCode()."/".Mage::getStoreConfig('admin/url/custom_path'); } else { return Mage::getStoreConfig('admin/url/custom_path'); } @@ -909,8 +908,8 @@ protected function _vcl_sub_maintenance_allowed_ips() { */ protected function _vcl_sub_https_redirect_fix() { $baseUrl = Mage::getBaseUrl(Mage_Core_Model_Store::URL_TYPE_WEB); - $baseUrl = str_replace(array('http://','https://'), '', $baseUrl); - $baseUrl = rtrim($baseUrl,'/'); + $baseUrl = str_replace(array('http://', 'https://'), '', $baseUrl); + $baseUrl = rtrim($baseUrl, '/'); switch (Mage::getStoreConfig('turpentine_varnish/servers/version')) { case 4.0: @@ -982,7 +981,7 @@ protected function _vcl_sub_synth_https_fix() { $tpl = $this->_vcl_sub_synth(); - if(!$tpl){ + if ( ! $tpl) { $tpl = <<_vcl_sub_https_redirect_fix(); - if(Mage::getStoreConfig('turpentine_varnish/servers/version') == '4.0' || Mage::getStoreConfig('turpentine_varnish/servers/version') == '4.1'){ + if (Mage::getStoreConfig('turpentine_varnish/servers/version') == '4.0' || Mage::getStoreConfig('turpentine_varnish/servers/version') == '4.1') { $vars['vcl_synth'] = $this->_vcl_sub_synth_https_fix(); } } - foreach (array('','top') as $position) { + foreach (array('', 'top') as $position) { $customIncludeFile = $this->_getCustomIncludeFilename($position); if (is_readable($customIncludeFile)) { $key = 'custom_vcl_include'; diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version3.php b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version3.php index 2247404b0..d025b737c 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version3.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version3.php @@ -35,8 +35,7 @@ public function generate($doClean = true) { $customTemplate = $this->_getCustomTemplateFilename(); if ($customTemplate) { $tplFile = $customTemplate; - } - else { + } else { $tplFile = $this->_getVclTemplateFilename(self::VCL_TEMPLATE_FILE); } $vcl = $this->_formatTemplate(file_get_contents($tplFile), diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version4.php b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version4.php index 72512cbcf..78c44c642 100644 --- a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version4.php +++ b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Version4.php @@ -35,8 +35,7 @@ public function generate($doClean = true) { $customTemplate = $this->_getCustomTemplateFilename(); if ($customTemplate) { $tplFile = $customTemplate; - } - else { + } else { $tplFile = $this->_getVclTemplateFilename(self::VCL_TEMPLATE_FILE); } $vcl = $this->_formatTemplate(file_get_contents($tplFile), @@ -94,13 +93,13 @@ protected function _vcl_directors() $backendNodes = Mage::helper('turpentine/data')->cleanExplode(PHP_EOL, Mage::getStoreConfig('turpentine_vcl/backend/backend_nodes')); - for($i = 0, $iMax = count($backendNodes); $i < $iMax; $i++) { + for ($i = 0, $iMax = count($backendNodes); $i < $iMax; $i++) { $tpl .= <<_vcl_director_backend($host, $port, $prefix . $number, $probeUrl, $backendOptions); + $backends .= $this->_vcl_director_backend($host, $port, $prefix.$number, $probeUrl, $backendOptions); $number++; } diff --git a/app/code/community/Nexcessnet/Turpentine/controllers/EsiController.php b/app/code/community/Nexcessnet/Turpentine/controllers/EsiController.php index 061f913f6..576e77c34 100644 --- a/app/code/community/Nexcessnet/Turpentine/controllers/EsiController.php +++ b/app/code/community/Nexcessnet/Turpentine/controllers/EsiController.php @@ -183,9 +183,9 @@ protected function _getEsiBlock($esiData) { } } $layout = Mage::getSingleton('core/layout'); - Mage::getSingleton( 'core/design_package' ) - ->setPackageName( $esiData->getDesignPackage() ) - ->setTheme( $esiData->getDesignTheme() ); + Mage::getSingleton('core/design_package') + ->setPackageName($esiData->getDesignPackage()) + ->setTheme($esiData->getDesignTheme()); // dispatch event for adding handles to layout update Mage::dispatchEvent( diff --git a/contrib/tools/esi-decoder.php b/contrib/tools/esi-decoder.php index cf2bb6685..553e3f2fa 100644 --- a/contrib/tools/esi-decoder.php +++ b/contrib/tools/esi-decoder.php @@ -1,16 +1,16 @@