Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/develop' into UI
Browse files Browse the repository at this point in the history
  • Loading branch information
Natalia Momotenko committed Feb 23, 2015
2 parents fa33312 + ba82edc commit 7e7e1e9
Show file tree
Hide file tree
Showing 745 changed files with 8,166 additions and 11,387 deletions.
35 changes: 35 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,38 @@
0.42.0-beta9
=============
* Framework Improvements:
* Layout Models are moved from Core module to appropriate modules
* View components are moved from Core to Theme module
* Rest of theme related configuration files are refactored
* StoreManagerInterface is moved from Framework to App folder
* ZF1 controller libraries are updated
* Class definitions in multi-tenant mode are removed
* DI configuration became more optimal: OM cached configuration uses the general pattern for all argument types in application
* Varnish 4 configuration is updated
* Layout Processing became more fast
* HTML response minified
* App Components and Specific Helper Components are moved from the Magento_Core Module
* UI improvements:
* Add to cart operation became asynchronous and doesn`t reload page (AJAX call)
* Fixed Defects:
* When Inline Translation is enabled, JQuery buttons for translate were broken
* Base URL has invalid place inside Magento Admin Address on "Web Configuration" step of installation wizard
* Inability of submit Product from keyboard while Product Creation
* Sold products aren't displayed in Bestsellers
* Compiled definitions can cause unexpected errors compared to runtime definitions
* Accessibility improvements:
* WAI-ARIA attributes are added to Frontend Layered Navigation and Customer Dropdown, Frontend Product Page Tabs, Frontend Cart Summary collapsible panels, Frontend forms and notifications, Frontend Checkout pages
* Tests improvements:
* Added mechanism of replacing 3-rd party credentials in functional tests
* Update of end-to-end tests for create product, update product, promoted product, out of stock product, create product with new category, unassign products on category, create backend customer with injectable test
* Various improvements:
* JS template engine became unified on Backend and Frontend
* Increased unit test coverage for Magento/Indexer module
* Version number info became accessible at a public URL
* GitHub requests:
* [#1027](https://github.com/magento/magento2/issues/1027) -- Can't add new subctegory
* [#921](https://github.com/magento/magento2/issues/921) -- Change resource ids from Magento_Adminhtml::* to Magento_Backend

0.42.0-beta8
=============
* Various improvements:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ class AjaxMarkAsRead extends \Magento\AdminNotification\Controller\Adminhtml\Not
*/
public function execute()
{
if (!$this->getRequest()->getPost()) {
if (!$this->getRequest()->getPostValue()) {
return;
}
$notificationId = (int)$this->getRequest()->getPost('id');
Expand Down
10 changes: 5 additions & 5 deletions app/code/Magento/AdminNotification/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,15 @@
"description": "N/A",
"require": {
"php": "~5.5.0|~5.6.0",
"magento/module-store": "0.42.0-beta8",
"magento/module-core": "0.42.0-beta8",
"magento/module-backend": "0.42.0-beta8",
"magento/framework": "0.42.0-beta8",
"magento/module-store": "0.42.0-beta9",
"magento/module-core": "0.42.0-beta9",
"magento/module-backend": "0.42.0-beta9",
"magento/framework": "0.42.0-beta9",
"lib-libxml": "*",
"magento/magento-composer-installer": "*"
},
"type": "magento2-module",
"version": "0.42.0-beta8",
"version": "0.42.0-beta9",
"license": [
"OSL-3.0",
"AFL-3.0"
Expand Down
6 changes: 3 additions & 3 deletions app/code/Magento/Authorization/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@
"description": "Authorization module provides access to Magento ACL functionality.",
"require": {
"php": "~5.5.0|~5.6.0",
"magento/module-backend": "0.42.0-beta8",
"magento/framework": "0.42.0-beta8",
"magento/module-backend": "0.42.0-beta9",
"magento/framework": "0.42.0-beta9",
"magento/magento-composer-installer": "*"
},
"type": "magento2-module",
"version": "0.42.0-beta8",
"version": "0.42.0-beta9",
"license": [
"OSL-3.0",
"AFL-3.0"
Expand Down
3 changes: 1 addition & 2 deletions app/code/Magento/Backend/App/AbstractAction.php
Original file line number Diff line number Diff line change
Expand Up @@ -202,8 +202,7 @@ public function dispatch(\Magento\Framework\App\RequestInterface $request)
}

if ($request->isDispatched() && $request->getActionName() !== 'denied' && !$this->_isAllowed()) {
$this->_response->setHeader('HTTP/1.1', '403 Forbidden');
$this->_response->setHttpResponseCode(403);
$this->_response->setStatusHeader(403, '1.1', 'Forbidden');
if (!$this->_auth->isLoggedIn()) {
return $this->_redirect('*/auth/login');
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ protected function _performLogin(\Magento\Framework\App\RequestInterface $reques
$postLogin = $request->getPost('login');
$username = isset($postLogin['username']) ? $postLogin['username'] : '';
$password = isset($postLogin['password']) ? $postLogin['password'] : '';
$request->setPost('login', null);
$request->setPostValue('login', null);

try {
$this->_auth->login($username, $password);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public function aroundDispatch(
if ($key) {
$postData = $request->getPost($key);
$value = is_array($postData) ? $postData : explode(',', $postData);
$request->setPost($key, $value ? $value : null);
$request->setPostValue($key, $value ? $value : null);
}
return $proceed($request);
}
Expand Down
4 changes: 2 additions & 2 deletions app/code/Magento/Backend/Controller/Adminhtml/Denied.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,12 +41,12 @@ public function execute()
if (!$this->_auth->isLoggedIn()) {
/** @var \Magento\Backend\Model\View\Result\Redirect $resultRedirect */
$resultRedirect = $this->resultRedirectFactory->create();
$resultRedirect->setHeader('HTTP/1.1', '403 Forbidden');
$resultRedirect->setStatusHeader(403, '1.1', 'Forbidden');
return $resultRedirect->setPath('*/auth/login');
}
/** @var \Magento\Backend\Model\View\Result\Page $resultPage */
$resultPage = $this->resultPageFactory->create();
$resultPage->setHeader('HTTP/1.1', '403 Forbidden');
$resultPage->setStatusHeader(403, '1.1', 'Forbidden');
$resultPage->addHandle('adminhtml_denied');
return $resultPage;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ public function execute()
{
/** @var \Magento\Backend\Model\View\Result\Page $resultPage */
$resultPage = $this->resultPageFactory->create();
$resultPage->setHeader('HTTP/1.1', '404 Not Found');
$resultPage->setStatusHeader(404, '1.1', 'Forbidden');
$resultPage->setHeader('Status', '404 File not found');
$resultPage->addHandle('adminhtml_noroute');
return $resultPage;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ protected function _filterPostData($data)
*/
public function execute()
{
$data = $this->getRequest()->getPost();
$data = $this->getRequest()->getPostValue();
/** @var \Magento\Backend\Model\View\Result\Redirect $resultRedirect */
$resultRedirect = $this->resultRedirectFactory->create();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ public function execute()
{
/** @var \Magento\Backend\Model\View\Result\Redirect $redirectResult */
$redirectResult = $this->resultRedirectFactory->create();
if ($this->getRequest()->isPost() && ($postData = $this->getRequest()->getPost())) {
if ($this->getRequest()->isPost() && ($postData = $this->getRequest()->getPostValue())) {
if (empty($postData['store_type']) || empty($postData['store_action'])) {
$redirectResult->setPath('adminhtml/*/');
return $redirectResult;
Expand Down
38 changes: 19 additions & 19 deletions app/code/Magento/Backend/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,28 +3,28 @@
"description": "N/A",
"require": {
"php": "~5.5.0|~5.6.0",
"magento/module-store": "0.42.0-beta8",
"magento/module-directory": "0.42.0-beta8",
"magento/module-core": "0.42.0-beta8",
"magento/module-developer": "0.42.0-beta8",
"magento/module-eav": "0.42.0-beta8",
"magento/module-cron": "0.42.0-beta8",
"magento/module-theme": "0.42.0-beta8",
"magento/module-reports": "0.42.0-beta8",
"magento/module-sales": "0.42.0-beta8",
"magento/module-quote": "0.42.0-beta8",
"magento/module-catalog": "0.42.0-beta8",
"magento/module-user": "0.42.0-beta8",
"magento/module-backup": "0.42.0-beta8",
"magento/module-email": "0.42.0-beta8",
"magento/module-customer": "0.42.0-beta8",
"magento/module-translation": "0.42.0-beta8",
"magento/module-require-js": "0.42.0-beta8",
"magento/framework": "0.42.0-beta8",
"magento/module-store": "0.42.0-beta9",
"magento/module-directory": "0.42.0-beta9",
"magento/module-core": "0.42.0-beta9",
"magento/module-developer": "0.42.0-beta9",
"magento/module-eav": "0.42.0-beta9",
"magento/module-cron": "0.42.0-beta9",
"magento/module-theme": "0.42.0-beta9",
"magento/module-reports": "0.42.0-beta9",
"magento/module-sales": "0.42.0-beta9",
"magento/module-quote": "0.42.0-beta9",
"magento/module-catalog": "0.42.0-beta9",
"magento/module-user": "0.42.0-beta9",
"magento/module-backup": "0.42.0-beta9",
"magento/module-email": "0.42.0-beta9",
"magento/module-customer": "0.42.0-beta9",
"magento/module-translation": "0.42.0-beta9",
"magento/module-require-js": "0.42.0-beta9",
"magento/framework": "0.42.0-beta9",
"magento/magento-composer-installer": "*"
},
"type": "magento2-module",
"version": "0.42.0-beta8",
"version": "0.42.0-beta9",
"license": [
"OSL-3.0",
"AFL-3.0"
Expand Down
8 changes: 7 additions & 1 deletion app/code/Magento/Backend/etc/adminhtml/di.xml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,13 @@
<preference for="Magento\Backend\App\ConfigInterface" type="Magento\Backend\App\Config" />
<preference for="Magento\Framework\App\Response\Http\FileFactory" type="Magento\Backend\App\Response\Http\FileFactory" />
<preference for="Magento\Framework\View\Layout\GeneratorPool" type="Magento\Backend\Model\View\Layout\GeneratorPool" />
<preference for="pageLayoutGeneratorPool" type="Magento\Backend\Model\View\Layout\GeneratorPool" />
<type name="Magento\Framework\View\Result\Page">
<arguments>
<argument name="layoutReaderPool" xsi:type="object">pageConfigRenderPool</argument>
<argument name="generatorPool" xsi:type="object">Magento\Backend\Model\View\Layout\GeneratorPool</argument>
<argument name="template" xsi:type="string">Magento_Theme::root.phtml</argument>
</arguments>
</type>
<preference for="Magento\Framework\View\Layout\Reader\Block" type="Magento\Backend\Model\View\Layout\Reader\Block" />
<preference for="Magento\Framework\Model\ActionValidator\RemoveAction" type="Magento\Framework\Model\ActionValidator\RemoveAction\Allowed" />
<preference for="Magento\Framework\Session\Config\ConfigInterface" type="Magento\Backend\Model\Session\AdminConfig" />
Expand Down
10 changes: 5 additions & 5 deletions app/code/Magento/Backup/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@
"description": "N/A",
"require": {
"php": "~5.5.0|~5.6.0",
"magento/module-store": "0.42.0-beta8",
"magento/module-backend": "0.42.0-beta8",
"magento/module-cron": "0.42.0-beta8",
"magento/framework": "0.42.0-beta8",
"magento/module-store": "0.42.0-beta9",
"magento/module-backend": "0.42.0-beta9",
"magento/module-cron": "0.42.0-beta9",
"magento/framework": "0.42.0-beta9",
"magento/magento-composer-installer": "*"
},
"type": "magento2-module",
"version": "0.42.0-beta8",
"version": "0.42.0-beta9",
"license": [
"OSL-3.0",
"AFL-3.0"
Expand Down
32 changes: 16 additions & 16 deletions app/code/Magento/Bundle/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,25 +3,25 @@
"description": "N/A",
"require": {
"php": "~5.5.0|~5.6.0",
"magento/module-store": "0.42.0-beta8",
"magento/module-catalog": "0.42.0-beta8",
"magento/module-tax": "0.42.0-beta8",
"magento/module-core": "0.42.0-beta8",
"magento/module-backend": "0.42.0-beta8",
"magento/module-sales": "0.42.0-beta8",
"magento/module-checkout": "0.42.0-beta8",
"magento/module-catalog-inventory": "0.42.0-beta8",
"magento/module-customer": "0.42.0-beta8",
"magento/module-catalog-rule": "0.42.0-beta8",
"magento/module-eav": "0.42.0-beta8",
"magento/module-gift-message": "0.42.0-beta8",
"magento/framework": "0.42.0-beta8",
"magento/module-webapi": "0.42.0-beta8",
"magento/module-quote": "0.42.0-beta8",
"magento/module-store": "0.42.0-beta9",
"magento/module-catalog": "0.42.0-beta9",
"magento/module-tax": "0.42.0-beta9",
"magento/module-core": "0.42.0-beta9",
"magento/module-backend": "0.42.0-beta9",
"magento/module-sales": "0.42.0-beta9",
"magento/module-checkout": "0.42.0-beta9",
"magento/module-catalog-inventory": "0.42.0-beta9",
"magento/module-customer": "0.42.0-beta9",
"magento/module-catalog-rule": "0.42.0-beta9",
"magento/module-eav": "0.42.0-beta9",
"magento/module-gift-message": "0.42.0-beta9",
"magento/framework": "0.42.0-beta9",
"magento/module-webapi": "0.42.0-beta9",
"magento/module-quote": "0.42.0-beta9",
"magento/magento-composer-installer": "*"
},
"type": "magento2-module",
"version": "0.42.0-beta8",
"version": "0.42.0-beta9",
"license": [
"OSL-3.0",
"AFL-3.0"
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Captcha/Model/Observer.php
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ public function checkUserCreate($observer)
if (!$captchaModel->isCorrect($this->_getCaptchaString($controller->getRequest(), $formId))) {
$this->messageManager->addError(__('Incorrect CAPTCHA'));
$this->_actionFlag->set('', \Magento\Framework\App\Action\Action::FLAG_NO_DISPATCH, true);
$this->_session->setCustomerFormData($controller->getRequest()->getPost());
$this->_session->setCustomerFormData($controller->getRequest()->getPostValue());
$url = $this->_urlManager->getUrl('*/*/create', ['_nosecret' => true]);
$controller->getResponse()->setRedirect($this->redirect->error($url));
}
Expand Down
14 changes: 7 additions & 7 deletions app/code/Magento/Captcha/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,16 @@
"description": "N/A",
"require": {
"php": "~5.5.0|~5.6.0",
"magento/module-store": "0.42.0-beta8",
"magento/module-customer": "0.42.0-beta8",
"magento/module-checkout": "0.42.0-beta8",
"magento/module-core": "0.42.0-beta8",
"magento/module-backend": "0.42.0-beta8",
"magento/framework": "0.42.0-beta8",
"magento/module-store": "0.42.0-beta9",
"magento/module-customer": "0.42.0-beta9",
"magento/module-checkout": "0.42.0-beta9",
"magento/module-core": "0.42.0-beta9",
"magento/module-backend": "0.42.0-beta9",
"magento/framework": "0.42.0-beta9",
"magento/magento-composer-installer": "*"
},
"type": "magento2-module",
"version": "0.42.0-beta8",
"version": "0.42.0-beta9",
"license": [
"OSL-3.0",
"AFL-3.0"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ public function execute()

$storeId = $this->getRequest()->getParam('store');
$refreshTree = false;
$data = $this->getRequest()->getPost();
$data = $this->getRequest()->getPostValue();
if ($data) {
$category->addData($this->_filterCategoryPostData($data['general']));
if (!$category->getId()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ public function __construct(
*/
public function execute()
{
$data = $this->getRequest()->getPost();
$data = $this->getRequest()->getPostValue();
$resultRedirect = $this->resultRedirectFactory->create();
if ($data) {
$setId = $this->getRequest()->getParam('set');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ public function execute()
$productId = $this->getRequest()->getParam('id');
$resultRedirect = $this->resultRedirectFactory->create();

$data = $this->getRequest()->getPost();
$data = $this->getRequest()->getPostValue();
if ($data) {
try {
$product = $this->initializationHelper->initialize($this->productBuilder->build($this->getRequest()));
Expand Down
3 changes: 2 additions & 1 deletion app/code/Magento/Catalog/Controller/Product/Gallery.php
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,8 @@ public function execute()
{
$result = null;
if (!$this->_initProduct()) {
if (isset($_GET['store']) && !$this->getResponse()->isRedirect()) {
$store = $this->getRequest()->getQuery('store');
if (isset($store) && !$this->getResponse()->isRedirect()) {
$result = $this->resultRedirectFactory->create();
$result->setPath('');
} elseif (!$this->getResponse()->isRedirect()) {
Expand Down
3 changes: 2 additions & 1 deletion app/code/Magento/Catalog/Controller/Product/View.php
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,8 @@ public function __construct(
*/
protected function noProductRedirect()
{
if (isset($_GET['store']) && !$this->getResponse()->isRedirect()) {
$store = $this->getRequest()->getQuery('store');
if (isset($store) && !$this->getResponse()->isRedirect()) {
$resultRedirect = $this->resultRedirectFactory->create();
return $resultRedirect->setPath('');
} elseif (!$this->getResponse()->isRedirect()) {
Expand Down
Loading

0 comments on commit 7e7e1e9

Please sign in to comment.