Skip to content

Commit 0be7be5

Browse files
author
Maxim Medinskiy
committed
Merge remote-tracking branch 'origin/MAGETWO-35026' into BUGS
2 parents 051fc59 + 6f1129a commit 0be7be5

File tree

2 files changed

+19
-15
lines changed

2 files changed

+19
-15
lines changed

app/code/Magento/Customer/view/frontend/templates/form/register.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@
164164
require([
165165
'jquery',
166166
'mage/mage'
167-
], function(jQuery){
167+
], function($){
168168

169169
var dataForm = $('#form-validate');
170170
var ignore = <?php echo $_dob->isEnabled() ? '\'input[id$="full"]\'' : 'null'; ?>;

app/code/Magento/RequireJs/Block/Html/Head/Config.php

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66

77
namespace Magento\RequireJs\Block\Html\Head;
88

9+
use Magento\Framework\RequireJs\Config as RequireJsConfig;
10+
911
/**
1012
* Block responsible for including RequireJs config on the page
1113
*/
1214
class Config extends \Magento\Framework\View\Element\AbstractBlock
1315
{
1416
/**
15-
* @var \Magento\Framework\RequireJs\Config
17+
* @var RequireJsConfig
1618
*/
1719
private $config;
1820

@@ -28,15 +30,15 @@ class Config extends \Magento\Framework\View\Element\AbstractBlock
2830

2931
/**
3032
* @param \Magento\Framework\View\Element\Context $context
31-
* @param \Magento\Framework\RequireJs\Config $config
33+
* @param RequireJsConfig $config
3234
* @param \Magento\RequireJs\Model\FileManager $fileManager
3335
* @param \Magento\Framework\View\Page\Config $pageConfig
3436
* @param \Magento\Framework\View\Asset\ConfigInterface $bundleConfig
3537
* @param array $data
3638
*/
3739
public function __construct(
3840
\Magento\Framework\View\Element\Context $context,
39-
\Magento\Framework\RequireJs\Config $config,
41+
RequireJsConfig $config,
4042
\Magento\RequireJs\Model\FileManager $fileManager,
4143
\Magento\Framework\View\Page\Config $pageConfig,
4244
\Magento\Framework\View\Asset\ConfigInterface $bundleConfig,
@@ -59,28 +61,30 @@ protected function _prepareLayout()
5961
$requireJsConfig = $this->fileManager->createRequireJsConfigAsset();
6062
$assetCollection = $this->pageConfig->getAssetCollection();
6163

62-
if ($this->bundleConfig->isBundlingJsFiles()) {
63-
$after = \Magento\Framework\RequireJs\Config::REQUIRE_JS_FILE_NAME;
64+
$assetCollection->insert(
65+
$requireJsConfig->getFilePath(),
66+
$requireJsConfig,
67+
RequireJsConfig::REQUIRE_JS_FILE_NAME
68+
);
6469

70+
if ($this->bundleConfig->isBundlingJsFiles()) {
6571
$bundleAssets = $this->fileManager->createBundleJsPool();
6672
$staticAsset = $this->fileManager->createStaticJsAsset();
6773

6874
/** @var \Magento\Framework\View\Asset\File $bundleAsset */
6975
if (!empty($bundleAssets) && $staticAsset !== false) {
76+
$bundleAssets = array_reverse($bundleAssets);
7077
foreach ($bundleAssets as $bundleAsset) {
71-
$assetCollection->insert($bundleAsset->getFilePath(), $bundleAsset, $after);
72-
$after = $bundleAsset->getFilePath();
78+
$assetCollection->insert(
79+
$bundleAsset->getFilePath(),
80+
$bundleAsset,
81+
RequireJsConfig::REQUIRE_JS_FILE_NAME
82+
);
7383
}
74-
$assetCollection->insert($staticAsset->getFilePath(), $staticAsset, $after);
84+
$assetCollection->insert($staticAsset->getFilePath(), $staticAsset, RequireJsConfig::CONFIG_FILE_NAME);
7585
}
7686
}
7787

78-
$assetCollection->insert(
79-
$requireJsConfig->getFilePath(),
80-
$requireJsConfig,
81-
\Magento\Framework\RequireJs\Config::REQUIRE_JS_FILE_NAME
82-
);
83-
8488
return parent::_prepareLayout();
8589
}
8690

0 commit comments

Comments
 (0)