Merge pull request #20 from magento-mpi/public-pulls

[Github] Merge public Github commits

app/code/Magento/Catalog/Helper/Product/View.php

@@ -116,24 +116,26 @@ class View extends \Magento\Framework\App\Helper\AbstractHelper
             $pageConfig->setPageLayout($settings->getPageLayout());
         }
 
+        $urlSafeSku = rawurlencode($product->getSku());
+
         // Load default page handles and page configurations
         if ($params && $params->getBeforeHandles()) {
             foreach ($params->getBeforeHandles() as $handle) {
                 $resultPage->addPageLayoutHandles(
-                    ['id' => $product->getId(), 'sku' => $product->getSku(), 'type' => $product->getTypeId()],
+                    ['id' => $product->getId(), 'sku' => $urlSafeSku, 'type' => $product->getTypeId()],
                     $handle
                 );
             }
         }
 
         $resultPage->addPageLayoutHandles(
-            ['id' => $product->getId(), 'sku' => $product->getSku(), 'type' => $product->getTypeId()]
+            ['id' => $product->getId(), 'sku' => $urlSafeSku, 'type' => $product->getTypeId()]
         );
 
         if ($params && $params->getAfterHandles()) {
             foreach ($params->getAfterHandles() as $handle) {
                 $resultPage->addPageLayoutHandles(
-                    ['id' => $product->getId(), 'sku' => $product->getSku(), 'type' => $product->getTypeId()],
+                    ['id' => $product->getId(), 'sku' => $urlSafeSku, 'type' => $product->getTypeId()],
                     $handle
                 );
             }

app/code/Magento/PageCache/Model/Observer/ProcessLayoutRenderElement.php

@@ -37,7 +37,7 @@ class ProcessLayoutRenderElement
             'page_cache/block/esi',
             [
                 'blocks' => json_encode([$block->getNameInLayout()]),
-                'handles' => urlencode(json_encode($layout->getUpdate()->getHandles()))
+                'handles' => json_encode($layout->getUpdate()->getHandles())
             ]
         );
         return sprintf('<esi:include src="%s" />', $url);

app/code/Magento/Review/Controller/Product/ListAction.php

@@ -25,8 +25,10 @@ class ListAction extends \Magento\Review\Controller\Product
             $pageConfig->setPageLayout($product->getPageLayout());
         }
         $update = $this->_view->getLayout()->getUpdate();
+
+        $urlSafeSku = rawurlencode($product->getSku());
         $this->_view->addPageLayoutHandles(
-            ['id' => $product->getId(), 'sku' => $product->getSku(), 'type' => $product->getTypeId()]
+            ['id' => $product->getId(), 'sku' => $urlSafeSku, 'type' => $product->getTypeId()]
         );
 
         $this->_view->loadLayoutUpdates();

lib/internal/Magento/Framework/View/Layout/Reader/Container.php

@@ -86,7 +86,8 @@ class Container implements Layout\ReaderInterface
             default:
                 break;
         }
-        return $this->readerPool->interpret($readerContext, $currentElement);
+        $this->readerPool->interpret($readerContext, $currentElement);
+        return $this;
     }
 
     /**

lib/internal/Magento/Framework/View/Layout/Reader/Move.php

@@ -34,7 +34,7 @@ class Move implements Layout\ReaderInterface
     public function interpret(Context $readerContext, Layout\Element $currentElement)
     {
         $this->scheduleMove($readerContext->getScheduledStructure(), $currentElement);
-        return false;
+        return $this;
     }
 
     /**

lib/internal/Magento/Framework/View/Layout/Reader/Remove.php

@@ -36,6 +36,6 @@ class Remove implements Layout\ReaderInterface
     {
         $scheduledStructure = $readerContext->getScheduledStructure();
         $scheduledStructure->setElementToRemoveList((string)$currentElement->getAttribute('name'));
-        return false;
+        return $this;
     }
 }

lib/internal/Magento/Framework/View/Page/Config/Reader/Body.php

@@ -63,10 +63,11 @@ class Body implements Layout\ReaderInterface
         /** @var \Magento\Framework\View\Layout\Element $element */
         foreach ($bodyElement as $element) {
             if ($element->getName() === self::BODY_ATTRIBUTE) {
-                $this->setBodyAttributeTosStructure($readerContext, $element);
+                $this->setBodyAttributeToStructure($readerContext, $element);
             }
         }
-        return $this->readerPool->interpret($readerContext, $bodyElement);
+        $this->readerPool->interpret($readerContext, $bodyElement);
+        return $this;
     }
 
     /**
@@ -76,7 +77,7 @@ class Body implements Layout\ReaderInterface
      * @param Layout\Element $element
      * @return $this
      */
-    protected function setBodyAttributeTosStructure(Layout\Reader\Context $readerContext, Layout\Element $element)
+    protected function setBodyAttributeToStructure(Layout\Reader\Context $readerContext, Layout\Element $element)
     {
         if ($element->getAttribute('name') == PageConfig::BODY_ATTRIBUTE_CLASS) {
             $readerContext->getPageConfigStructure()->setBodyClass($element->getAttribute('value'));