Missing type="application/ld+json" on script tag when using Zen or Mothership or Adaptivetheme themes

I am facing the same problem. Turned out that the Zen theme, which we are using as the base theme, is stripping out the '#attributes' item :(

Ugh. Maybe submit a feature request with a patch that makes that optional using a variable, then some documentation could be added to Schema Metatag to apply the patch and hardcode the variable to disable the feature?

Working on a fix..

This appears to solve the problem in my limited testing. Note: I also uploaded a similar patch for Zen theme itself.

Patch works as expected. I have some reservations about the effect of the patch though. While it is solving the immediate problem by letting us have *application/ld+json* as the script type, we still can't use any other type. MDN suggests other values (e.g. *module*) are acceptable. So if anyone tries any value other than *text/ng-template* or *application/ld+json*, they will be stuck and will most likely add another ticket to this project's issue queue.

It appears to me that the goal of Zen's html_tag process function was to omit the *application/javascript* type as that is the default for script tags in HTML *5*. So if we just drop *application/javascript* and *keep* everything else, that could save some potential future trouble regarding the script tag.

This is a Zen problem, and Damien created an issue and patch on that queue to fix it. I think it's better to leave the fix there. And there people can discuss other reasons to include or exclude various types. For reference, the patch is at https://www.drupal.org/project/zen/issues/2934644.

I am going to let Zen fix this, it feels pretty heavy-handed to try to fix a theme problem here. I will also add a note to the project page pointing to that patch.

Rather than marking this fixed or won't fix, which will close this issue, I'll mark it postponed so the issue stays in the queue where it can be found by others.

That's complete valid, thanks progga and KarenS.

Turns out this issue is more widespread. Our theme is based on Mothership and does the same as Zen. I had to wrap/kill the preprocess function with Damien's if clause in both, Mothership's template.php and our subtheme's own template.php. Updating the D7 README.txt accordingly.

This also appears to be happening with the Aurora theme's template.php:

/**
  * Implements hook_process_html_tag
  *
  * - From http://sonspring.com/journal/html5-in-drupal-7#_pruning
  */
function aurora_process_html_tag(&$vars) {
  if (theme_get_setting('aurora_html_tags')) {
    $el = &$vars['element'];

    // Remove type="..." and CDATA prefix/suffix.
    unset($el['#attributes']['type'], $el['#value_prefix'], $el['#value_suffix']);

    // Remove media="all" but leave others unaffected.
    if (isset($el['#attributes']['media']) && $el['#attributes']['media'] === 'all') {
      unset($el['#attributes']['media']);
    }
  }
}

Yes, looks like many themers took the blog post in the comment of this code snippet to heart, and that code is overly aggressive.
The README for the released version, as well as current HEAD mentions this and points to two issues where folks can copy and paste code examples from to fix the issue. Not a great solution.

Nicole Sciacca's (nsciacca) metatag_jsonld sandbox project had this theme callback to explicitly set the right type and it has never failed me. I'm not enough of a themer to determine whether schema_metatag could also do an endrun around aggressive settings in template.php.
/**
* Theme callback for a json-ld tag.
*/
function theme_metatag_jsonld($variables) {
$result = json_decode($variables['element']['#value']);
if (json_last_error() === JSON_ERROR_NONE) {
$output = '

\n";
return $output;
}
return FALSE;
}

As an fyi, the Aurora theme has a "Prune HTML tags" checkbox in its theme settings that enables/disables this trimming for all tags.

The HTML5 Tools module also has a "Simplify javascript tags" checkbox that strips out all type attributes for Javascript tags when enabled.

Thanks @chicagomom, I updated the known issues section of https://www.drupal.org/docs/7/modules/schemaorg-metatag/basic-usage.

I have rebuilt the #5 with the latest version of the module.

Thanks @amir_razavi. I am closing this issue again, since we all agreed that Zen and other themes need to fix the problem, rather than schema_metatag module.

The same problem also exists for the beloved AdaptiveTheme. I guess that should also be added to the module description page for Drupal 7:

process.inc, line 47ff.:

/**
 * Process variables for the html tag
 */<em></em>
function adaptivetheme_process_html_tag(&$vars) {
  $tag = &$vars['element'];
  if ($tag['#tag'] === 'style' || $tag['#tag'] === 'script') {
    // Remove redundant type attribute and CDATA comments.
    unset($tag['#attributes']['type'], $tag['#value_prefix'], $tag['#value_suffix']);
    // Remove media="all" but leave others unaffected.
    if (isset($tag['#attributes']['media']) && $tag['#attributes']['media'] === 'all') {
      unset($tag['#attributes']['media']);
    }
  }
}