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

Introduce new naming convention for class files #493

Closed
lukecarbis opened this issue May 2, 2014 · 2 comments
Closed

Introduce new naming convention for class files #493

lukecarbis opened this issue May 2, 2014 · 2 comments
Milestone

Comments

@lukecarbis
Copy link
Contributor

Issue #448 Introduces a new file naming convention for the file containing the WP_Stream_Author class. As @westonruter points out, this is inline with the file naming convention in WordPress core.

Any files dedicated to a single class should adopt the filename format of:
class-dash-separated-class-name.php

For example, settings.php which contains the class WP_Stream_Settings should be renamed to class-wp-stream-settings.php.

This is also a good opportunity to explore whether or not it would be helpful to split the includes folder into an includes and a classes folder.

@westonruter
Copy link
Contributor

Also would be nice to use this as an opportunity to implement autoloading, so that we can kiss require_once( 'classes/...' ) goodbye.

@lukecarbis
Copy link
Contributor Author

We'll see this in 2.0.0.

@frankiejarrett frankiejarrett added this to the 2.0.0 milestone Jun 5, 2014
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

3 participants