- Bugs
- Reporting a Bug
- Fixing a Bug
- Features
- Security Vulnerabilities
- Coding Style
- PHPDoc
- Naming Conventions
- Variables
- Functions/Methods
- Constants
- Namespaces
- Classes
- Abstract Classes
- Interfaces
- Traits
Before you attempt to write a bug fix, first read the documentation to see if you're perhaps using Opulence incorrectly.
To report a bug, create a new issue with a descriptive title, steps to reproduce the bug (eg a failing PHPUnit test), and information about your environment.
To fix a bug, create a pull request on the latest stable branch (1.2) of the main Opulence repository with the fix and relevant PHPUnit tests.
We always appreciate when you want to add a new feature to Opulence. For minor, backwards-compatible features, create a pull request on the latest stable branch (1.2) of the main Opulence repository. Do not submit pull requests to individual libraries' repositories. For major, possibly backwards-incompatible features, create a pull request on the develop branch. All new features should come with PHPUnit tests proving their functionality. Pull requests should never be sent to the master branch.
Opulence strives to not create any unnecessary library dependencies. This even includes having dependencies on other Opulence libraries. If your change will introduce a new dependency to a library, create an issue and ask about it before implementing it. If your feature is a useful combination of multiple Opulence libraries, it's possible it'll be recommended to go into the Opulence\Framework library.
Opulence takes security seriously. If you find a security vulnerability, please email us at bugs@opulencephp.com.
Opulence follows PSR-1 and PSR-2 coding standards and uses PSR-4 autoloading. It uses StyleCI to automatically enforce code style, so you don't have to worry about that. If you'd like to run the formatter locally, a config for the PHP CS Fixer is bundled with Opulence.
Use PHPDoc to document all class properties, methods, and functions. Constructors only need to document the parameters. Method/function PHPDoc must include one blank line between the description and the following tag. Here's an example:
class Book
{
/** @var string The title of the book */
private $title;
/**
* @param string $title The title of the book
*/
public function __construct(string $title)
{
$this->setTitle($title);
}
/**
* Gets the title of the book
*
* @return string The title of the book
*/
public function getTitle() : string
{
return $this->title;
}
/**
* Sets the title of the book
*
* @param string $title The title of the book
* @return $this For object chaining
*/
public function setTitle(string $title) : self
{
$this->title = $title;
return $this;
}
}Inspired by Code Complete, Opulence uses a straightforward approach to naming things.
All variable names:
- Must be lower camel case, eg
$emailAddress - Must NOT use Hungarian Notation
All function/method names:
- Must be succinct
- Your method name should describe exactly what it does, nothing more, and nothing less
- If you are having trouble naming a method, that's probably a sign that it is doing too much and should be refactored
- Must be lower camel case, eg
compileList()- Acronyms in function/method names ≤ 2 characters long, capitalize each character, eg
startIO() - "Id" is an abbreviation (not an acronym) for "Identifier", so it should be capitalized
Id
- Acronyms in function/method names ≤ 2 characters long, capitalize each character, eg
- Must answer a question if returning a boolean variable, eg
hasAccess()oruserIsValid()- Always think about how your functio
7FE7
n/method will be read aloud in an
ifstatement.if (userIsValid())reads better thanif (isUserValid()).
- Always think about how your functio
7FE7
n/method will be read aloud in an
- Must use
getXXX()andsetXXX()for functions/methods that get and set properties, respectively- Don't name a method that returns a username
username(). Name itgetUsername()so that its purpose is unambiguous.
- Don't name a method that returns a username
All class constants' names:
- Must be upper snake case, eg
TYPE_SUBSCRIBER
All namespaces:
- Must be Pascal case, eg
Opulence\QueryBuilders- For namespace acronyms ≤ 2 characters long, capitalize each character, eg
IO
- For namespace acronyms ≤ 2 characters long, capitalize each character, eg
All class names:
- Must be succinct
- Your class name should describe exactly what it does, nothing more, and nothing less
- If you are having trouble naming a class, that's probably a sign that it is doing too much and should be refactored
- Must be Pascal case, eg
ListCompiler- For class name acronyms ≤ 2 characters long, capitalize each character, eg
IO - Class filenames should simply be the class name with .php appended, eg ListCompiler.php
- For class name acronyms ≤ 2 characters long, capitalize each character, eg
Class properties should appear before any methods. The following is the preferred ordering of class properties and methods:
- Constants
- Public static properties
- Public properties
- Protected static properties
- Protected properties
- Private static properties
- Private properties
- Magic methods
- Public static methods
- Public abstract methods
- Public methods
- Protected static methods
- Protected abstract methods
- Protected methods
- Private static methods
- Private methods
Note: Methods of the same visibility should be ordered alphabetically.
All abstract class names:
- Must be Pascal case, eg
ConnectionPool - Must NOT use
Abstract,Base, or any other word in the name that implies it is an abstract class
All interface names:
- Must be preceded by an
I, egIUser
All trait names:
- Must be Pascal case, eg
ListValidator - Must be NOT use
T,Trait, or any other word in the name that implies it is a trait