amtgard / ORK3

Version 3 of the Online Record Keeper
Other
23 stars 11 forks source link

Convert ORK to PSR-4 #98

Open BenOvermyer opened 8 years ago

BenOvermyer commented 8 years ago

The current version of the ORK uses its own form of autoloading and doesn't make use of namespacing. In order to make it easier to incorporate external resources into the ORK and to better manage dependencies, we should refactor the existing code to conform to PSR-4.

This is a monumental effort, but the end result will be well worth it, especially if it makes a complete rewrite unnecessary.

esdraelon commented 6 years ago

I have been reviewing PSR requirements on and off for the last 2 years. I am not sold on this. I understand the impetus, but it will be a lot of work, and it largely solves a problem which may not be completely pertinent.

Just for reference, I have been writing most of my new code in PSR-standards, including libraries, so I have nothing against it. It's just with limited coding resources available, I'm hoping to focus on value-add for our users.

BenOvermyer commented 6 years ago

See my comment on #99. Same deal; I might spend some time this month on this issue.