Support » Plugin: JQuery Accordion Menu Widget » Update to fix warning msg on newer WP core

  • dcwp_jquery_accordion_widget.php needs the following changes:

    parent::WP_Widget( 'nav_menu', __('Custom Menu'), $widget_ops );
    
    		$this->WP_Widget($id_base, __($name, 'dcjqaccordion'), $widget_ops);

    changed to:

    parent::__construct( 'nav_menu', __('Custom Menu'), $widget_ops );
    
    		$this->__construct($id_base, __($name, 'dcjqaccordion'), $widget_ops);

    Could you make this change or add me as a developer to this project so I can?

    https://wordpress.org/plugins/jquery-vertical-accordion-menu/

Viewing 10 replies - 1 through 10 (of 10 total)
  • jasdeepm13

    (@jasdeepm13)

    Well Done @jdian !!
    This works very well. I was struggling to find the fix.

    Thanks again.. 🙂

    I got it working more or less, without notices. Did not find any good alternative, so I tried to fix that old plugin somehow. Changes are in dcwp_jquery_accordion_widget.php

    Step one: rename function dc_jqaccordion_widget() to function __construct()

    Step two: comment out //parent::WP_Widget( ‘nav_menu’, __(‘Custom Menu’), $widget_ops );

    Step three: Change $this->WP_Widget($id_base, __($name, ‘dcjqaccordion’), $widget_ops); to parent::__construct($id_base, __($name, ‘dcjqaccordion’), $widget_ops);

    Works for me so far.

    I had to make additional changes to the ones listed above – any chance we can get these committed to the plugin?

    I didn’t note them all because they were numerous, but if we can get access to this plugin we should be able to put all of these in place.

    Hi Tony (@tonydjukic),
    it seems that the author of this plugin has stopped all activities.
    As this plugin is great and does exactly what I need, I want to ask you if there is any chance that you can upload your changes to github, somewhere else or that you can send me your latest version?

    I am sure that there are a lot of people, which are still interrested in this plugin.

    Thank you in advance.
    Peter

    @pwi1000, unfortunately once I put it into production, the entire thing just imploded – no idea why. I intend on revisiting to seeing if I can find where it all went wrong, but I haven’t had the time lately. To avoid the errors I made a simplified shortcode within the theme I was building that does exactly what the specific site needed. Sorry. If I can swing the time to re-work the plugin, I will and I’ll post here.

    I will appreciate if you can publish the solution. Using php7 and WP4.8. Many thanks!

    Solved!

    Just 2 changes on /wp-content/plugins/jquery-vertical-accordion-menu/dcwp_jquery_accordion_widget.php:

    1) Replace this line:

    function spacious_service_widget_pp() {

    With this new one:

    public function __construct(){

    2) Add this new function after closing the first one:

    public function spacious_service_widget_pp()
    {
    self::__construct();
    }

    Worked for me! (php7 / WP4.8)

    @gus0074

    You know this line doesn’t actually existing in this plugin, right?

    function spacious_service_widget_pp()

    Here you go everyone:

    jQuery Accordion Menu Plugin – New version GitHub

    Anyone interested in forking this can do so and I’ll add all changes to the main file. Maybe we can find a way to get this fixed once and for all and commit the entire thing to the repository.

    The last error is the one I’ve fixed multiple times but my fixes always render the plugin useless – not sure what’s happening but what good is an errorless plugin if it doesn’t work, right?

    You are right @tonydjukic, maybe my code was modified from the original. Anyway I solved the problem renaming the constructor funcion and adding a function afterwards. Cheers to you!

Viewing 10 replies - 1 through 10 (of 10 total)
  • The topic ‘Update to fix warning msg on newer WP core’ is closed to new replies.