Problem/Motivation

Currently all fields that the file entity has are added to the wysiwyg popup. As evidenced in this issue #2062365: Several types of fields are not being handled correctly when file is embedded in the WYSIWYG tag fields don't work because they should be an array and are stored as a string.

Proposed resolution

We discussed rolling back the ability to override the fields in the wysiwyg functionality, but decided that a better solution would be to simply add a whitelist of field types that we know will work in overrides.

Remaining tasks

Postponed on #2058303: Displaying embedded media without using the File Entity module's built-in formatter ignores attribute overrides

User interface changes

API changes

Original report by @mirzu

CommentFileSizeAuthor
#109 Screenshot 2017-02-28 22.51.43.png67.47 KBandyg5000
#92 media-wysiwyg_override_whitelist-2062721-91.patch11.57 KBAnas_maw
#90 media-wysiwyg_override_whitelist-2062721-90.patch11.52 KBAnas_maw
#89 media-wysiwyg_override_whitelist-2062721-89.patch11.5 KBrooby
#89 interdiff-2062721-83-89.txt645 bytesrooby
#88 interdiff-2062721-83-88.txt645 bytesrooby
#86 media-wysiwyg-override-white-list-2062721-86.patch11.54 KBanbarasan.r
#83 media-wysiwyg-override-white-list-2062721-83.patch11.5 KBmkhamash
#77 media-wysiwyg-override-white-list-2062721-77.patch11.91 KBhawkeye.twolf
#77 interdiff-72-77.txt2.09 KBhawkeye.twolf
#76 media-wysiwyg-override-white-list-2062721-76.patch2.09 KBhawkeye.twolf
#76 interdiff-72-76.txt2.09 KBhawkeye.twolf
#71 media-wysiwyg-override-white-list-2062721-71.patch11.93 KBazinck
#68 interdiff-64-68.txt5.52 KBheddn
#68 media-wysiwyg-override-white-list-2062721-68.patch12.5 KBheddn
#65 interdiff-64_65.patch5.52 KBheddn
#65 media-wysiwyg-override-white-list-2062721-65.patch12.49 KBheddn
#64 media-wysiwyg-override-white-list-2062721-64.patch11.71 KBsvenryen
#61 interdiff_56-60.txt1016 bytesheddn
#61 media-wysiwyg-override-white-list-2062721-60.patch12.44 KBheddn
#59 media-wysiwyg-override-white-list-2062721-59.patch12.44 KBheddn
#50 interdiff_49-50.txt1.61 KBheddn
#50 media-wysiwyg-override-white-list-2062721-50.patch12.44 KBheddn
#49 interdiff_47-49.txt2.78 KBheddn
#49 media-wysiwyg-override-white-list-2062721-49.patch10.65 KBheddn
#47 interdiff_44-47.txt852 bytesheddn
#47 media-wysiwyg-override-white-list-2062721-47.patch12.35 KBheddn
#46 media-wysiwyg-override-white-list-2062721-46.patch16.52 KBheddn
#46 interdiff_44-46.txt5.5 KBheddn
#44 media-wysiwyg-override-white-list-2062721-44.patch11.73 KBheddn
#44 interdiff_41-44.txt974 bytesheddn
#41 media-wysiwyg-override-white-list-2062721-41.patch11.67 KBheddn
#41 interdiff_39-41.txt3.93 KBheddn
#38 media-wysiwyg-override-white-list-2062721-39.patch7.93 KBheddn
#38 interdiff.txt9.42 KBheddn
#31 media-wysiwyg-override-white-list-2062721-31.patch7.02 KBseworthi
#20 media-wysiwyg-override-white-list-2062721-20.patch5.31 KBbneil
#20 media-2062721-interdiff-18-20.txt1.14 KBbneil
#18 media-2062721-wysiwyg-override-white-list-18.patch5.26 KBbneil
#18 media-2062721-interdiff-15-18.txt732 bytesbneil
#16 media-2062721-wysiwyg-override-white-list-16.patch3.88 KBbneil
#16 media-2062721-interdiff-15-16.txt887 bytesbneil
#15 media-2062721-wysiwyg-override-white-list-15.patch4.99 KBbneil
#12 media-2062721-wysiwyg-override-white-list-12.patch4.99 KBmirzu
#11 media-2062721-wysiwyg-override-white-list-11.patch4.14 KBmirzu
#10 media-2062721-wysiwyg-override-white-list-10.patch3.29 KBmirzu
#9 media-2062721-wysiwyg-override-white-list-9.patch3.28 KBmirzu
#8 media-2062721-wysiwyg-override-white-list-8.patch3.56 KBmirzu
#7 media-2062721-wysiwyg-override-white-list-7.patch283 bytesmirzu
#4 media-2062721-wysiwyg-override-white-list-3.patch2.14 KBmirzu
#2 media-2062721-wysiwyg-override-white-list.patch2.08 KBmirzu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mirzu’s picture

Here's a first pass at a patch.

mirzu’s picture

mirzu’s picture

Status: Active » Needs review
mirzu’s picture

Added a variable get and named things better.

genjohnson’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #4 looks good and works for me.

jvandyk’s picture

Patch refers to both "overridden" and "overwritten". These are different things. The description to "Override in WYSIWYG" is unclear. What does checking this checkbox actually do? (I think it stores attribute-value pairs into the text that the button inserts into the field, to be interpreted by the filter.

mirzu’s picture

Added update hook, because those are always fun to write.

mirzu’s picture

Rerolled patch with brain on.

mirzu’s picture

fixed some spelling and stuff.

mirzu’s picture

I shouldn't be allowed to spell things.

mirzu’s picture

Changed the way that the filter chooses fields to include.

mirzu’s picture

Added an update to the install that will make the fields show up as expected when media is installed.

mirzu’s picture

Status: Reviewed & tested by the community » Needs review

I think this handles install and update in a sane way now. Definitely needs testing. It would be extra nice if anyone who has an existing site with lots of media images would test out the update.

dmducc’s picture

OK...this maybe be me patching against the wrong version of the module (I used: drush dl media-7.x-2.x-dev)

But it seems that media_update_7213() was already used for the following (line 1024):

/**
 * Remove the obsolete file_extensions variable.
 */
function media_update_7213() {

I changed to media_update_7219() and I was able to use this patch to great success.

I received the error message when trying to add images to a Taxonomy Term description, and then render that description through a view in a custom page.
I did reload a new test image, but might have been able to resave the tax term instead.

dmduke

bneil’s picture

Here's a patch that changes the update to media_update_7219().

In my testing, I noticed that on install, text and text_long fields (tested with alt/title on image type) are not overridable by default without saving the field's form first.

bneil’s picture

Renamed variable wysiwyg_override_field_types to media__wysiwyg_override_field_types to follow other media variable naming scheme.

Added a variable_del for the new variable in media_uninstall().

Status: Needs review » Needs work

The last submitted patch, media-2062721-wysiwyg-override-white-list-16.patch, failed testing.

bneil’s picture

Status: Needs work » Needs review
FileSize
732 bytes
5.26 KB

Left out the update hook on the earlier patch.

aaron’s picture

Status: Needs review » Needs work
+      return "Updated the following field instances: $updated_string so they can't be overridden when the file is inserted in the WYSIWYG.\n
+        Updated the following fields $set_to_default so that they continue to show up when a file is inserted.";

This should be a translated string.

bneil’s picture

Status: Needs work » Needs review
FileSize
1.14 KB
5.31 KB
  • Condensed update short description
  • Translated string
  • Created a string of the array of values from $set_to_default
bneil’s picture

Issue tags: +7.x-2.0 beta blocker

Adding beta blocker tag.

lmorgan923’s picture

I'm really new to this and have only the beginning of a glimmer of an idea of what I'm doing, but I just ran the patch in #20, and after running the update script, got this message:

The following updates returned messages
media module
Update #7219 - Updated the following field instances: field_tags so they can't be overridden when the file is inserted in the WYSIWYG. Updated the following fields field_file_image_alt_text, field_file_image_title_text so that they continue to show up when a file is inserted.

So it looks to me like the patch ran (plus the messages in my Terminal window were all positive). However, I'm still getting the error message that sent me out looking for a fix, which is: Warning: Invalid argument supplied for foreach() in taxonomy_field_formatter_prepare_view() (line 1666 of /home/mysitemodules/taxonomy/taxonomy.module) plus the same warning for line 1589.

I understand that this issue isn't closed, but - and here's my newbie question - should I have run any of the earlier patches as well? I'm assuming no, that each patch incorporates the earlier patches, but I'd hate to be wrong and miss something that might help.

lmorgan923’s picture

Issue summary: View changes

changed link to issue number

sheldonkreger’s picture

Patch in #20 is working for me.

gmclelland’s picture

Status: Needs review » Needs work

The last submitted patch, 20: media-wysiwyg-override-white-list-2062721-20.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 20: media-wysiwyg-override-white-list-2062721-20.patch, failed testing.

gmclelland’s picture

SocialNicheGuru’s picture

the patch no longer applies. there is no includes/media.filter.inc file in the newest media 7.2 dev module

blacklabel_tom’s picture

Component: WYSIWYG integration » Code
Issue summary: View changes

Is this issue still 7.x-2.0 beta blocker?

Cheers

Tom

pbuyle’s picture

WYSIWYG integration has been moved to a separated Media WYSIWYG module (#2142571: Spin-off WYSIWYG support in dedicated module / project). The patch needs to be rerolled for it.

Note: as of today, the WYSIWYG related part of media_form_field_ui_field_edit_form_alter() has not been moved to the media_wysiwyg module.

seworthi’s picture

Rerolled patch against current dev.

Added config option to "Media browser settings" to select field type from available types in system.

Changed default setting on field config form from TRUE to FALSE.

seworthi’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 31: media-wysiwyg-override-white-list-2062721-31.patch, failed testing.

ann b’s picture

I don't have a tags field (field_tags) defined for any of my file types, but the Media Browser Plus module does add a term reference field called field_folder. I'm getting the following errors after embedding files using the Add Media button in the WYSIWYG:

Warning: Invalid argument supplied for foreach() in taxonomy_field_formatter_prepare_view() (line 1666 of .../modules/taxonomy/taxonomy.module).

Warning: Invalid argument supplied for foreach() in taxonomy_field_formatter_view() (line 1589 of .../modules/taxonomy/taxonomy.module).

Do I need to override these field formatter hooks to add field_folder to the white list?

Test Server Versions:
Drupal Core 7.28
Media Module 7.x-2.0-alpha3
Media Browser Plus for Media 2.x 7.x-3.0-beta3

Thank you,
Ann

bneil’s picture

Ann,

You can eliminate that warning by editing the field on each file type that has the field and unchecking "Override in WYSIWYG". Term reference fields are not supported as overridable in the WYSIWYG. This issue is about providing a whitelist of field types which are allowed by default instead of all of them.

ann b’s picture

Thanks @bneil. The warning messages no longer appear after unchecking the Override in WYSIWYG checkbox, just as you said. Sorry for the noise.

heddn’s picture

heddn’s picture

Status: Needs work » Needs review
FileSize
9.42 KB
7.93 KB

The previous patch in #2062721-31: Add a white list of file fields that can be overwritten when the file is added in the wysiwyg was originally written prior to media_wysiwyg getting split into a sub-module. The patch makes the needed step to move all the functionality into the sub-module. It also adds a couple things that were missing.

  • variable_delete in .install was wrong.
  • Form media_wysiwyg_format_form checks if the attached field has a cardinality of 1, since multi-valued fields don't work.

Status: Needs review » Needs work

The last submitted patch, 38: media-wysiwyg-override-white-list-2062721-39.patch, failed testing.

heddn’s picture

Status: Needs work » Needs review
heddn’s picture

While this doesn't explicitly depend on #2325671: file_image formatter ignores alt & title overrides on files, none of the overrides will display unless that gets committed. Let's see how the testbot likes things, but I think this will fail until the file_entity patch gets committed. I think that's why we are experiencing the failed tests.

Status: Needs review » Needs work

The last submitted patch, 41: media-wysiwyg-override-white-list-2062721-41.patch, failed testing.

heddn’s picture

Status: Needs work » Postponed
heddn’s picture

Fixes an undefined index issue when viewing media from outside of WYSIWYG.

azinck’s picture

The approach in #44 means that fields without data stored in them on the entity will not get overridden. Without the patch that functionality works fine.

+++ b/media.module
@@ -1009,15 +1001,6 @@ function media_file_displays_alter(&$displays, $file, $view_mode) {
-  // Override the fields of the file when requested by the WYSIWYG.
-  if (isset($file->override) && isset($file->override['fields'])) {
-    $instance = field_info_instances('file', $file->type);
-    foreach ($file->override['fields'] as $field_name => $value) {
-      if (!isset($instance[$field_name]['settings']) || !isset($instance[$field_name]['settings']['wysiwyg_override']) || $instance[$field_name]['settings']['wysiwyg_override']) {
-        $file->{$field_name} = $value;}
-    }
-  }
-

This is needed in order for the overrides to work properly for fields without data in the DB.

heddn’s picture

While re-rolling the patch and splitting out the functionality into the appropriate place, I seem to have lost that functionality. Updated patch attached.

heddn’s picture

Ignore #46, the patch and interdiff are all wrong. Try this one instead.

azinck’s picture

@heddn: I don't think you need media_wysiwyg_field_attach_view_alter() anymore now that you've restored the override functionality via media_wysiwyg_field_attach_view_alter(). Is there a reason to keep that around?

heddn’s picture

heddn’s picture

FileSize
12.44 KB
1.61 KB

Fields, alt and title text are all getting pulled in from the overridden macro. What is missing is the height/width dimensions. This updated patch picks up that missing piece.

klokie’s picture

Status: Postponed » Needs review

patch in #50 works for me.

The last submitted patch, 47: media-wysiwyg-override-white-list-2062721-47.patch, failed testing.

The last submitted patch, 49: media-wysiwyg-override-white-list-2062721-49.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 50: media-wysiwyg-override-white-list-2062721-50.patch, failed testing.

heddn’s picture

bneil’s picture

Status: Postponed » Needs review

Status: Needs review » Needs work

The last submitted patch, 50: media-wysiwyg-override-white-list-2062721-50.patch, failed testing.

heddn’s picture

Status: Needs work » Needs review
FileSize
12.44 KB

Reroll

Status: Needs review » Needs work

The last submitted patch, 59: media-wysiwyg-override-white-list-2062721-59.patch, failed testing.

heddn’s picture

Testbot, come and get it.

ben.bunk’s picture

I like the direction of this patch, for us we're looking for a way to override the title field on a document file type and this patch doesn't seem to address that concern. Is there any reason why this only works for images? Would the group be open to making these changes work for other file types?

heddn’s picture

re: #62
There is no limitation of image or other media type.

svenryen’s picture

FileSize
11.71 KB

The patch had to be re-rolled. Here's the updated patch.

heddn’s picture

Not a clean interdiff, but the main piece is that I added logic to check zero indexed macro code and non-indexed macro code in media_wysiwyg_form_wysiwyg_profile_form_alter()

Status: Needs review » Needs work

The last submitted patch, 65: interdiff-64_65.patch, failed testing.

The last submitted patch, 65: media-wysiwyg-override-white-list-2062721-65.patch, failed testing.

heddn’s picture

heddn’s picture

Status: Needs work » Needs review
Danny Englander’s picture

I tested this patch as I ran into the same issue. However, I got an error:

PHP Fatal error: Cannot redeclare media_wysiwyg_update_7201() (previously declared...

... and that's because the latest dev actually includes another db update with the same name, 7201. I attempted to change this patch to 7202 but I don't think it ended up working in the end. I no longer see "Override in WYSIWYG" in my file entity field settings. The field appears briefly when I upload an image via WYSIWYG but then it goes away. I attempted this again on a brand new install of drupal 7 with minimal modules enabled but I probably did not fix the patch correctly to account for the hook_update(n) name collision.

azinck’s picture

Re-roll against latest dev to fix the probs Danny Englander pointed out. Full disclosure: I haven't tested this, just updated the update hook name and the install hook to refer to the correct update hook.

vegantriathlete’s picture

I've installed the -dev version and applied the patch.

The error does go away. However, that's only part of the solution as we do need to be able to specify term references for images embedded in the wysiwyg. Also, whether I check or uncheck Term reference is the media browser configuration the term references still are not available in the media browser in the wysiwyg. But, I think that is what's currently intended. (There is a comment right in the patch that says "// Only single valued fields can be overridden.")

I'm going to go back to the issue that was closed as a duplicate of this one, as I think it needs to be reopened. This issue / patch does not solve that problem.

vegantriathlete’s picture

So, I'm happy to mark this RTBC if others are happy with just my confirmation that the patch applies and works as designed.

However, I have reopened #2062365: Several types of fields are not being handled correctly when file is embedded in the WYSIWYG since I think that the ability to override term references should still be addressed. And, in fact, there is a proposed solution in the comments.

Devin Carlson’s picture

Component: Code » Media WYSIWYG
stred’s picture

#71 git rid of the warning when an embedded video is displayed. I did not test the rest... does the job for me, thanks!

hawkeye.twolf’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
2.09 KB
2.09 KB

Uploaded the wrong patch file, ignore this one.

hawkeye.twolf’s picture

Uploaded wrong file, updating patch with minor coding standards fix.

The last submitted patch, 76: media-wysiwyg-override-white-list-2062721-76.patch, failed testing.

Danny Englander’s picture

I tried the new patches, ran drush updb and though I no longer get the error #70, I don't see "If checked, then this field may be overridden in the WYSIWYG editor." when I add a select list to a file. Then it never shows up for any view mode. Gosh, maybe I am totally misunderstanding how this is supposed to work. My work around for now is to simply use a text field with the value I need.

Dave Reid’s picture

Status: Reviewed & tested by the community » Needs review
Issue tags: -7.x-2.0 beta blocker +7.x-2.0 release blocker

I would like to get some extra testing on the latest version (patch in #77) before commit. Moving to release blocker for 2.0 release.

Status: Needs review » Needs work

The last submitted patch, 77: media-wysiwyg-override-white-list-2062721-77.patch, failed testing.

mkhamash’s picture

FileSize
11.5 KB

Re-roll against the latest dev 7.x-2.0-beta1+8-dev (2015-Aug-19) [Sha 6b94e2f6c], the overrides to fields are not being saved or applied, but the admin interface for fields override does work as expected.

Disclaimer: I have not properly tested this patch and it may not working due to regression issues, I have tested it with bleeding-edge media + media_ckeditor + ckeditor, with tens of patches so.

mkhamash’s picture

This patch will work greatly with the patch in issue #2352573: CKEditor Media WYSIWYG library.js does not pass entity fields into create_element in case of using ckeditor module + ckeditor >= 4.3, though this need more testing but it looks very promising.

anbarasan.r’s picture

Status: Needs work » Needs review
anbarasan.r’s picture

Attached the PATCH against 7.x-2.0-beta1. It might fail because the PATCH not against the latest dev version. But may be useful who want to apply on their site against 7.x-2.0-beta1.

Status: Needs review » Needs work

The last submitted patch, 86: media-wysiwyg-override-white-list-2062721-86.patch, failed testing.

rooby’s picture

Status: Needs work » Needs review
FileSize
645 bytes

@anbarasan.r

You can add do-not-test to your patch file name if you want the testbot to skip your patch.

Like this: media-wysiwyg-override-white-list-2062721-86-do-not-test.patch

rooby’s picture

Whoops I didn't mean to post that interdiff in the previous comment.

The patch in #83 didn't work for me.

I haven't done a full review of the patch but I did find an issue, which this patch fixes.

Anas_maw’s picture

Status: Needs review » Needs work

The last submitted patch, 90: media-wysiwyg_override_whitelist-2062721-90.patch, failed testing.

Anas_maw’s picture

Retest patch after fix.

Anas_maw’s picture

joseph.olstad’s picture

This is a 2.0 release blocker. @Anas_maw, have you used this patch on todays dev or beta5?

  • joseph.olstad committed 7296e28 on 7.x-2.x authored by heddn
    Issue #2062721 by heddn, mirzu, bneil, derek.deraps, Anas_maw, rooby,...
joseph.olstad’s picture

Status: Needs review » Fixed

this is now in 7.x-2.x dev

beta6 does NOT have this, but 7.x-2.x dev does...

a 7.x-2.0 release will be tagged probably in a few weeks from now.

Thanks to @heddn, @mirzu, @bneil, @derek.deraps, @Anas_maw, @rooby, @azinck, @mkhamash, @seworthi, @anbarasan.r, @svenryen

ar-jan’s picture

Nice, thanks all.

a 7.x-2.0 release will be tagged probably in a few weeks from now.

That's excellent news!

joseph.olstad’s picture

Here's a list of the media project 7.x-2.0 release blockers

a few left before 7.x-2.0 , maybe a few weeks might be optimistic, haven't yet had a close enough look at the remaining items.

sylus’s picture

This patch is causing issues for me with Entity Translation all of a sudden my Behat tests checking for the alt attribute on the French Side of the page are no longer working. I have traced it to this commit, as one commit prior (2.0-beta6) is working as expected.

    Given I am on "en/content/drupal-wxt"                                                                # Drupal\DrupalExtension\Context\MinkContext::visit()
    Then I should see the image alt "Web Experience Toolkit Logo" in the "Content Well" region           # TestSubContext::assertAltRegion()
    And I am on "fr/contenu/wxt-drupal"                                                                  # Drupal\DrupalExtension\Context\MinkContext::visit()
    Then I should see the image alt "Boîte à outils de expérience Web Logo" in the "Content Well" region # TestSubContext::assertAltRegion()
      No alt text matching "Boîte à outils de expérience Web Logo" in the "Content Well" region on the page http://127.0.0.1:8888/fr/contenu/wxt-drupal (Exception)

I will create a new issue shortly.

Status: Fixed » Closed (fixed)

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

joseph.olstad’s picture

@sylus, I must have came accross this problem with alt ant title text in french as well (however only when the image style is used and not 'rendered_file', I created a working core patch but maybe its not necessary. We'd have to confirm the details first though before jumping to conclusions.

Here's the core issue I created and provided a working patch, however if this is caused by contrib (this patch in particular) it would be ideal to fix this in 'media' rather than make a change to 'core'.

Was this resolved for you?

here's the related issue:
#2835135: image formatter needs to handle alt/title from file entities on images for multi language support

joseph.olstad’s picture

joseph.olstad’s picture

Status: Closed (fixed) » Needs review

this issue appears to be responsible for the regression described by @sylus.

If my suspicions are correct:
Here's the change in the function that I suspect needs review / refactoring. While this works for a unilingual site, multilingual 'alt' and 'title' are getting the default language values under certain conditions as described in #2835135: image formatter needs to handle alt/title from file entities on images for multi language support .

@@ -308,6 +341,34 @@ function _media_wysiwyg_wysiwyg_allowed_attributes_default() {
  * Optional custom settings can override how the file is displayed.
  */
 function media_wysiwyg_get_file_without_label($file, $view_mode, $settings = array(), $langcode = NULL) {
+  // Get the override alt & title from the fields override array. Grab the
+  // "special" field names from the token replacement in file_entity, then see
+  // if an override is provided and needed.
+  $pattern = '/\[(\w+):(\w+)\]/';
+  $alt = variable_get('file_entity_alt', '[file:field_file_image_alt_text]');
+  $title = variable_get('file_entity_title', '[file:field_file_image_title_text]');
+  $overrides = array(
+    'alt' => $alt,
+    'title' => $title,
+  );
+  foreach ($overrides as $field_type => $field_name) {
+    preg_match($pattern, $field_name, $matches);
+    if (!empty($matches[2])) {
+      $field_name = $matches[2];
+      $langcode = field_language('file', $file, $field_name);
+      if (isset($settings['fields'][$field_name][$langcode]['value'])) {
+        if (empty($settings['attributes'][$field_type])) {
+          $settings['attributes'][$field_type] = $settings['fields'][$field_name][$langcode]['value'];
+        }
+      }
+      if (isset($settings['fields'][$field_name][$langcode][0]['value'])) {
+        if (empty($settings['attributes'][$field_type])) {
+          $settings['attributes'][$field_type] = $settings['fields'][$field_name][$langcode][0]['value'];
+        }
+      }
+    }
+  }
+
joseph.olstad’s picture

Possibly related sections of code.
#1553094: Alt and Title support for Images

joseph.olstad’s picture

@sylus , on my local system I tested backing off all lines of code related to this issue, I also backed off the core patch that fixes the 'alt' and 'title' regression you mentioned, when my fix was backed off, and all changes related to this issue were backed off, it did not fix the multilingual alt / title text issue.

So currently the only way I'm getting 'alt' and 'title' text is via the core patch #2835135: image formatter needs to handle alt/title from file entities on images for multi language support

@sylus, can you please double check again which git hash was causing the issue you noticed? Open a new issue for it in the 'media' queue, and I'll have a look.

joseph.olstad’s picture

@sylus , it could be that we're not talking about the same french 'alt' issue.
mine is documented in #2835135

joseph.olstad’s picture

Status: Needs review » Closed (works as designed)
sylus’s picture

Hey no problem ^_^

I take a look again on latest media release and see if i can reproduce. I'll also create a new issue with exact repro steps if problem occurs.

Thanks a bunch! :)

andyg5000’s picture