WordPress.org

Support

Support » Plugins and Hacks » [Resolved] [Plugin: All Authors Page] Bug – Line 420 – case sensitivity

[Resolved] [Plugin: All Authors Page] Bug – Line 420 – case sensitivity

  • Line 420 of allauthorpage.php has an extra < in it before the word There:
    echo “<div style=\”color: red; text-align: center;\” ><h1><There was a problem updating the database</h1></div>

    should be:
    echo “<div style=\”color: red; text-align: center;\” ><h1>There was a problem updating the database</h1></div>

    Also lines 203-206 the user roles all need to be lower case:
    $adm = strpos($roleCheck, ‘admin’);
    $edtr = strpos($roleCheck, ‘editor’);
    $ctrbr = strpos($roleCheck, ‘contributor’);
    $athr = strpos($roleCheck, ‘author’);

    I’ve forked this (privately) and am tweaking a few things. Like changing listing order from highest user role to lowest and the output format. I’m going to quickly hack in some changes I need to the output, but I’ll try if I can to do it clean and add some hooks and fliters so the output can be customized more easily. Get in touch if you want the changes (reply here or twitter @jb510)

    http://wordpress.org/extend/plugins/all-author-page/

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

    @aktdev

    Hey Jon Brown thanks for looking after my plugin ;p

    These are valid points which I have added to the latest update / patch has, plus a few extras.

    Thanks a lot for the tweaks

    This is actually the first time I’m checking the forums 🙁 sorry

    No worries @aktdev! It is a cool and unique plugin glad you’re still working on it.

    Since I have nothing in the repo and even if I did I’m sure I’d go months without checking the forums myself there is no judging going on there.

    Plugin Author AKTDev

    @aktdev

    To be honest I didn’t even know there was a forum added with the plugin :$

    Handy though

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘[Resolved] [Plugin: All Authors Page] Bug – Line 420 – case sensitivity’ is closed to new replies.