Skip to content

Correct plugin generation for vendored plugins.#138

Merged
lorenzo merged 2 commits into
masterfrom
issue-7062
Jul 26, 2015
Merged

Correct plugin generation for vendored plugins.#138
lorenzo merged 2 commits into
masterfrom
issue-7062

Conversation

@markstory
Copy link
Copy Markdown
Member

Add additional template variables and templates to handle vendor plugins correctly.

Refs cakephp/cakephp#7062

Add additional template variables and templates to handle
vendor plugins correctly.

Refs cakephp/cakephp#7062
@markstory markstory added this to the 1.0.11 milestone Jul 25, 2015
@antograssiot
Copy link
Copy Markdown
Contributor

@markstory didn't tested it yet but it looks strange to me that the _modifyAutoloader() function is unchanged because the main application composer.json would need to be update with Company\\Example\\?

@markstory
Copy link
Copy Markdown
Member Author

@antograssiot I missed that as its not in the test cases. I'll get that updated tonight.

@markstory markstory self-assigned this Jul 25, 2015
Use the correctly slashed namespace in the app's composer.json.

Refs #7062
@markstory
Copy link
Copy Markdown
Member Author

@antograssiot Fixed 😄

@antograssiot
Copy link
Copy Markdown
Contributor

Looks good to me 👍

lorenzo added a commit that referenced this pull request Jul 26, 2015
Correct plugin generation for vendored plugins.
@lorenzo lorenzo merged commit 68dcbbf into master Jul 26, 2015
@lorenzo
Copy link
Copy Markdown
Member

lorenzo commented Jul 26, 2015

Great job!

@lorenzo lorenzo deleted the issue-7062 branch July 26, 2015 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants