No Code Attached Yet
avatar chmst
chmst
6 Jun 2021

Steps to reproduce the issue

Create a field of type checkboxes.
Add one or two text / value pairs with content in text and value
Add another and let text and value empty.

grafik

Expected result

Empty text / value is ignored

grafik

Actual result

For the empty text/value pair, a checkbox without label is displayed

grafik

System information (as much as possible)

Additional comments

avatar chmst chmst - open - 6 Jun 2021
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jun 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 Jun 2021
avatar brianteeman
brianteeman - comment - 6 Jun 2021

should they be made required? or ignored and not saved

avatar chmst
chmst - comment - 6 Jun 2021

I would say required. Not sure if this makes difficulties with subforms, but this shpud be independent and ignored and not saved would also be ok.

avatar brianteeman
brianteeman - comment - 6 Jun 2021

I dont know either just wanted to clarify ;)

avatar richard67
richard67 - comment - 6 Jun 2021

I think the text and value fields should be required so it’s not possible to save with an empty text or value.

avatar AndySDH
AndySDH - comment - 17 Aug 2021

I think the text and value fields should be required so it’s not possible to save with an empty text or value.

An empty value could actually still be useful as you may want an option (eg. in a List field) to not save a value in the database.
For example having a text "None" and a an empty value as the first option in the List dropdown could make sense.

However, the text part should indeed probably be required. An empty text doesn't make sense.

avatar brianteeman
brianteeman - comment - 25 Feb 2022

Please test #37138

avatar richard67 richard67 - change - 25 Feb 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-02-25 07:53:19
Closed_By richard67
Labels Added: No Code Attached Yet
Removed: ?
avatar richard67 richard67 - close - 25 Feb 2022
avatar richard67
richard67 - comment - 25 Feb 2022

Closing as having a pull request. Please test #37138 . Thanks in advance.

Add a Comment

Login with GitHub to post a comment