Problem/Motivation

When you create a file field, you can enable the description field for each file. This field description is supposed to override the file name on display.
It is working properly with the "Generic file" formatter but not with the "Table of files" formatter.

More than just fixing this bug (it was working in 7.x) we decided to add a setting on the concerned formatters to let the site builder choose if she wants the file name to be overriden by the description.

Proposed resolution

- pass the file description to the theme so it can be used instead of the file name
- add a setting to the "Generic file" formatter to use the description when available
- add a setting to the "Table of files" formatter to use the description when available
- add an upgrade path to disable the new settings for existing "Table of files" formatters (Backward Compatibility)
- increase test coverage to ensure these new settings are working

Remaining tasks

Contributor tasks needed
Task Novice task? Contributor instructions Complete?
Create a patch Instructions Done
Update the issue summary Instructions Done
Add automated tests Instructions Done
Manually test the patch Novice Instructions Done
Embed before and after screenshots in the issue summary Novice Instructions Done
Review patch to ensure that it fixes the issue, stays within scope, is properly documented, and follows coding standards Instructions Done

User interface changes

A new setting allows the site builder to define if she want the file name to be overriden by the description when it's available.
The file description is shown, when available and when the formatter allows it, instead of the file name when using the "Table of files" or "Generic files" formatter.

API changes

None.

Data model changes

A new setting on the file_table and file_default formatters.

CommentFileSizeAuthor
#106 file_description_formatter_setting-2677990-106.patch20.81 KBDuaelFr
#104 interdiff.2677990.99.104.txt4.32 KBDuaelFr
#104 file_description_formatter_setting-2677990-104.patch20.75 KBDuaelFr
#99 interdiff.2677990.94.99.txt772 bytesDuaelFr
#99 file_description_formatter_setting-2677990-99.patch20.78 KBDuaelFr
#97 interdiff.2677990.94.97.txt772 bytesDuaelFr
#97 file_description_formatter_setting-2677990-97.patch45.32 KBDuaelFr
#94 interdiff.2677990.89.94.txt1.33 KBDuaelFr
#94 file_description_formatter_setting-2677990-94.patch20.8 KBDuaelFr
#91 interdiff.2677990.79.89.txt480 bytesDuaelFr
#89 interdiff.2677990-78-89.txt10.53 KBagomezmoron
#89 file_description_formatter_setting-2677990-89.patch20.55 KBagomezmoron
#85 interdiff.2677990.78.81.txt480 bytesagomezmoron
#85 file_description_formatter_setting-2677990-81.patch10.49 KBagomezmoron
#84 uploaded_file_2677990_postpatch_table_of_files.png8.08 KBagomezmoron
#84 upload_file_2677990_upload_postpatch.png10.97 KBagomezmoron
#84 uploaded_file_2677990_prepatch_table_of_files.png7.37 KBagomezmoron
#84 upload_file_2677990_upload_prepatch.png11.59 KBagomezmoron
#60 file_description_formatter_setting-2677990-60.patch20.37 KBDuaelFr
#60 interdiff.2677990.52.60.txt2.88 KBDuaelFr
#52 file_description_formatter_setting-2677990-52.patch20.33 KBDuaelFr
#52 file_description_formatter_setting-2677990-52-tests-only.patch2.34 KBDuaelFr
#52 interdiff.2677990.41.52.txt8.05 KBDuaelFr
#48 file_description_formatter_setting-2677990-41-tests-only.patch2.34 KBDuaelFr
#48 file_description_formatter_setting-2677990-41.patch20.08 KBDuaelFr
#41 file_description_formatter_setting-2677990-41-tests-only.patch2.34 KBDuaelFr
#41 file_description_formatter_setting-2677990-41.patch20.08 KBDuaelFr
#41 interdiff.2677990.34.41.txt1.06 KBDuaelFr
#34 file_description_formatter_setting-2677990-34-tests-only.patch2.34 KBDuaelFr
#34 file_description_formatter_setting-2677990-34.patch19.03 KBDuaelFr
#34 interdiff.2677990.30.34.txt11.73 KBDuaelFr
#30 interdiff.2677990.11.30.txt5.38 KBDuaelFr
#30 file_description_formatter_setting-2677990-30.patch7.87 KBDuaelFr
#30 file_description_formatter_setting-2677990-30-tests-only.patch2.34 KBDuaelFr
#11 interdiff.2677990.10.11.txt1.66 KBDuaelFr
#11 table_of_files_2677990-11.patch3.21 KBDuaelFr
#11 table_of_files_2677990-11-tests-only.patch2.34 KBDuaelFr
#10 table_of_files_2677990-10.patch2.15 KBgvso
#8 interdiff-2677990-2-8.txt1.21 KBheykarthikwithu
#8 2677990-8.patch2.2 KBheykarthikwithu
#2 table_of_files_2677990-2.patch889 bytesDuaelFr
#72 file_description_formatter_setting-2677990-72.patch20.58 KBDuaelFr
#72 interdiff.2677990.60.72.txt1.98 KBDuaelFr
#75 file_description_formatter_setting-2677990-75.patch20.54 KBDuaelFr
#79 interdiff.2677990.75.78.txt555 bytesDuaelFr
#79 file_description_formatter_setting-2677990-78.patch20.55 KBDuaelFr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DuaelFr created an issue. See original summary.

DuaelFr’s picture

Issue summary: View changes
Status: Active » Needs review
Issue tags: +Needs tests
FileSize
889 bytes

Added steps to reproduce + patch fixing the issue.
I suppose we are going to need tests...

heykarthikwithu’s picture

1. Installed drupal standard 8.0.x
2. Edit or create a content type
3. Add a file field with description enabled, with unlimited values (correct me if iam wrong)
4. Choose the "Table of files" formatter
5. Create a node, upload a file, add a description
5. Show the node

the file name is displayed and the description does not appear anywhere

After applying the patch, i was getting the same i.e file name itself was displayed.

DuaelFr’s picture

Did you clear the cache after having the patch applied?
It works well on my Vanilla Drupal and on the project which needs this fix.

N.kishorekumar’s picture

Status: Needs review » Reviewed & tested by the community

Created file field, with unlimited values and selected formatter as TableFormatter
In node creation, uploaded the file and gave the description for the file

After Applying this patch, the file name is replaced with the description

Moving it to RTBC

catch’s picture

Status: Reviewed & tested by the community » Needs work

Patch looks fine, but this needs test coverage.

heykarthikwithu’s picture

Assigned: Unassigned » heykarthikwithu

working on this

heykarthikwithu’s picture

Assigned: heykarthikwithu » Unassigned
Status: Needs work » Needs review
FileSize
2.2 KB
1.21 KB

Status: Needs review » Needs work

The last submitted patch, 8: 2677990-8.patch, failed testing.

gvso’s picture

Status: Needs work » Needs review
FileSize
2.15 KB

The test only evaluates files without description so far. I couldn't find a way of providing a description to the file in the test.

Changing status to "Needs review", so the bot can test it.

DuaelFr’s picture

Thanks you @gvso for starting this test.
Here is an upgrade that injects a description in the field then test that it is displayed with the default formatter and the "Table of files" formatter.

The last submitted patch, 11: table_of_files_2677990-11-tests-only.patch, failed testing.

gvso’s picture

Status: Needs review » Reviewed & tested by the community

RBTC +1

alexpott’s picture

Status: Reviewed & tested by the community » Needs review

Are we sure that we should change this behaviour on existing sites?

heykarthikwithu’s picture

Are we sure that we should change this behaviour on existing sites?

@alexpott yes, because this is same behaviour which is in 7.x :)

DuaelFr’s picture

I really think this is an issue as you can ask for a description that's not shown nor even used anywhere with this formatter.
In my case, that's a client that asked me why it didn't work. I never expected it was on purpose.
Who could judge?

Edit: @heykarthikwithu good point!

alexpott’s picture

It's good to know this is how it was in 7.x.

So one thing that I think it confusing is the if you then go and disable the description test on the field the description is still used. This is the same for generic files. To me that is also unexpected. If the display used the description depending on whether that box was ticked that'd make more sense to me. Generic files will also keep using the description regardless of this setting.

DuaelFr’s picture

Shouldn't we open a follow-up issue for that particular thing?
It's going to need tests and also be delayed for a futur minor release.

heykarthikwithu’s picture

actually I want to say this.

"Enable Description field" setting is at the field level (admin/structure/types/manage/article/fields/node.article.field_file_),
so description will be enable/disabled irrespective of fields formatter level setting (admin/structure/types/manage/article/display) this value may be 'Generic file', 'Table of formatter' or 'URL to file'.

So, IMO
Should we also have settings at field formatter level?? i.e for 'Table of formatter'

alexpott’s picture

@DuaelFr the problem is we're changing existing behaviour which sites other than yours might now be relying on. And once we change this patch there's no way to get back to the way it used to behave without disabling the description and re-saving every node.

@heykarthikwithu I agree this should be a setting on the display - whether to use the description if it is available.

DuaelFr’s picture

Status: Needs review » Needs work
Issue tags: +Needs issue summary update, +Needs title update, +Novice

According to the last comments we have to:
- add a setting to the "Generic file" formatter to use the description when available (Enabled by default)
- add a setting to the "Table of files" formatter to use the description when available (Disabled by default)
- increase test coverage to ensure these new settings are working

alexpott’s picture

add a setting to the "Table of files" formatter to use the description when available (Disabled by default)

Well it can default to enabled for new sites. But existing sites should not be changed.

DuaelFr’s picture

@alexpott Sure but it would need an upgrade path and I don't understand how to write upgrade path tests ;)

alexpott’s picture

@DuaelFr never too late to learn :) - any of the tests that extend UpdatePathTestBase is a good place to start. And whatever the defaults you'll probably need an upgrade path for both displays.

tstoeckler’s picture

Issue tags: +DrupalBCDays
hauruck’s picture

I have tested the patch from #11 on the 8.2.x branch and it didn't resolve the problem...

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

DuaelFr’s picture

Assigned: Unassigned » DuaelFr

I'll spend a couple hours on this right now.

DuaelFr’s picture

Title: "Table of files" formatter (TableFormatter) does not use files descriptions » Add a setting on "Table of files" and "Generic files" formatters to use files descriptions (or not)
Version: 8.1.x-dev » 8.2.x-dev
Category: Bug report » Feature request
Issue summary: View changes
Issue tags: -Needs issue summary update, -Needs title update, -Novice

Updated the title and the Issue Summary to reflect the new orientation of this issue.
Removed from the IS but kept in case it's needed:

Steps to reproduce the "Table of files" formatter issue

  1. Install drupal standard
  2. Edit or create a content type
  3. Add a (multiple) file field with description enabled
  4. Choose the "Table of files" formatter
  5. Create a node, upload a file, add a description
  6. Show the node

Expected result: the file name is replaced by the description
Current result: the file name is displayed and the description does not appear anywhere

DuaelFr’s picture

DuaelFr’s picture

Issue summary: View changes

I added the tasks table in the issue summary.

Status: Needs review » Needs work

The last submitted patch, 30: file_description_formatter_setting-2677990-30.patch, failed testing.

DuaelFr’s picture

Fixed the tests and added upgrade tests.
*crossing fingers*

The last submitted patch, 34: file_description_formatter_setting-2677990-34.patch, failed testing.

DuaelFr’s picture

Requeueing the test as it looks like a random failure.

The last submitted patch, 34: file_description_formatter_setting-2677990-34.patch, failed testing.

The last submitted patch, 34: file_description_formatter_setting-2677990-34.patch, failed testing.

The last submitted patch, 34: file_description_formatter_setting-2677990-34.patch, failed testing.

DuaelFr’s picture

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.0-beta1 was released on August 3, 2016, which means new developments and disruptive changes should now be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

DuaelFr’s picture

Issue summary: View changes
esolitos’s picture

Status: Needs review » Reviewed & tested by the community

I read the patch in #41, applied the patch on a couple of sites and it all looks very good.

xjm’s picture

#2654668: Show file description instead of filename if description has been provided was also in the RTBC queue; looks like a partial dupe. This issue offers a more complete solution, I think.

Also rearranging the files shown since an old patch is listed at the top.

The last submitted patch, 41: file_description_formatter_setting-2677990-41.patch, failed testing.

DuaelFr’s picture

Reupload of patches from #41 to be tested against 8.3.x.

Status: Needs review » Needs work
DuaelFr’s picture

Status: Needs work » Reviewed & tested by the community

Testbot is happy. Let's go back to RTBC :)

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
+++ b/core/modules/file/config/schema/file.schema.yml
@@ -70,6 +70,10 @@ field.field_settings.file:
+    show_description:
+      type: boolean
+      label: 'Replace the file name by its description when available'

The config key and the label seem to be a mismatch to me. show_description implies that a description is going to additionally shown whereas the reality is that the description is used instead of the filename. Perhaps the setting should be use_description_as_link_text since this is what is happening.

DuaelFr’s picture

Status: Needs review » Needs work

The last submitted patch, 52: file_description_formatter_setting-2677990-52.patch, failed testing.

The last submitted patch, 52: file_description_formatter_setting-2677990-52.patch, failed testing.

DuaelFr’s picture

Status: Needs work » Reviewed & tested by the community

Testbot is happy now. Let's go back to RTBC as the changes between #48 and #52 are minor.

Wim Leers’s picture

  1. +++ b/core/modules/file/file.install
    @@ -120,3 +122,22 @@ function file_requirements($phase) {
    + * This is made to ensure Backward Compatibility.
    

    Nit: s/made/done/

  2. +++ b/core/modules/file/file.install
    @@ -120,3 +122,22 @@ function file_requirements($phase) {
    +    /** @var EntityViewDisplay $display */
    

    Needs FQCN.

  3. +++ b/core/modules/file/src/Tests/Update/FileUpdateTest.php
    @@ -0,0 +1,57 @@
    +    // Check that field_file_generic formatter has no use_description_as_link_text setting.
    ...
    +    // Check that field_file_table formatter has no use_description_as_link_text setting.
    ...
    +    // Check that field_file_generic formatter still has no use_description_as_link_text
    ...
    +    // Check that field_file_table formatter has a use_description_as_link_text setting
    

    Nit: 80 cols violations.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs change record
+++ b/core/modules/file/file.install
@@ -120,3 +122,22 @@ function file_requirements($phase) {
+ * This is made to ensure Backward Compatibility.

Also unnecessary capitalisation of Backward Compatibility.

Let's fix this and #58 and add a change record.

DuaelFr’s picture

DuaelFr’s picture

Status: Needs work » Needs review
Issue tags: -Needs change record

And there is the change record https://www.drupal.org/node/2838648
I hope my english is not too bad :/

The last submitted patch, 60: file_description_formatter_setting-2677990-60.patch, failed testing.

The last submitted patch, 60: file_description_formatter_setting-2677990-60.patch, failed testing.

DuaelFr’s picture

It really looks like random failures o_O

DuaelFr’s picture

Status: Needs review » Reviewed & tested by the community

\o/ It passed !
Go back to RTBC as it was only comment fixes.
Please review the attached change record too (I'm not good at documentation).

The last submitted patch, 60: file_description_formatter_setting-2677990-60.patch, failed testing.

The last submitted patch, 60: file_description_formatter_setting-2677990-60.patch, failed testing.

The last submitted patch, 60: file_description_formatter_setting-2677990-60.patch, failed testing.

The last submitted patch, 60: file_description_formatter_setting-2677990-60.patch, failed testing.

The last submitted patch, 60: file_description_formatter_setting-2677990-60.patch, failed testing.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

I've just spotted a problem we need to update the generic file formatters to but to have the setting set to TRUE. Because after this patch is applied if you just go to field settings and press save on a generic file field the configuration will change (to have settings.use_description_as_link_text set to TRUE). Therefore we need to handle this in the update as well.

DuaelFr’s picture

Here we go!

Status: Needs review » Needs work

The last submitted patch, 72: file_description_formatter_setting-2677990-72.patch, failed testing.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.0-alpha1 will be released the week of January 30, 2017, which means new developments and disruptive changes should now be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

DuaelFr’s picture

Status: Needs work » Needs review
Issue tags: +seville2017
FileSize
20.54 KB

Rerolled against 8.4.x
Triggering test bot while doing some mentoring

Status: Needs review » Needs work

The last submitted patch, 75: file_description_formatter_setting-2677990-75.patch, failed testing.

DuaelFr’s picture

Issue tags: -seville2017 +DevDaysSeville

Wrong tag, sorry

DuaelFr’s picture

Assigned: Unassigned » DuaelFr

I'm on it

DuaelFr’s picture

Assigned: DuaelFr » Unassigned
Status: Needs work » Needs review
FileSize
20.55 KB
555 bytes
Haza’s picture

Status: Needs review » Reviewed & tested by the community

Issues have been addressed, patch green and tested and already got RTBC before, so, let's go back to RTBC !

iMiksu’s picture

Issue summary: View changes
Issue tags: +Needs screenshots

I assume that in #80 the patch was tested and reviewed to ensure that it fixes the issue, stays within scope, is properly documented, and follows coding standards?

However, we could use some screenshots here.

DuaelFr’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Novice
+++ b/core/modules/file/file.install
@@ -116,3 +118,29 @@ function file_requirements($phase) {
+ * Force "Table of files" formatters' "use_description_as_link_text" settings off.

That's a bit more than 80 chars :p


Adding screenshots and fixing this patch are novice tasks. Have fun mates!

agomezmoron’s picture

Assigned: Unassigned » agomezmoron

Working on it

agomezmoron’s picture

agomezmoron’s picture

Assigned: agomezmoron » Unassigned
Status: Needs work » Needs review
FileSize
10.49 KB
480 bytes

Fixing 80 chars in the line as @DuaelFr suggested :)

agomezmoron’s picture

Issue summary: View changes
agomezmoron’s picture

Issue tags: -Needs screenshots

Status: Needs review » Needs work

The last submitted patch, 85: file_description_formatter_setting-2677990-81.patch, failed testing.

agomezmoron’s picture

Re-uploading .patch & interdiff, I missed some changes.

Sorry!

heykarthikwithu’s picture

Status: Needs review » Reviewed & tested by the community
DuaelFr’s picture

Issue tags: -Novice
FileSize
480 bytes

Thank you all for your help here! I hope we can see this one commited soon :)

Beware, the interdiff in #89 is wrong, though. I applied #79 then #89 to check differences and here is the right interdiff file.

agomezmoron’s picture

Oh! My apologizes and thanks @DuaelFr!

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
+++ b/core/modules/file/file.install
@@ -116,3 +118,30 @@ function file_requirements($phase) {
+ * Force "Table of files" formatters' "use_description_as_link_text"
+ * settings off.

This change is actually incorrect. hook_update_N function descriptions need to be on one line. And it is important because they are actually displayed to the user. Also I thought they are excluded from the 80 character limit but I can't find that documented on https://www.drupal.org/docs/develop/coding-standards/api-documentation-a... so I think the 80 character limit applies.

How about Add 'use_description_as_link_text' setting to file field formatters.

Also the second part of the file docblock probably could explain the choice of default values better than just say This is done to ensure backward compatibility. - imo that line should be removed and inline docs added to the function about why it's TRUE for file_default and FALSE for file_table.

DuaelFr’s picture

Let's try toget this one fixed before the summer ;)
@alexpott I made the changes you asked for, I hope my comments are not too bad now.

esolitos’s picture

Status: Needs review » Reviewed & tested by the community

Changes are ok as comments are perfectly understandable, back to RTBC. :)

Status: Reviewed & tested by the community » Needs work
DuaelFr’s picture

Status: Needs review » Needs work
DuaelFr’s picture

tstoeckler’s picture

Status: Needs work » Needs review
tstoeckler’s picture

Status: Needs review » Reviewed & tested by the community

Back to RTBC. Thanks, @DuaelFr!

DuaelFr’s picture

\o/ Thank you for the reviews :)

larowlan’s picture

Status: Reviewed & tested by the community » Needs work

This is looking great, thanks - one think we need one minor change to the update path

  1. +++ b/core/modules/file/config/schema/file.schema.yml
    @@ -78,6 +82,10 @@ field.formatter.settings.file_rss_enclosure:
     field.formatter.settings.file_table:
       type: mapping
       label: 'Table of files format settings'
    +  mapping:
    +    use_description_as_link_text:
    +      type: boolean
    +      label: 'Replace the file name by its description when available'
    

    if we change this from type: mapping to type: field.formatter.settings.file_default then we don't need to duplicate the new property. Given they both are the same, makes sense I think.

  2. +++ b/core/modules/file/file.install
    @@ -116,3 +118,31 @@ function file_requirements($phase) {
    +          // The file_default formatter always displayed available descriptions
    

    nit: should be blank line after break; - can be fixed on commit

  3. +++ b/core/modules/file/file.install
    @@ -116,3 +118,31 @@ function file_requirements($phase) {
    +    $display->set('content', $fields_settings)->save();
    

    I think we should keep track of which displays we actually made changes too and only call ::set and ::save if we actually changed something

  4. +++ b/core/modules/file/src/Plugin/Field/FieldFormatter/DescriptionAwareFileFormatterBase.php
    @@ -0,0 +1,52 @@
    + * Base class for file formatters that have to deal with files descriptions.
    + */
    

    should be 'file descriptions' not 'files descriptions', can be fixed on commit

  5. +++ b/core/modules/file/src/Tests/FileFieldDisplayTest.php
    @@ -173,4 +174,49 @@ public function testDescToggle() {
    +    $field_storage_settings = array(
    ...
    +    );
    +    $field_settings = array(
    ...
    +    );
    +    $widget_settings = array();
    ...
    +    $display->setComponent($field_name, array(
    

    needs to use short array syntax now

  6. +++ b/core/modules/file/tests/fixtures/update/drupal-8.file_formatters_update_2677990.php
    @@ -0,0 +1,88 @@
    +$field_file_generic_2677990 = Yaml::decode(file_get_contents(__DIR__ . '/field.storage.node.field_file_generic_2677990.yml'));
    ...
    +$connection->insert('config')
    

    seeing this 'read some yml, insert into config' pattern fairly regularly, we need a helper class I reckon, if someone feels like opening a follow-up task. Would be a class with static methods that took the connection as an argument.

Thanks again for the hard work here

DuaelFr’s picture

#103 points 1 to 5 done

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.0-alpha1 will be released the week of July 31, 2017, which means new developments and disruptive changes should now be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

DuaelFr’s picture

Wim Leers’s picture

Status: Needs review » Reviewed & tested by the community
DuaelFr’s picture

❤️

larowlan’s picture

Adding credits for reviews that helped shape the patch

larowlan’s picture

Status: Reviewed & tested by the community » Fixed

Fixed on commit

diff --git a/core/modules/file/src/Tests/FileFieldDisplayTest.php b/core/modules/file/src/Tests/FileFieldDisplayTest.php
index c01a41c..9589db1 100644
--- a/core/modules/file/src/Tests/FileFieldDisplayTest.php
+++ b/core/modules/file/src/Tests/FileFieldDisplayTest.php
@@ -177,7 +177,7 @@ public function testDescToggle() {
   /**
    * Tests description display of File Field.
    */
-  function testDescriptionDefaultFileFieldDisplay() {
+  public function testDescriptionDefaultFileFieldDisplay() {
     $field_name = strtolower($this->randomMachineName());
     $type_name = 'article';
     $field_storage_settings = [

Committed as daf25aa and pushed to 8.5.x

Published change record

  • larowlan committed daf25aa on 8.5.x
    Issue #2677990 by DuaelFr, agomezmoron, heykarthikwithu, gvso, alexpott...

Status: Fixed » Closed (fixed)

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