Hi,

In Wetkit 4.9 the following warning for WYSIWYG embedded images occurs.

Warning: Missing argument {2,3,4,5,6} for media_wysiwyg_filter(), called in /profiles/wetkit/modules/contrib/metatag/metatag.inc on line 168 and defined in media_wysiwyg_filter() (line 15 of /profiles/wetkit/modules/contrib/media/modules/media_wysiwyg/includes/media_wysiwyg.filter.inc).

The error goes away if the base Media Module is used... We are unsure what else is affected at this time.

Discovery:

The patch media_alt_attributes-2129273-24.patch modifies media_wysiwyg_filter() code.

+function media_wysiwyg_filter($text, $filter, $format, $langcode, $cache, $cache_id)

CommentFileSizeAuthor
#2 metatag_module_missing-2761477-2.patch1.06 KBsylus
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fgjohnson created an issue. See original summary.

sylus’s picture

sylus CreditAttribution: sylus commented
FileSize
metatag_module_missing-2761477-2.patch1.06 KB

This is due to this issue which while RBTC isn't part of media module yet.

Metatag is still calling the old way of leveraging this function. I did a grep to make sure no other module is.

Here is a patch that resolves this issue and will be part of next release.

sylus’s picture

sylus CreditAttribution: sylus commented
Status: Active » Fixed
sylus’s picture

sylus CreditAttribution: sylus commented
Related issues: +#2129273: Pass langcode to media_wysiwyg_token_to_markup resolving media alt attributes in interface
fgjohnson@lojoh.ca’s picture

fgjohnson@lojoh.ca CreditAttribution: fgjohnson@lojoh.ca commented
Issue summary: View changes

  • sylus committed 4b7ca76 on 7.x-4.x 
    Fixed WetKit Metatag for Issue #2761477 by sylus: Media Module - Missing...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.