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.

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

Added assets to Injectable property stubs #195

Closed
wants to merge 7 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@thecodeassassin

thecodeassassin commented Mar 24, 2014

Makes it easier to access the assets manager from the controller in an IDE.

@thecodeassassin

This comment has been minimized.

Show comment
Hide comment
@thecodeassassin

thecodeassassin Mar 31, 2014

can somebody take a look at this pull request?

thecodeassassin commented Mar 31, 2014

can somebody take a look at this pull request?

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Apr 29, 2014

Collaborator

IDE stubs are generated using a script, I think this changes are going to be lost.

Collaborator

ghost commented Apr 29, 2014

IDE stubs are generated using a script, I think this changes are going to be lost.

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost May 20, 2014

Collaborator

Please update this script instead of editing the generated stubs: https://github.com/phalcon/phalcon-devtools/blob/master/ide/gen-stubs.php#L202

Collaborator

ghost commented May 20, 2014

Please update this script instead of editing the generated stubs: https://github.com/phalcon/phalcon-devtools/blob/master/ide/gen-stubs.php#L202

@phalcon phalcon closed this May 20, 2014

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