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

Trace: Use autoloader #119

Open
ScreamingDev opened this issue Jul 28, 2017 · 0 comments
Open

Trace: Use autoloader #119

ScreamingDev opened this issue Jul 28, 2017 · 0 comments

Comments

@ScreamingDev
Copy link
Contributor

ScreamingDev commented Jul 28, 2017

Rollover this issue among releases. This one just keeps track of a bigger process by linking pull requests to it.

Currently some classes are included automagically and some manually.
The manual ones are upsetting when you want to reuse a class in some other context way before such classes exists or have been loaded,
which can't be loaded earlier as they contain logic and other calls beside the class itself.
Simple solution would be an autoloader.

  • Remove logic from all class files (only class itself should be in there).

This issue keeps track of refactoring the code so that the already implemented autoloader can be used properly. Let's start a list and edit important things to it that come up during this process:

  • We deal with namespaces here so we no longer support PHP 5.2 .
@ScreamingDev ScreamingDev added this to the 1.5.0 milestone Jul 28, 2017
@ScreamingDev ScreamingDev self-assigned this Jul 28, 2017
@ScreamingDev ScreamingDev removed their assignment Sep 6, 2017
@ScreamingDev ScreamingDev removed this from the 1.5.0 milestone Sep 6, 2017
ScreamingDev pushed a commit that referenced this issue Sep 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant