dev:ide:phpstorm:meta does not detect some aliases #108

Closed
amenk opened this Issue Apr 5, 2013 · 4 comments

Comments

Projects
None yet
3 participants
@amenk
Contributor

amenk commented Apr 5, 2013

Example: The helper catalog/product (Should resolve to Mage_Catalog_Helper_Product) is not listed in the meta file.

@adrian-green

This comment has been minimized.

Show comment Hide comment
@adrian-green

adrian-green Apr 6, 2013

I think this is because Mage/* helpers are not declared in config xml, and therefore are not in Config global/helpers. (The method getClassMapForGroup walks the models and helpers nodes).
I suppose another strategy might be to glob files in "Helper" directories and go from there...

I think this is because Mage/* helpers are not declared in config xml, and therefore are not in Config global/helpers. (The method getClassMapForGroup walks the models and helpers nodes).
I suppose another strategy might be to glob files in "Helper" directories and go from there...

@cmuench

This comment has been minimized.

Show comment Hide comment
@cmuench

cmuench Apr 6, 2013

Member

Thanks for the hint. I try to fix that for the mage modules.

Member

cmuench commented Apr 6, 2013

Thanks for the hint. I try to fix that for the mage modules.

@cmuench

This comment has been minimized.

Show comment Hide comment
@cmuench

cmuench Apr 6, 2013

Member

I checked the helper definitions and it's really instable. Some core module has a definition and some not.

This is the list of modules which have not an "global/helpers" entry in config.xml:

Backup
Bundle
Captcha
Catalog
CatalogIndex
Centinel
Checkout
Cms
Customer
Dataflow
Directory
Downloadable
Eav
Index
Install
Log
Media
Newsletter
Page
Payment
Paypal
Persistent
Poll
Rating
Reports
Review
Rss
Rule
Sales
Shipping
Sitemap
Tag
Tax
Usa
Weee
Widget
Wishlist

Member

cmuench commented Apr 6, 2013

I checked the helper definitions and it's really instable. Some core module has a definition and some not.

This is the list of modules which have not an "global/helpers" entry in config.xml:

Backup
Bundle
Captcha
Catalog
CatalogIndex
Centinel
Checkout
Cms
Customer
Dataflow
Directory
Downloadable
Eav
Index
Install
Log
Media
Newsletter
Page
Payment
Paypal
Persistent
Poll
Rating
Reports
Review
Rss
Rule
Sales
Shipping
Sitemap
Tag
Tax
Usa
Weee
Widget
Wishlist

cmuench added a commit that referenced this issue Apr 6, 2013

@cmuench

This comment has been minimized.

Show comment Hide comment
@cmuench

cmuench Apr 6, 2013

Member

I just created a new version in develop branch with a patch for Data helper (by @adrian-green) and a patch for missing core module definitions.
It's also now tested with a unit test.

Member

cmuench commented Apr 6, 2013

I just created a new version in develop branch with a patch for Data helper (by @adrian-green) and a patch for missing core module definitions.
It's also now tested with a unit test.

@amenk amenk referenced this issue in enriquepiatti/Magicento Apr 9, 2013

Closed

Some core definitions in PHPstorm meta file #38

@cmuench cmuench closed this Apr 26, 2013

steverobbins pushed a commit to steverobbins/n98-magerun that referenced this issue Apr 7, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment