Opened 9 years ago
Closed 9 years ago
#35004 closed defect (bug) (wontfix)
All code should adhere to WordPress coding standards
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 4.6 |
Component: | External Libraries | Keywords: | |
Focuses: | Cc: |
Description (last modified by )
[34981] committed by @dd32 does not adhere to the WordPress Coding Standards
Specifically:
Files should be named descriptively using lowercase letters. Hyphens should separate words.
I understand this is an upstream library, but renaming the files and their includes is trivial and makes things much more consistent. There is a precedent already as we do this with PHPMailer and probably other 3rd party libs.
Change History (4)
#2
@
9 years ago
For reference, the changeset being referred to is https://build.trac.wordpress.org/changeset/34946 or [34981].
Agreed with @dd32 on this one.
Note: See
TracTickets for help on using
tickets.
The libraries which we have renamed are also single file libraries (phpmailer, phpass, pemftp). Other libraries we use (id3, diff, simplepie, etc) don't get renamed either - they just don't include underscores in filenames.
For simplicity, I opted to include the library as-is to reduce the maintenance burden of upgrading it, and although underscores are not in line with our naming, cause no ill effects (which is also why we keep TinyMCE as-is, also including underscores).