Double model error displayed for required MediaLibraryPickerFields

OrchardCMS / Orchard

Orchard is a free, open source, community-focused Content Management System built on the ASP.NET MVC platform.

https://orchardproject.net

BSD 3-Clause "New" or "Revised" License

2.37k stars 1.12k forks source link

Double model error displayed for required MediaLibraryPickerFields #8699

Closed AndreaPiovanelli closed 1 year ago

AndreaPiovanelli commented 1 year ago

https://github.com/OrchardCMS/Orchard/blob/2ef2ad8e99948625ebebb7e05ca609eaa85c2894/src/Orchard.Web/Modules/Orchard.MediaLibrary/Handlers/MediaLibraryPickerFieldLocalizationExtensionHandler.cs#L122

If MLPF is required and no media has been selected, a double ModelError is displayed (one by MediaLibraryPickerFieldDriver and one by MediaLibraryPickerFieldLocalizationExtensionHandler).

AndreaPiovanelli commented 1 year ago

@sebastienros this has been fixed by #8700 , which of course is based on dev branch, as it is related to localization on MLPF.