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

autoloaded library/classes via composer classmap #422

Merged
merged 1 commit into from Jan 13, 2017
Merged

autoloaded library/classes via composer classmap #422

merged 1 commit into from Jan 13, 2017

Conversation

bradymiller
Copy link
Sponsor Member

@bradymiller bradymiller commented Jan 6, 2017 •
edited

Autoloading classes in OpenEMR.

@bradymiller bradymiller force-pushed the composer-autoload-trial_1 branch 5 times, most recently from 174f41e to 60640c4 Compare January 8, 2017 08:21
@bradymiller
Copy link
Sponsor Member Author

bradymiller commented Jan 11, 2017 •
edited

I think I need to place a use at top of scripts to track where the classes are used (and it's a good habit), which will make going to namespaces in the future much easier.
(this was a bad idea and just threw a bunch of php warnings, so not a good idea to do use for all classes; will need to rely on codebase search to seek out use of classes when incorporate namespaces on legacy classes in the future)

@bradymiller
Copy link
Sponsor Member Author

Extensive testing went well, so bringing this into the codebase.
See here for more details of this ongoing project:
http://www.open-emr.org/wiki/index.php/Composer#Autoloader

@bradymiller bradymiller merged commit 262062d into openemr:master Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@bradymiller