WordPress.org

Ready to get started?Download WordPress

Forums

AVH Extended Categories Widgets
[resolved] [PATCH] Lots of errors in WP3.6 (15 posts)

  1. flynsarmy
    Member
    Posted 10 months ago #

    After upgrading to WP3.6 I'm getting a bunch of errors like:

    Strict standards: Declaration of AVH_Walker_Category_Checklist::start_lvl() should be compatible with Walker::start_lvl(&$output, $depth = 0, $args = Array) in /extended-categories-widget/3.3/class/avh-ec.widgets.php on line 1003

    this is caused by you not implimenting the functions in the same way the Walker class impliments them - ie no default values.

    In /extended-categories-widget/3.3/class/avh-ec.widgets.php and /extended-categories-widget/3.3/class/avh-ec.core.php
    change

    function start_lvl (&$output, $depth, $args)
    function end_lvl (&$output, $depth, $args)
    function start_el (&$output, $category, $depth, $args)
    function end_el (&$output, $category, $depth, $args)

    to

    function start_lvl( &$output, $depth = 0, $args = array() )
    function end_lvl( &$output, $depth = 0, $args = array() )
    function start_el( &$output, $object, $depth = 0, $args = array(), $current_object_id = 0 )
    function end_el( &$output, $object, $depth = 0, $args = array() )
    function walk( $elements, $max_depth)

    You have a third argument ($args) for your walk method that doesn't exist in the Walker class. I'm not sure how you'd fix this other than to rename your walk method to avh_walk() then update your references in avh_wp_category_checklist() and avh_wp_group_category_checklist() methods.

    Next up is

    Strict standards: Non-static method AVH_EC_Singleton::getInstance() should not be called statically, assuming $this from incompatible context in /path/to/plugins/extended-categories-widget/3.3/class/avh-ec.admin.php on line 28

    In /extended-categories-widget/3.3/avh-ec.client.php change

    function &getInstance ($class, $arg1 = null)

    to

    public static function &getInstance ($class, $arg1 = null)

    And finally

    Strict standards: Redefining already defined constructor for class AVH_EC_Category_Group in /extended-categories-widget/3.3/class/avh-ec.category-group.php on line 36

    Just remove the PHP5 constructor and dump all the code in your PHP4 constructor. Should remove this warning.

    http://wordpress.org/plugins/extended-categories-widget/

  2. flynsarmy
    Member
    Posted 10 months ago #

    One more I missed. on the post list page in admin I'm getting

    Strict standards: Only variables should be passed by reference in /path/to/wp-content/plugins/extended-categories-widget/3.3/class/avh-ec.core.php on line 209

    It's annoying and wasteful but in method getBaseDirectory() change

    $return = end(explode('/', $directory));

    to

    $directories = explode('/', $directory);
    $return = end($directories);
  3. stephancaspar
    Member
    Posted 10 months ago #

    @flynsarmy: Do you have a running version for wp 3.6?

  4. flynsarmy
    Member
    Posted 10 months ago #

    Download here. Let me know once you've got it so I can remove from my dropbox

  5. flynsarmy
    Member
    Posted 10 months ago #

    Dupe. ignore

  6. stephancaspar
    Member
    Posted 10 months ago #

    Thanks a lot. Downloaded it, testing it on monday.

  7. stephancaspar
    Member
    Posted 10 months ago #

    Testing right now. The site recognises the associated group (which can be seen by introducing custom widget titles), but does not display the corresponding categories. Any Idea why?

  8. flynsarmy
    Member
    Posted 10 months ago #

    No idea. I didn't change any code in particular in my version, just fixed the warnings. You'll have to debug any issues yourself.

  9. flynsarmy
    Member
    Posted 9 months ago #

    Another two:

    Notice: Undefined variable: seldata in /path/to/wp-content/plugins/extended-categories-widget/3.3/class/avh-ec.admin.php on line 101

    At the top of the displayCategoryGroupForm method add

    $seldata = '';

    And finally

    Notice: Undefined offset: 47 in /path/to/wp-content/plugins/extended-categories-widget/3.3/class/avh-ec.admin.php on line 130

    change

    if (isset($_POST['avhec_categorygroup']) && $tax_meta[$taxonomy][$term_id]['category_group_term_id'] != $_POST['avhec_categorygroup']) {

    to

    if (isset($_POST['avhec_categorygroup']) && (!isset($tax_meta[$taxonomy][$term_id]['category_group_term_id']) || $tax_meta[$taxonomy][$term_id]['category_group_term_id'] != $_POST['avhec_categorygroup'])) {
  10. ianarmstrong
    Member
    Posted 9 months ago #

    Sure be nice if the author was as interested in fixing this as everyone else is...

  11. petervanderdoes
    Member
    Plugin Author

    Posted 9 months ago #

    [sarcasm]Aahhh the life of an open source developer. The donations just keep flowing in, I could almost quit my day job.[/sarcasm]

    Anywho, just updated to v3.8.0, all the above patches have been implemented. Thanks everybody.

    P.S. All my plugins are on github as well to make it more convenient to submit patches. (https://github.com/petervanderdoes)

  12. esmi
    Forum Moderator
    Posted 9 months ago #

    [sarcasm]Aahhh the life of an open source developer. The donations just keep flowing in, I could almost quit my day job.[/sarcasm]

    You & me both! ;-)

  13. merosler
    Member
    Posted 9 months ago #

    Hello,

    I just updated your plugin to 3.8.0 and now when I click the "Installed Plugins" link in my wordpress dashbaord, it brings me to the "AVH Extended Categories - Overview" page and I am unable to access my plugins. Can you suggest how I can fix this?

    thank you,
    Matt

  14. petervanderdoes
    Member
    Plugin Author

    Posted 9 months ago #

    I just released 3.8.2 that fix the problem, to update go to the page:
    wp-admin/update-core.php and update the plugin.

  15. flynsarmy
    Member
    Posted 9 months ago #

    There are still warnings (and an error) in this build. PLEASE turn WP_DEBUG on! Pull request sent
    https://github.com/petervanderdoes/AVH-Extended-Categories/pull/1

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.