Filter mb_aoi_extensions fails unless at least one item checked in "All-In-One"

This topic has 12 replies, 3 voices, and was last updated 5 years, 11 months ago by criv23.

Viewing 13 posts - 1 through 13 (of 13 total)

Author

Posts
December 28, 2018 at 1:27 AM #12811
Theodore Darling
Participant
First, if there are no modules checked in the "all-in-one" settings page, then the filter "mb_aio_extensions" never gets called. I'll add my code change below that makes this work for me.

Next, even if the options are changed in the filter, they don't show as checked for the page.

Finally, every time I un-check the dashboard, it just returns filled back in.

Here is the code that I changed in the file class-mb-aio-loader.php that shows the content in the menu, but it doesn't check the boxes. Maybe it can be updated in the next version. Original code commented out.
```
        public function load_premium_extensions() {
        $option = get_option( 'meta_box_aio' );

        // Change the commented below to this. Otherwise, unless there is an extension already selected,
        // the code below will never call the filter.
        // Allows developers to filter the list of premium extensions.
        $extensions = apply_filters( 'mb_aio_extensions', $option['extensions'] );
        
        if ( ! is_array( $extensions ) || empty( $extensions ) ) {
            return;
        }
        foreach ( $extensions as $extension ) {
            require_once dirname( __FILE__ ) . "/extensions/$extension/$extension.php";
        }
        
        /*
        $option = get_option( 'meta_box_aio' );

                if ( empty( $option['extensions'] ) ) {
            return;
        }

        // Allows developers to filter the list of premium extensions.
        $extensions = apply_filters( 'mb_aio_extensions', $option['extensions'] );
        
        if ( ! is_array( $extensions ) ) {
            return;
        }
        foreach ( $extensions as $extension ) {
            require_once dirname( __FILE__ ) . "/extensions/$extension/$extension.php";
        }*/
    }
```
Add New Note to this Reply
January 2, 2019 at 9:25 AM #12841

Anh Tran
Keymaster

Hi Theodore,

Thanks a lot for your feedback. I've fixed 2 first issues. Will release new version soon.

Regarding the last issue, if you run the filter to enable an extension, then it's always enable, even if you uncheck from the list. The filter has higher priority than the option.

Add New Note to this Reply

January 2, 2019 at 10:51 PM #12853

Theodore Darling
Participant

Hi Anh,

I understand if adding a filter to enable an extension. And I know that the extensions are enabled whether or not they are checked.

My third point was in relation to the "Dashboard" checkbox above the extensions. Un-checking that doesn't remove anything from the dashboard and when the page is refreshed, it remains checked.

Add New Note to this Reply

January 3, 2019 at 12:39 PM #12863

Anh Tran
Keymaster

Thanks a lot for your feedback! I've just updated the AIO extension with fixes for these bugs. Please update and let me know if there's anything wrong.

Add New Note to this Reply

January 6, 2019 at 12:48 AM #12901
criv23
Participant
I may be experiencing the same issue. Any extensions I select do not remain checked. I also have them set in a filter but they are still showing as unchecked:
```
add_filter( 'mb_aio_extensions', function( $extensions ) {
    $extensions = ['mb-admin-columns','mb-settings-page','mb-user-meta','meta-box-builder','meta-box-columns','meta-box-conditional-logic','meta-box-group','meta-box-show-hide','meta-box-tabs','meta-box-updater','mb-term-meta'];

    return $option;
} );
```
The same can be said for the Dashboard checkbox like Theodore mentioned.

I can tell the plugins are not activated because my tabs, groups, etc. are not displaying correctly. If I install one of the plugins separately (e.g. meta-box-tabs), my tabs show up correctly. This was working fine around 12/26.

Any help would be appreciated.

Add New Note to this Reply
January 7, 2019 at 5:08 PM #12925

Anh Tran
Keymaster

Hi criv23,

Are you using the latest version 1.6.18? Did the 1.6.17 work for you?

Add New Note to this Reply

January 8, 2019 at 2:45 AM #12936

criv23
Participant

Yes, I'm using 1.6.18. I believe it was working on 1.6.17 or 1.6.16.

Whatever version it was on during 12/26 is when it was working. Is there a way to download previous releases? If so, I can let you know which version was working.

Add New Note to this Reply

January 8, 2019 at 3:11 PM #12944

Anh Tran
Keymaster

I've just sent you version 1.6.17 via email. Please take a look.

Add New Note to this Reply

January 8, 2019 at 9:37 PM #12955

criv23
Participant

Thanks Anh! I just tested 1.6.17 and that does work for me. So something in 1.6.18 is causing it to not save the data.

Add New Note to this Reply

January 9, 2019 at 3:24 PM #12962

Anh Tran
Keymaster

Thanks for your feedback. It's strange that I couldn't replicate the bug in the latest version 1.6.18 :(.

Add New Note to this Reply

January 11, 2019 at 10:22 AM #12981
criv23
Participant
I'm not sure what's causing it either. I deactivated every other plugin when I had 1.6.18 installed and it still wasn't working so it couldn't have been a compatibility issue.

Here are the only two metabox filters I have in my functions.php :
```
add_filter( 'mb_aio_load_free_extensions', '__return_false' );

add_filter( 'mb_aio_extensions', function( $extensions ) {
    $extensions = ['mb-admin-columns','mb-settings-page','mb-user-meta','meta-box-builder','meta-box-columns','meta-box-conditional-logic','meta-box-group','meta-box-show-hide','meta-box-tabs','meta-box-updater','mb-term-meta'];

    return $option;
} );
```
Let me know if you need any more info and I can help!

Add New Note to this Reply
January 11, 2019 at 10:40 AM #12983

Anh Tran
Keymaster

Hi criv23,

In the callback filter mb_aio_extension, can you change the code to return $extensions instead of $option and see if it works?

Add New Note to this Reply

January 11, 2019 at 11:28 AM #12987

criv23
Participant

Hi Anh,

Switching option to extensions also works in 1.6.17.

Add New Note to this Reply
Author

Posts

Viewing 13 posts - 1 through 13 (of 13 total)

You must be logged in to reply to this topic.