Masonry Gallery – Structured Data Bug

Hi Elliott,

Thank you for looking into this! I added the code below at the end of the functions.php that removes the “hentry” class if it’s a page and it fixed the first error “missing updated & author”.

 function ga_remove_hentry( $classes ) {
    if ( is_page() ) {
        $classes = array_diff( $classes, array( 'hentry' ) );
    }
    return $classes;
}
add_filter( 'post_class','ga_remove_hentry' );

I still need a fix for the second issue reporting that “contenturl” is not part of the “schema.org/webpage” markup. I think that the proper way to fix this is by having a conditional code in functions.php file to replace the “contenturl” markup that comes from “schema.or/imageobject” with “significantLink” or even just a “URL” markup ONLY for images that link to a page, but I don’t know how to code this.

Thank you,
George

Hi Elliott,

Changing line 108 in the helper-markup.php file is the easy shortcut that fixes this particular error on the Masonry Gallery element when page links are used instead of the lightbox. BUT it creates errors on all other elements (sliders, image gallery, etc) that use the “imageobject” markup when the attached images DON’T include a link to a page.

The problem with the Masonry Gallery is that it does not take into consideration images having links to a page that display the page url link instead of the image url and by default attaches the property “contentUrl” to the page url which should be attached only to an image url. The code needs to be changed so “contenturl” property is correctly attached to an image url BUT “significantLink” or “url” should be attached to images that include page links. By default, Images not attached to a page display the image url in which case the “contenturl” property is correctlu applied, while images that are attached to a page display the title or alt url in which case the “contenturl” property produces errors.

That is why I suggested earlier that a conditional code is needed to be added so it will apply the correct property on images without a link to a page and to images with a link to a page.

Thank you,
George

Hi Elliott!

Brilliant! As you suggested I replaced ‘image_url’ with ‘image_link’ on line 576 in the masonry_entries.php file and then created a new case for ‘image_link’ on the helper-markup.php file that returns ‘itemprop’ = ‘significantLink’ and now both properties for images without a link to a page and for images with a link to a page return the correct markup without producing errors.

Shouldn’t this fix be added as a permanent fix to the upcoming theme’s update since it also solves the errors on the demo pages?

Thank you very much for your help… You are brilliant!

Best regards,
George