do not validate value in SeedDMS_Core_Attribute::setValue(), it should have been done before

This commit is contained in:
Uwe Steinmann 2021-03-06 15:26:18 +01:00
parent 2a5a5b2053
commit ceed62bf2b

View File

@ -177,12 +177,14 @@ class SeedDMS_Core_Attribute { /* {{{ */
if($values) { if($values) {
$vsep = $this->getValueSetSeparator(); $vsep = $this->getValueSetSeparator();
if($valueset) { if($valueset) {
/* Validation should have been done before
$error = false; $error = false;
foreach($values as $v) { foreach($values as $v) {
if(!in_array($v, $valueset)) { $error = true; break; } if(!in_array($v, $valueset)) { $error = true; break; }
} }
if($error) if($error)
return false; return false;
*/
$valuesetstr = $this->_attrdef->getValueSet(); $valuesetstr = $this->_attrdef->getValueSet();
$value = $vsep.implode($vsep, $values); $value = $vsep.implode($vsep, $values);
} else { } else {
@ -203,12 +205,14 @@ class SeedDMS_Core_Attribute { /* {{{ */
} }
if($valueset) { if($valueset) {
/* Validation should have been done before
$error = false; $error = false;
foreach($values as $v) { foreach($values as $v) {
if(!in_array($v, $valueset)) { $error = true; break; } if(!in_array($v, $valueset)) { $error = true; break; }
} }
if($error) if($error)
return false; return false;
*/
$value = $valuesetstr[0].implode($valuesetstr[0], $values); $value = $valuesetstr[0].implode($valuesetstr[0], $values);
} else { } else {
$value = ','.implode(',', $values); $value = ','.implode(',', $values);