Feat - Add configurable table question type#31
Conversation
|
@ccailly suggested making the column limit a validation criterion (like those already present in Forms) rather than managing it manually. The benefits would be:
|
| use GlpiPlugin\Advancedforms\Model\QuestionType\TableQuestionConfig; | ||
| use GlpiPlugin\Advancedforms\Tests\AdvancedFormsTestCase; | ||
|
|
||
| final class TableQuestionValidationTest extends AdvancedFormsTestCase |
There was a problem hiding this comment.
Missing test for validateExtraDataInput()
We need two test cases (at least): one confirming the method accepts max_rows=50, and one confirming it rejects max_rows=51.
In addition to avoiding the need to redevelop the line count validation system, adding conditions based on the number of lines will also allow users to set visibility conditions based on the number of lines. |
|
Awaiting customer validation |
Co-authored-by: Stanislas <skita@teclib.com> Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
d1ebeb3 to
0c43bb3
Compare
Checklist before requesting a review
Please delete options that are not relevant.
Description
Adds a new Table question type to the plugin, letting form
designers collect tabular data with fully configurable columns.
Changes
QuestionTypeValidationInterface— blocks submission when a filled row leaves a required cell empty.Tests
Screenshots (if appropriate):
Fill form exemple :