XoopsModules25x/xfguestbook

View on GitHub

Showing 1,027 of 1,027 total issues

The method get_user_data is not named in camelCase.
Open

    public static function get_user_data($user_id)
    {
        global $xoopsUser;
        /** @var Helper $helper */
        $helper = Helper::getInstance();
Severity: Minor
Found in class/Utility.php by phpmd

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 get_badips is not named in camelCase.
Open

    public static function get_badips($all = null)
    {
        global $xoopsDB;
        $ret    = [];
        $sql    = 'SELECT * FROM ' . $xoopsDB->prefix('xfguestbook_badips');
Severity: Minor
Found in class/Utility.php by phpmd

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 email_exist is not named in camelCase.
Open

    public static function email_exist($email)
    {
        if (!\filter_var($email, \FILTER_VALIDATE_EMAIL)) {
            return false;
        }
Severity: Minor
Found in class/Utility.php by phpmd

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

XXX found
Open

//$XXXHandler = xoops_getModuleHandler('XXX', $moduleDirName);
Severity: Minor
Found in header.php by fixme

TODO found
Open

        TODO if you want to use the render code below,
Severity: Minor
Found in class/Common/Breadcrumb.php by fixme

TODO found
Open

                \curl_setopt($curlHandle, \CURLOPT_SSL_VERIFYPEER, true); //TODO: how to avoid an error when 'Peer's Certificate issuer is not recognized'
Severity: Minor
Found in class/Common/VersionChecks.php by fixme

TODO found
Open

        // TODO set exportSchema
Severity: Minor
Found in testdata/index.php by fixme
Severity
Category
Status
Source
Language