Bug #70210

Epic #69955: Optimize new Extbase/ Fluid based rewrite of EXT:form

EXT:form - Validation of radiogroup/ checkboxgroup elements broken

Added by Bjoern Jacob about 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Must have
Category:
Form Framework
Target version:
Start date:
2015-09-30
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Right now the validation of radiogroup/ checkboxgroup elements is not working perfectly. The elements are validated but no validation message is shown.

Associated revisions

Revision edbc1d93 (diff)
Added by Ralf Zimmermann about 4 years ago

[BUGFIX] EXT:form - Validation of radiogroup/ checkboxgroup elements

The "required" validator has to check array values.

Resolves: #70210
Releases: master
Change-Id: I58b8fb93d7e684854d1834c7994c837922467f82
Reviewed-on: https://review.typo3.org/43710
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Bjoern Jacob about 4 years ago

  • Sprint Focus set to On Location Sprint

#2 Updated by Ralf Zimmermann about 4 years ago

  • Status changed from New to In Progress
  • Assignee set to Ralf Zimmermann

#3 Updated by Gerrit Code Review about 4 years ago

  • Status changed from In Progress 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 http://review.typo3.org/43710

#4 Updated by Gerrit Code Review about 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43710

#5 Updated by Gerrit Code Review about 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43710

#6 Updated by Bjoern Jacob about 4 years ago

Form to test this issue:


prefix = tx_form
confirmation = 1

postProcessor {
    1 = mail
    1 {
        recipientEmail = your@mail.com
        senderEmail = your@mail.com
    }
}

10 = CHECKBOXGROUP
10 {
    class = fieldset-subgroup
    legend {
        value = Edit this legend
    }
    name = checkboxgroup

    10 = CHECKBOX
    10 {
        label {
            value = Option 1
        }
        value = option1
    }

    20 = CHECKBOX
    20 {
        label {
            value = Option 2
        }
        value = option2
    }

    30 = CHECKBOX
    30 {
        label {
            value = Option 3
        }
        value = option3
    }
}

20 = RADIOGROUP
20 {
    class = fieldset-subgroup
    legend {
        value = Edit this legend
    }
    name = radiogroup

    10 = RADIO
    10 {
        label {
            value = Option 1
        }
        value = option4
    }

    20 = RADIO
    20 {
        label {
            value = Option 2
        }
        value = option5
    }
    30 = RADIO
    30 {
        label {
            value = Option 3
        }
        value = option6
    }
}

30 = CHECKBOX
30 {
    name = checkbox1
    label {
        value = Option 4
    }
    value = option7
}

40 = RADIO
40 {
    name = option1
    label {
        value = Option 5
    }
    value = option8
}

50 = SUBMIT
50 {
    name = 6
    value = Submit form
}

rules {
    10 = required
    10.element = checkboxgroup

    20 = required
    20.element = radiogroup

    30 = required
    30.element = checkbox1

    40 = required
    40.element = option1
}

#7 Updated by Gerrit Code Review about 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43710

#8 Updated by Gerrit Code Review about 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43710

#9 Updated by Gerrit Code Review about 4 years ago

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

#10 Updated by Gerrit Code Review about 4 years ago

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

#11 Updated by Gerrit Code Review about 4 years ago

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

#12 Updated by Gerrit Code Review about 4 years ago

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

#13 Updated by Ralf Zimmermann about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#14 Updated by Anja Leichsenring almost 4 years ago

  • Sprint Focus deleted (On Location Sprint)

#15 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF