Bug #88241

ext:form restor "min items > 1" behavior within the Inspector-PropertyGridEditor

Added by Ralf Zimmermann 11 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2019-04-28
Due date:
% Done:

0%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
Remote Sprint

Description

With #80420 we accidentally lost the behavior that an inspector editor with type "Inspector-PropertyGridEditor" must contain at least one item to be valid.
Since (#80420) we now have the use cases that an "Inspector-PropertyGridEditor" editor can also contain no items.

  • restore the warning flash message if the user want to delete the last item
  • make the "Inspector-PropertyGridEditor" configurable to avoid the warning flash message if the user want to delete the last item

2019-11-21_11-15.png View (13.6 KB) Mathias Brodala, 2019-11-21 11:15

2019-11-21_11-15_1.png View (13.1 KB) Mathias Brodala, 2019-11-21 11:16


Related issues

Related to TYPO3 Core - Feature #80420: Allow multiple recipients in EmailFinisher Closed 2017-03-23

History

#1 Updated by Gerrit Code Review 11 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/60615

#2 Updated by Susanne Moog 5 months ago

  • Sprint Focus set to Remote Sprint

#3 Updated by Mathias Brodala 4 months ago

  • Related to Feature #80420: Allow multiple recipients in EmailFinisher added

#4 Updated by Mathias Brodala 4 months ago

As discussed: the property grid editor already properly marks the whole field as "required" as long as no item was added. Saving the form is impossible in this case. This was verified once more with recipients of the mail finisher and items of a multi-select form element:

#5 Updated by Mathias Brodala 4 months ago

  • Status changed from Under Review to Closed

Also available in: Atom PDF