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.

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

Fix for all modules using YiiModule etc aliases #65

Open
wants to merge 65 commits into
base: master
Choose a base branch
from
Open

Fix for all modules using YiiModule etc aliases #65

wants to merge 65 commits into from

Conversation

circulon
Copy link

This Fixes issues with different application structures like Clevertech's Yii Boilerplate

Added YiiModule Alias to all required Modules
Fixed some Modules so they now use internal aliases
This should fix issues when using non standard applecation deployments like Clevertech's Yii Boilerplate
@thyseus
Copy link
Owner

thyseus commented Nov 29, 2012

Hey BlindGenius,

thanks for this work. I would like to merge commit b947244 and 5d72f41 into master - your cleanup is
good and necessary. For backward compatibility i would like to keep the database column names as
they are tough, so i can not agree to the other commits.

@ghost ghost assigned thyseus Nov 29, 2012
@circulon
Copy link
Author

No Worries.

I have not fully tested all the modules yet but have been through them and think I got all the aliases sorted.

NP on the column names as I was just cleaning up as a few of the tables have a few different ways of naming columns.

Happy to help ;)

On 29/11/2012, at 9:06 PM, Herbert Maschke [email protected] wrote:

Hey BlindGenius,

thanks for this work. I would like to merge commit b947244 and 5d72f41 into master - your cleanup is
good and necessary. For backward compatibility i would like to keep the database column names as
they are tough, so i can not agree to the other commits.


Reply to this email directly or view it on GitHub.

Cleaned up some extraneous whitespace
Cleaned up some extraneous whitespace
Cleaned extraneous whitespace
Removed some older incorrect layout calls which caused only the controller action content to be rendered
Cleaned up some extraneous whitespace
Cleaned up some extraneous whitespace
Cleaned extraneous whitespace
Removed some older incorrect layout calls which caused only the controller action content to be rendered
…pache (nginx) servers which don't like short tags
Conflicts:
	registration/RegistrationModule.php
	registration/controllers/YumRegistrationController.php
	user/controllers/YumCsvController.php
	user/controllers/YumUserController.php
	user/views/user/view.php
	usergroup/controllers/YumUsergroupController.php
	usergroup/views/groups/_view.php
- removed accidentally included merge tags
@chiewmey
Copy link

chiewmey commented Jun 6, 2013

Can I know where to find yum installation guide for boilerplate?

@circulon
Copy link
Author

circulon commented Jun 6, 2013

IIRC you can put all the modules in common/modules/
Then in the common/config/main.php follow the install instructions but change the class in the 'user' component to point to the common location instead
ie 'class' => 'application.modules.user.components.YumWebUser',

There was a decent amount of work in changing the absolute paths to relative paths with aliases so you may want to review the changes in my repo
HTH

On 06/06/2013, at 5:28 PM, chiewmey [email protected] wrote:

Can I know where to find yum installation guide for boilerplate?


Reply to this email directly or view it on GitHub.

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.

4 participants