[Resolved] Error with layouts

[Author]

Posts

[Justin]

I noticed a bug in the layouts class.

wpddl.layouts-templates-settings.class.php
LINE 80: list( $type, $my_class ) = explode( '|', $context );

But in my particular case, there is an issue
Notice: Undefined offset: 1 in /wp-content/plugins/layouts/classes-auto/wpddl.layouts-templates-settings.class.php on line 80

Because you need to place your
if ( ! $context ) { return null; }

above your LIST, in my opinion.

What you have:
// Get type {post types, taxonomies, wordpress archives slugs, 404} and class {page, archive}
list( $type, $my_class ) = explode( '|', $context );

$menu_title = $toolsetAdminBar->get_title( 'layouts', true, $type, $my_class, null );
$post_id = null;

if ( ! $context ) {
return null;
}

it SHOULD be:
if ( ! $context ) {
return null;
}

// Get type {post types, taxonomies, wordpress archives slugs, 404} and class {page, archive}
list( $type, $my_class ) = explode( '|', $context );

$menu_title = $toolsetAdminBar->get_title( 'layouts', true, $type, $my_class, null );
$post_id = null;

[Nigel]

Hi Justin

Your suggestion makes sense to me, but I've passed it to my colleagues for comment and I'll get back to you with their response.

[Justin]

Great thanks!

[Nigel]

Hi Justin

The developers reviewed this and made changes based on your comments.

It will be included in the next version of Layouts, but in the meantime if you need to you can apply the fix by unzipping the archive available here hidden link and replacing the same file in the /classes-auto/ directory of the Layouts plugin.

Thanks again for your feedback.

[Beda]

There is as well an errata, with the patch now:
https://toolset.com/errata/notice-undefined-offset-1-in-wpddl-layouts-templates-settings-class-php/