Feature request from johngrogg, posted on GitHub Jul 22, 2015
It would be super useful to have the Categories assigned to a product be added to the returned data (similar to how extension_attributes are available). Even better if you could filter based on them as well via searchCriteria .
... View more
Feature request from nevvermind, posted on GitHub Jun 18, 2015
As Composer packages get used more and more, if not careful, some of their extra configuration may clash at some point in the future.
Not only that, but having an en-gross extra object, it won't be easy to figure out which config belongs to what package.
It would be great if Magento 2 would spearhead good practice (in my opinion at least) and use namespaced configs. We already have the best candidate for this: the package vendor + name combo - magento/magento2ce :
"extra": {
"magento": {
"magento2ce": {
"component_paths": {
"package-specific": "stuff"
}
}
}
}
If not good practice, at least M2 will shield itself from possible conflicts with other package configs.
This seems like a small change anyway. But with lots of benefits from where I'm standing.
... View more
Feature request from danslo, posted on GitHub Aug 15, 2015
There are things like Magento\Customer\Block\Adminhtml\Edit\BackButton which don't contain any customer specific functionality, preferably they should be moved to a more generic place ( Magento\Framework\View or Magento\Ui ?) so other modules can make use of them without having dependency on Magento_Customer .
... View more
Feature request from mage2pro, posted on GitHub Sep 30, 2015
The issue is similar to (but not the same as) https://github.com/magento/magento2/issues/1909 and https://github.com/magento/magento2/issues/1980
My fix with plugins: https://mage2.pro/t/96
Surely the possibility to translate product attributes should be in the core.
... View more
Feature request from mage2pro, posted on GitHub Nov 02, 2015
The problem line is
https://github.com/magento/magento2/blob/f578e54e093c31378ca981cfe336f7e651194585/app/code/Magento/Cms/Controller/Adminhtml/Page/Save.php#L62
I propose to change it to
if (!$this->dataProcessor->validate($model->getData())) {
My detailed explanation: https://mage2.pro/t/159
... View more
Feature request from licentia, posted on GitHub Oct 15, 2015
Magento should save the fee charged by Paypal for the transaction. This is very useful when trying to calculate Customer Equity & LTV. We have precise values and not approximate ones.
Check this doc: https://developer.paypal.com/docs/classic/api/merchant/GetTransactionDetails_API_Operation_NVP/ and the FEEAMT field.
FEEAMT should be added to the
protected $_getTransactionDetailsResponse
in
Magento\Paypal\Model\Api\Nvp
And stored in the
sales_order_payment table
... View more
Feature request from mage2pro, posted on GitHub Nov 02, 2015
An event on a backend CMS page form creation:
https://github.com/magento/magento2/blob/f578e54e093c31378ca981cfe336f7e651194585/app/code/Magento/Cms/Block/Adminhtml/Page/Edit/Tab/Content.php#L99
No any event on a backend CMS block form creation:
https://github.com/magento/magento2/blob/c743dec47b2e5de036eb5638fec44a54bfb31040/app/code/Magento/Cms/Block/Adminhtml/Block/Edit/Form.php#L61-L158
... View more
Feature request from mage2pro, posted on GitHub Oct 25, 2015
Varien_Object::_construct() :
https://github.com/OpenMage/magento-mirror/blob/1.9.2.1/lib/Varien/Object.php#L154-L156
I see it is already restored (by duplication) in the base model, base block, base form and so on:
\Magento\Framework\Model\AbstractModel::_construct() :
https://github.com/magento/magento2/blob/02e0378c33054acb0cdb8d731d1e2b2c2069bc1b/lib/internal/Magento/Framework/Model/AbstractModel.php#L201-L203
\Magento\Framework\View\Element\AbstractBlock::_construct()
https://github.com/magento/magento2/blob/02e0378c33054acb0cdb8d731d1e2b2c2069bc1b/lib/internal/Magento/Framework/View/Element/AbstractBlock.php#L223-L228
\Magento\Framework\Data\Form\AbstractForm::_construct()
https://github.com/magento/magento2/blob/02e0378c33054acb0cdb8d731d1e2b2c2069bc1b/lib/internal/Magento/Framework/Data/Form/AbstractForm.php#L71-L73
Why does it absent in \Magento\Framework\DataObject ?
... View more
Feature request from mage2pro, posted on GitHub Nov 13, 2015
https://github.com/magento/magento2/blob/2335247d4ae2dc1e0728ee73022b0a244ccd7f4c/app/code/Magento/Catalog/Block/Product/AbstractProduct.php#L374-L379
... View more
Feature request from mage2pro, posted on GitHub Oct 27, 2015
There is an internal field \Magento/Framework/View/Layout/Builder::$isBuilt
https://github.com/magento/magento2/blob/ffa02d82c6344c76ea6de8b50d9612b3200ea2c7/lib/internal/Magento/Framework/View/Layout/Builder.php#L61
Need a public method.
There is an event layout_generate_blocks_after :
https://github.com/magento/magento2/blob/ffa02d82c6344c76ea6de8b50d9612b3200ea2c7/lib/internal/Magento/Framework/View/Layout/Builder.php#L133
But it is inconvenient to listen for the event while an extension just need to get the current layout state: built or not.
... View more
Feature request from mage2pro, posted on GitHub Nov 04, 2015
Similar to https://github.com/magento/magento2/issues/2248 and https://github.com/magento/magento2/issues/2249.
The «controller_action_catalog_product_save_entity_after» event firing :
https://github.com/magento/magento2/blob/16dc76df41fac703b322cc0f9ab3dba43742bbed/app/code/Magento/Catalog/Controller/Adminhtml/Product/Save.php#L105-L108
There is no similar event for a category:
https://github.com/magento/magento2/blob/16dc76df41fac703b322cc0f9ab3dba43742bbed/app/code/Magento/Catalog/Controller/Adminhtml/Category/Save.php#L73-L212
... View more
Feature request from brideo, posted on GitHub Nov 13, 2015
http://php.net/manual/en/language.oop5.late-static-bindings.php
Rather than this:
/**
* @return bool
*/
protected function _isAllowed()
{
return $this->_authorization->isAllowed(self::ADMIN_RESOURCE);
}
We should have this:
/**
* @return bool
*/
protected function _isAllowed()
{
return $this->_authorization->isAllowed(static::ADMIN_RESOURCE);
}
By using a static binding, we don't have to completely re-write the _isAllowed method in our controller action, instead we can simply define our ADMIN_RESOURCE constant, we then can let the AbstractAction handle our logic.
This would be a completely backwards compatible change.
... View more
Feature request from mage2pro, posted on GitHub Nov 17, 2015
See, for example:
https://github.com/magento/magento2/blob/2335247d4ae2dc1e0728ee73022b0a244ccd7f4c/app/code/Magento/Catalog/view/adminhtml/layout/catalog_product_new.xml#L55
<block
class="Magento\Catalog\Block\Adminhtml\Product\Edit\Js"
name="catalog_product_js"
template="catalog/product/js.phtml"
/>
is the same as
<block
class="Magento\Catalog\Block\Adminhtml\Product\Edit\Js"
name="catalog_product_js"
template="Magento_Catalog::catalog/product/js.phtml"
/>
But it is not possible for CSS and JS declarations:
https://github.com/magento/magento2/blob/2335247d4ae2dc1e0728ee73022b0a244ccd7f4c/app/code/Magento/Catalog/view/adminhtml/layout/catalog_product_new.xml#L12
<css src="Magento_Catalog::product/product.css"/>
It would be nice to have an abilily skip the current module prefix and use some shorter syntax like <css src="product/product.css"/> or <css src="::product/product.css"/>
... View more
Feature request from mage2pro, posted on GitHub Nov 16, 2015
https://mage2.pro/t/topic/209
... View more
Feature request from hiephm, posted on GitHub Jan 18, 2016
Although Magento 2 allows hooking to event when logging in to add additional validations (just like Magento 1), there is no way to change the error message if those validation failed (e.g by throwing Exception) since a default message is used for general Exception:
# \Magento\Customer\Controller\Account\LoginPost::execute()
...
} catch (\Exception $e) {
$this->messageManager->addError(__('Invalid login or password.'));
}
In the other hand, the same logic in Magento 1 is more customizable:
# \Mage_Customer_AccountController::loginPostAction()
switch ($e->getCode()) {
...
default:
$message = $e->getMessage();
}
$session->addError($message);
... View more
Feature request from brendanmckeown, posted on GitHub Jan 20, 2016
Both of the API routes in module-integration/etc/webapi.xml to get an authentication token work over http. These routes should be secure and only acceptable over https, because you are passing sensitive account information in the request body. This can be achieved by adding secure="true" to each <route> node:
<route url="/V1/integration/admin/token" method="POST" secure="true">
<service class="Magento\Integration\Api\AdminTokenServiceInterface" method="createAdminAccessToken"/>
<resources>
<resource ref="anonymous"/>
</resources>
</route>
<route url="/V1/integration/customer/token" method="POST" secure="true">
<service class="Magento\Integration\Api\CustomerTokenServiceInterface" method="createCustomerAccessToken"/>
<resources>
<resource ref="anonymous"/>
</resources>
</route>
If this was an intentional decision, is there a way I can override this route in a custom module and enforce this behavior?
... View more
Feature request from ovdev1, posted on GitHub Jan 30, 2016
Just noticed when trying to load table rates that the directory_country_region table contains the wrong state code for Queensland Australia, should be QLD not QSD
... View more
Feature request from ankursachdeva11, posted on GitHub Feb 01, 2016
I am trying to create custom api using webapi and we need output in custom XML.
When we are trying to return some XML and hit the rest API url then XML shown as plain text in the .... tags. It is not creating XML in response.
it shows like this :
Is there any way to modify the response tag to get custom XML response.
... View more