WordPress.org

Support

Support » Plugins and Hacks » Ultimate Taxonomy Manager » [Resolved] [Plugin: Ultimate Taxonomy Manager] PHP Notices

[Resolved] [Plugin: Ultimate Taxonomy Manager] PHP Notices

  • See my other post. You MUST turn on PHP notices while developing, otherwise you won’t be able to tell when you’re writing bad code. And when someone tries to use this plugin on a server that is displaying PHP notices, they get notices popping up everywhere in the wp manager.

    Your code looks pretty good and pretty clean — nice job. The biggest thing you have to fix is when you read variables out of the $_GET array.

    E.g. this will throw a notice if your URL doesn’t have a ‘myvar’ parameter:

    if ( $_GET['myvar'] == 'something' )
    {
     // ... do something
    }

    Instead, you should spend the extra time to check whether or not the place in the array is set:

    if ( isset($_GET['myvar']) && $_GET['myvar'] == 'something' )
    {
      // ... do something
    }

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author XYDAC

    @deepakseth

    Hey, i had the isset checks at many places but as my error reeportin was off snd it didnlt show the warnings so i kinda missed checking everywhere.
    thanks for pointing the issue mate, i’ll make sure i remove the notices asap.

    Plugin Author XYDAC

    @deepakseth

    fixed: let me know if you still find any

    Thanks!

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘[Resolved] [Plugin: Ultimate Taxonomy Manager] PHP Notices’ is closed to new replies.
Skip to toolbar