Closed (fixed)
Project:
Path File
Version:
2.0.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
11 Dec 2020 at 05:59 UTC
Updated:
12 Jun 2026 at 13:50 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
chandravilasThis attach patch fulfils requirements for required mandatory field.
https://www.drupal.org/files/issues/2020-12-11/file_field_required-31877...
Comment #3
chandravilasComment #6
batigolixI think all fields should be mandatory because otherwise it doesnt maken a lot of sense. So I updated the merge request to reflect that. this can be reviewed
Comment #7
batigolixComment #8
batigolixComment #9
arjenk commentedTested MR !13.
First confirmed the non-required fields: a save with empty fields was possible, and even crashed the page with an error.
The patch adds a
->setRequired(TRUE)to the fields, and is clean. After applying the patch, the three fields were required, and save is blocked. (even URL alias had a test on starting with a '/').phpcs/phpstan all good, pipeline green. RTBC.
Comment #10
batigolix