Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change resource ids from Magento_Adminhtml::* to Magento_Backend #921

Closed
tzyganu opened this issue Jan 6, 2015 · 2 comments
Closed

Change resource ids from Magento_Adminhtml::* to Magento_Backend #921

tzyganu opened this issue Jan 6, 2015 · 2 comments
Labels
bug report

Comments

@tzyganu
Copy link
Contributor

tzyganu commented Jan 6, 2015

In the file /app/code/Magento/Backend/etc/acl.xml (and all the other acl.xml files) the prefix for the resource ids is Magento_Adminhtml::*. Example:

<resource id="Magento_Adminhtml::myaccount" title="My Account" sortOrder="50" />

It should be

<resource id="Magento_Backend::myaccount" title="My Account" sortOrder="50" />

since the Adminhtml module does not exist.
@vpelipenko
Copy link
Contributor

@tzyganu Thank you for contribution! We have created the issue in our internal bug tracker and will address it once we have it prioritized. But if you want to help with this fix, your pull requests are welcome.

Ticket: MAGETWO-32668

@vpelipenko vpelipenko added PS and removed CS labels Jan 13, 2015
anupdugar added a commit that referenced this issue Feb 20, 2015
@anupdugar
MAGETWO-32668: [GITHUB] Change resource ids from Magento_Adminhtml::*…
f16d24e 
… to Magento_Backend #921

- Replaced Magento_Adminhtml with Magento_Backend
@vpelipenko
Copy link
Contributor

@tzyganu, this issue was fixed and available here from 0.42.0-beta9 version. You can checkout it and verify.

magento-team pushed a commit that referenced this issue Apr 11, 2017
Merge pull request #921 from magento-tsg/2.1.6-develop-pr14
506a93f 
[TSG] Backporting for 2.1 (pr14)
magento-team pushed a commit that referenced this issue Dec 28, 2017
@omiroshnichenko
MAGETWO-84665: 8830: Can`t delete row in dynamicRows component #921
97def3a 
 - Merge Pull Request magento-engcom/magento2ce#921 from RomaKis/magento2:8830
 - Merged commits:
   1. 0a6a2d9
   2. 53c1fa3
   3. 1ab55ec
   4. cb24115
   5. bfb2e38
   6. 568d36e
   7. 0a72897
magento-team pushed a commit that referenced this issue Dec 28, 2017
MAGETWO-84665: 8830: Can`t delete row in dynamicRows component #921
e862fe8
magento-team pushed a commit that referenced this issue May 15, 2018
@OlgaVasyltsun
MAGETWO-90285: 8830: Can`t delete row in dynamicRows component #921
6a11839
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this issue Jun 22, 2018
@maghamed
Merge pull request magento#921 from magento-engcom/822-source-deducti…
8ffcf0a 
…on-for-virtual

MSI#822 Source deduction for virtual and downloadable products
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug report
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@maksek @tzyganu @vpelipenko