WMDE
/
LDAPAccountManager
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

missing setMarkAsRequired 

Same kind of missing call as before, in addMultiValueInputTextField this time.

Although, even after this patch, whenever submitting changes with an empty value on a required field (created via addMultiValueInputTextField), LAM seem to accept the changes and does not throw a warning.

Where in the code is the part actually checking for required fields ?

Cheers
This commit is contained in:
Kyoshiro 2017-07-21 16:50:12 +02:00 committed by GitHub
parent 204798567f
commit 6ee20cfd9b
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lam/lib/baseModule.inc
View File

@ -1305,6 +1305,7 @@ abstract class baseModule {
if ($label !== null) { if ($label !== null) {
$labelTextOut = new htmlOutputText($label, true, $required); $labelTextOut = new htmlOutputText($label, true, $required);
$labelTextOut->alignment = htmlElement::ALIGN_TOP; $labelTextOut->alignment = htmlElement::ALIGN_TOP;
$labelTextOut->setMarkAsRequired($required);
$container->addElement($labelTextOut); $container->addElement($labelTextOut);
} }
$subContainer = new htmlTable(); $subContainer = new htmlTable();
@ -1925,4 +1926,4 @@ abstract class baseModule {
} }
?> ?>