[#2309] - Removes the class aliases for the DI package.
- Closed
- 22 Oct 2013
- Medium
- Build: master
- # 2309
- Diff
- eddieajau:remove-di-aliases
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This PR removes the class alias file for the DI package from the framework. It is far better to teach developers the right way to deal with namespacing from day-one as this will open more opportunities for more of the Framework or other PSR-0 compatible code to be included in the CMS.
Ok. Since this is just correcting a new feature anyway, can we just merge it or do I need to double handle it in JoomlaCode?
Based on the fact that this is affecting new code committed only in this same beta phase, this is considered an update to that submitted code and not regarded as a bug or new feature but rather an update to an existing, accepted and merged PR.
garyamort
-
I would tend to agree with you. And since there hasn't been a CMS release with these classes in place, it is vital that if we are going to remove them, we do it sooner rather than later.