fucongcong/framework

fix_gpc_magic accesses the super-global variable $_POST.

Open

    public function fix_gpc_magic()
    {
        static $fixed = false;
        if (!$fixed && ini_get('magic_quotes_gpc')) {
            array_walk($_GET, '_fix_gpc_magic');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

fix_gpc_magic accesses the super-global variable $_FILES.

Open

    public function fix_gpc_magic()
    {
        static $fixed = false;
        if (!$fixed && ini_get('magic_quotes_gpc')) {
            array_walk($_GET, '_fix_gpc_magic');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

fix_gpc_magic accesses the super-global variable $_COOKIE.

Open

    public function fix_gpc_magic()
    {
        static $fixed = false;
        if (!$fixed && ini_get('magic_quotes_gpc')) {
            array_walk($_GET, '_fix_gpc_magic');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

fix_gpc_magic accesses the super-global variable $_GET.

Open

    public function fix_gpc_magic()
    {
        static $fixed = false;
        if (!$fixed && ini_get('magic_quotes_gpc')) {
            array_walk($_GET, '_fix_gpc_magic');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

fix_gpc_magic accesses the super-global variable $_REQUEST.

Open

    public function fix_gpc_magic()
    {
        static $fixed = false;
        if (!$fixed && ini_get('magic_quotes_gpc')) {
            array_walk($_GET, '_fix_gpc_magic');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The method _fix_gpc_magic uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        else {
            $item = stripslashes($item);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The method _fix_gpc_magic_files uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            else {
              $item = stripslashes($item);
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Avoid unused private methods such as '_fix_gpc_magic'.

Open

    private static function _fix_gpc_magic(&$item)
    {
        if (is_array($item)) {
            array_walk($item, '_fix_gpc_magic');
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Avoid unused private methods such as '_fix_gpc_magic_files'.

Open

    private static function _fix_gpc_magic_files(&$item, $key)
    {
        if ($key != 'tmp_name') {
            if (is_array($item)) {
              array_walk($item, '_fix_gpc_magic_files');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Method name "Kernal::_fix_gpc_magic_files" is not in camel caps format

Open

    private static function _fix_gpc_magic_files(&$item, $key)

• Exclude checks
  • Disable engine
  • Disable check

Method name "Kernal::fix_gpc_magic" is not in camel caps format

Open

    public function fix_gpc_magic()

• Exclude checks
  • Disable engine
  • Disable check

Method name "Kernal::_fix_gpc_magic" is not in camel caps format

Open

    private static function _fix_gpc_magic(&$item)

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

Method name "_fix_gpc_magic_files" should not be prefixed with an underscore to indicate visibility

Open

    private static function _fix_gpc_magic_files(&$item, $key)

• Exclude checks
  • Disable engine
  • Disable check

Method name "_fix_gpc_magic" should not be prefixed with an underscore to indicate visibility

Open

    private static function _fix_gpc_magic(&$item)

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing brace; newline found

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 16 spaces, found 14

Open

              $item = stripslashes($item);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 16 spaces, found 14

Open

              array_walk($item, '_fix_gpc_magic_files');

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing brace; newline found

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

The method _fix_gpc_magic is not named in camelCase.

Open

    private static function _fix_gpc_magic(&$item)
    {
        if (is_array($item)) {
            array_walk($item, '_fix_gpc_magic');
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The method fix_gpc_magic is not named in camelCase.

Open

    public function fix_gpc_magic()
    {
        static $fixed = false;
        if (!$fixed && ini_get('magic_quotes_gpc')) {
            array_walk($_GET, '_fix_gpc_magic');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The method _fix_gpc_magic_files is not named in camelCase.

Open

    private static function _fix_gpc_magic_files(&$item, $key)
    {
        if ($key != 'tmp_name') {
            if (is_array($item)) {
              array_walk($item, '_fix_gpc_magic_files');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check