Merge remote-tracking branches 'remotes/jack/MAGETWO-53265', 'remotes/jack/MAGETWO-53552', 'remotes/jack/MAGETWO-54152', 'remotes/jack/MAGETWO-54157', 'remotes/jack/MAGETWO-54160', 'remotes/jack/MAGETWO-54586-PublicPR-3638' and 'remotes/jack/MAGETWO-54587-Public-PR-3633' into PublicPrsCompilation

Author: Hayder Sharhan

app/code/Magento/Backend/Block/Widget/Grid/Column/Renderer/Date.php

@@ -52,7 +52,7 @@ public function __construct(
     protected function _getFormat()
     {
         $format = $this->getColumn()->getFormat();
-        if (!$format) {
+        if ($format === null) {
             if (self::$_format === null) {
                 try {
                     self::$_format = $this->_localeDate->getDateFormat(

app/code/Magento/Checkout/etc/acl.xml

@@ -0,0 +1,22 @@
+<?xml version="1.0"?>
+<!--
+/**
+ * Copyright © 2015 Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+-->
+<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Acl/etc/acl.xsd">
+    <acl>
+        <resources>
+            <resource id="Magento_Backend::admin">
+                <resource id="Magento_Backend::stores">
+                    <resource id="Magento_Backend::stores_settings">
+                        <resource id="Magento_Config::config">
+                            <resource id="Magento_Checkout::checkout" title="Checkout Section" translate="true" sortOrder="57" />
+                        </resource>
+                    </resource>
+                </resource>
+            </resource>
+        </resources>
+    </acl>
+</config>

app/code/Magento/Checkout/i18n/en_US.csv

@@ -27,6 +27,7 @@ State/Province,State/Province
 "One-page checkout is turned off.","One-page checkout is turned off."
 "Guest checkout is disabled.","Guest checkout is disabled."
 Checkout,Checkout
+"Checkout Section","Checkout Section"
 "Page not found.","Page not found."
 "Please agree to all the terms and conditions before placing the order.","Please agree to all the terms and conditions before placing the order."
 "Something went wrong while processing your order. Please try again later.","Something went wrong while processing your order. Please try again later."

app/code/Magento/ConfigurableProduct/Model/Product/Validator/Plugin.php

@@ -112,9 +112,11 @@ protected function _validateProductVariations(Product $parentProduct, array $pro
             $product->addData($productData);
             $product->setCollectExceptionMessages(true);
             $configurableAttribute = [];
-            $encodedData = $productData['configurable_attribute'];
-            if ($encodedData) {
-                $configurableAttribute = $this->jsonHelper->jsonDecode($encodedData);
+            if (!empty($productData['configurable_attribute'])) {
+                $encodedData = $productData['configurable_attribute'];
+                if ($encodedData) {
+                    $configurableAttribute = $this->jsonHelper->jsonDecode($encodedData);
+                }
             }
             $configurableAttribute = implode('-', $configurableAttribute);
 

app/code/Magento/Newsletter/Controller/Subscriber/NewAction.php

@@ -121,6 +121,15 @@ public function execute()
                 $this->validateGuestSubscription();
                 $this->validateEmailAvailable($email);
 
+                $subscriber = $this->_subscriberFactory->create()->loadByEmail($email);
+                if ($subscriber->getId()
+                    && $subscriber->getSubscriberStatus() == \Magento\Newsletter\Model\Subscriber::STATUS_SUBSCRIBED
+                ) {
+                    throw new \Magento\Framework\Exception\LocalizedException(
+                        __('This email address is already subscribed.')
+                    );
+                }
+
                 $status = $this->_subscriberFactory->create()->subscribe($email);
                 if ($status == \Magento\Newsletter\Model\Subscriber::STATUS_NOT_ACTIVE) {
                     $this->messageManager->addSuccess(__('The confirmation request has been sent.'));

app/code/Magento/Newsletter/i18n/en_US.csv

@@ -146,3 +146,4 @@ Unconfirmed,Unconfirmed
 Store,Store
 "Store View","Store View"
 "Newsletter Subscriptions","Newsletter Subscriptions"
+"This email address is already subscribed.","This email address is already subscribed."

app/code/Magento/Swatches/view/frontend/layout/catalogsearch_result_index.xml

@@ -9,4 +9,9 @@
     <head>
         <css src="Magento_Swatches::css/swatches.css"/>
     </head>
+    <body>
+        <referenceBlock name="category.product.type.details.renderers">
+            <block class="Magento\Swatches\Block\Product\Renderer\Listing\Configurable" as="configurable" template="Magento_Swatches::product/listing/renderer.phtml" />
+        </referenceBlock>
+    </body>
 </page>

lib/internal/Magento/Framework/View/Result/Layout.php

@@ -159,12 +159,13 @@ public function renderResult(ResponseInterface $response)
     {
         \Magento\Framework\Profiler::start('LAYOUT');
         \Magento\Framework\Profiler::start('layout_render');
-
+        
+        $this->eventManager->dispatch('layout_render_before');
+        $this->eventManager->dispatch('layout_render_before_' . $this->request->getFullActionName());
+        
         $this->applyHttpHeaders($response);
         $this->render($response);
 
-        $this->eventManager->dispatch('layout_render_before');
-        $this->eventManager->dispatch('layout_render_before_' . $this->request->getFullActionName());
         \Magento\Framework\Profiler::stop('layout_render');
         \Magento\Framework\Profiler::stop('LAYOUT');
         return $this;