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

Adapt existing code to PSR-4 #39

Closed
alombarte opened this issue Jan 13, 2014 · 1 comment
Closed

Adapt existing code to PSR-4 #39

alombarte opened this issue Jan 13, 2014 · 1 comment
Assignees
Milestone

Comments

@alombarte
Copy link
Contributor

All code should be converted to make it PSR-4 compliant.
This is (from http://www.php-fig.org/psr/psr-4/):

  • The term "class" refers to classes, interfaces, traits, and other similar structures.
    A fully qualified class name has the following form:
    ()*
  • The fully qualified class name MUST have a top-level namespace name, also known as a "vendor namespace".
  • The fully qualified class name MAY have one or more sub-namespace names.
  • The fully qualified class name MUST have a terminating class name.
  • Underscores have no special meaning in any portion of the fully qualified class name.
  • Alphabetic characters in the fully qualified class name MAY be any combination of lower case and upper case.
  • All class names MUST be referenced in a case-sensitive fashion.
  • When loading a file that corresponds to a fully qualified class name ...
  • A contiguous series of one or more leading namespace and sub-namespace names, not including the leading namespace separator, in the fully qualified class name (a "namespace prefix") corresponds to at least one "base directory".
  • The contiguous sub-namespace names after the "namespace prefix" correspond to a subdirectory within a "base directory", in which the namespace separators represent directory separators. The subdirectory name MUST match the case of the sub-namespace names.
  • The terminating class name corresponds to a file name ending in .php. The file name MUST match the case of the terminating class name.
  • Autoloader implementations MUST NOT throw exceptions, MUST NOT raise errors of any level, and SHOULD NOT return a value.
@ghost ghost assigned thedae Jan 13, 2014
@alombarte alombarte assigned lombartec and unassigned thedae Oct 8, 2015
@lombartec
Copy link
Member

Thanks for creating the task, I will have some time this weekend to do this 😄

obokaman-com added a commit that referenced this issue Oct 9, 2015
- Move tests to "tests/Sifo" folder to reproduce the same structure under /src
- Apply correct namespace to /src/Sifo files (most of them keep the same namespace. Only those under /Cache and /Debug folders have changed.)
- Fix names for those classes under /Cache and /Debug folders.
- Separate some classes in multiple files (MysqlStatement for instance).
- Add sifo-common-instance
obokaman-com added a commit that referenced this issue Oct 9, 2015
- Move tests to "tests/Sifo" folder to reproduce the same structure under /src
- Apply correct namespace to /src/Sifo files (most of them keep the same namespace. Only those under /Cache and /Debug folders have changed.)
- Fix names for those classes under /Cache and /Debug folders.
- Separate some classes in multiple files (MysqlStatement for instance).
- Add sifo-common-instance
- Add coverall actions to travis config.
@obokaman-com obokaman-com changed the title Adapt existing code to PSR-0 Adapt existing code to PSR-4 Oct 9, 2015
alombarte added a commit that referenced this issue Oct 11, 2015
Related with issue #39 (Adapt to PSR-4)
alombarte added a commit that referenced this issue Oct 12, 2015
- Bulk changes to adapt to PSR-2
alombarte added a commit that referenced this issue Oct 12, 2015
- Bulk changes to adapt to PSR-2 Smarty Plugins
alombarte added a commit that referenced this issue Oct 12, 2015
@kpicaza kpicaza closed this as completed Feb 25, 2022
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

No branches or pull requests

4 participants