Add a setting to make description a required field for file items

Using MR#3117 with 9.5.7, works. Thank you.

As this affects core and I have the need to apply multiple patches, I created a patch for 10.x to apply.

Re-rolled for 10.1.x. I based this MR on MR#3117 because the patch in the previous comment introduced some changes from the last version, most notably removing the constraints that were added in a previous commit. Maybe the patch was based on an earlier version?

I did not make any changes to the re-roll, so the updates noted in #85 ✨ Add a setting to make description a required field for file items Needs work still need to be implemented. I'm working on manually testing it.

Manual testing (WIP)

I used the Live preview via Tugboat (TIL I learned there's a Live preview via Tugboat!)

1. Create a file field (field_myfile1) in the Basic page content type admin/structure/types/manage/page/fields/add-field
2. Check "Enable Description field"
3. "Require the Description field" displays and is not enabled
4. Uncheck "Enable..."
5. "Require..." field is not displayed & Save
6. Create a file field (field_myfile2)
7. Enable Description but don't Require & Save
8. Create a file field (field_myfile3)
9. Enable Description and Require & Save
10. Add Basic page node & Add files to fields
11. Save without adding a description to the required field_myfile3
12. Unable to save (fails both client- and server-side validation)
13. Add a description to myfile3 and save
14. Not good* Can't save because I didn't add a description to myfile1. When I added the field, I Enabled description, enabled Require, and then Disabled description, but didn't disable Require. Now it's expecting a value, but the Description field doesn't display.

With the patch from #94 applied, we have observed PHP warnings, similar to 🐛 Undefined array key "description_field_required" in \Drupal\file\Plugin\Field\FieldWidget\FileWidget::formElement Needs work :

Undefined array key "description_field" in \Drupal\file\Plugin\Field\FieldWidget\FileWidget::formElement

To address the Undefined array key "description_field" in \Drupal\file\Plugin\Field\FieldWidget\FileWidget::formElement for issue 104, we can use this patch

Tested and it's working fine

@guardiola86 Thanks for your review and the status change, but this issue should be a Needs Work at the moment.

Which patch/MR have you reviewed? If the one from #105, this patch does not apply. The bigger issue is, that the content of that patch is significantly different from the MR 3915 and MR 4255 (see for example the default label "Require Description field"). Therefore I am not sure what was the starting point for this patch. Also the patch is missing interdiff, so we are unable to see what changes were made.

It is good to use MRs in case these are already created and not to upload new patches.

@Vanitha Sophia Thanks for working on this issue, but at the start I suggest to read this contributor guide: https://www.drupal.org/community/contributor-guide/task/create-a-patch-f... → before adding another patches here.

There is a failure in the MR pipeline, which does not looks like a random one, but related to the MR changes:

There was 1 error:
    
    1) Drupal\Tests\file\Functional\FileFieldDisplayTest::testDescToggle
    Behat\Mink\Exception\ElementNotFoundException: Button with
    id|name|label|value "Save and continue" not found.
    
    /builds/issue/drupal-2320877/core/tests/Drupal/Tests/WebAssert.php:144
    /builds/issue/drupal-2320877/core/tests/Drupal/Tests/UiHelperTrait.php:78
    /builds/issue/drupal-2320877/core/modules/file/tests/src/Functional/FileFieldDisplayTest.php:178
    /builds/issue/drupal-2320877/vendor/phpunit/phpunit/src/Framework/TestResult.php:728

https://git.drupalcode.org/issue/drupal-2320877/-/jobs/507087

claudiu.cristea → changed the visibility of the branch 2320877-description-required-10.0.x to hidden.

claudiu.cristea → changed the visibility of the branch 2320877-9.5 to hidden.

claudiu.cristea → changed the visibility of the branch 2320877-10.1.x to hidden.

Tests are green. Ready for a final review.

Drupal 10.3.x version patch

I tested the patch provided on #115 on Drupal 10.3.0, and it worked as expected. See below the test steps I executed:

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Manual testing has been performed in #100 ✨ Add a setting to make description a required field for file items Needs work