Showing 4,293 of 4,293 total issues
The method _attributesList is not named in camelCase. Open
Open
private function _attributesList()
{
$returnstr = '';
foreach ($this->attributes as $attr => $value) {
$returnstr .= "{$attr}='{$value}' ";- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method _renderAttributes is not named in camelCase. Open
Open
private function _renderAttributes()
{
$return = "class='{$this->_classList()}' ";
$return .= $this->_attributesList();
- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method _getSubpackage is not named in camelCase. Open
Open
private static function _getSubpackage( $result, $context )
{
if (is_null( $result )) {
$result = self::$packages;
}- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
TODO found Open
Open
//@TODO rewrite, this is old- Exclude checks
TODO found Open
Open
// @TODO other properties?- Exclude checks
TODO found Open
Open
// @TODO max values to options- Exclude checks
TODO found Open
Open
// @TODO 157 159 was commented out, and that was probably correct- Exclude checks
TODO found Open
Open
* @TODO: probably not supposed to be here- Exclude checks
TODO found Open
Open
// TODO Re-initialize ui components- Exclude checks
TODO found Open
Open
// TODO Explanation text for non-developers on page- Exclude checks
TODO found Open
Open
return parent::save($new, $old); // TODO: Change the autogenerated stub- Exclude checks
TODO found Open
Open
* @TODO 'get' would be more appropriate- Exclude checks
TODO found Open
Open
// @TODO a module class which was assigned to an area is not necessarily present- Exclude checks
TODO found Open
Open
KB.Events.trigger('kb.repaint'); // @TODO figure this out- Exclude checks
TODO found Open
Open
* TODO: Don't return non existing modules- Exclude checks
TODO found Open
Open
// @TODO remove- Exclude checks
TODO found Open
Open
// TODO clean up and remove all references & bindings- Exclude checks
TODO found Open
Open
// TODO maybe check if admin bar is around- Exclude checks
TODO found Open
Open
* @TODO clean up module header from legacy code- Exclude checks
TODO found Open
Open
// TODO Include a public context switch- Exclude checks