Skip to content

Commit 49bce68

Browse files
authored
Merge pull request #151 from catalyst/issue150
issue #150: fix undefined error
2 parents 9d7447e + 7124b5a commit 49bce68

File tree

1 file changed

+15
-13
lines changed

1 file changed

+15
-13
lines changed

classes/local/datasource/userlist.php

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -312,22 +312,24 @@ protected function from_repeatable(\stdClass $data) {
312312

313313
$repeathiddenname = self::FORM_REPEATHIDDENNAME;
314314
$defs = [];
315-
for ($i = 0; $i < $data->$repeathiddenname; ++$i) {
316-
if (isset($deletehidden[$i])) {
317-
continue;
318-
}
319-
$obj = new \stdClass();
320-
$defs[$i] = $obj;
321-
}
322-
323-
foreach ($columndefs as $columndef) {
324-
$names = $this->get_element_names($columndef);
325-
$formname = $names->ulelementname;
326-
foreach ($data->$formname as $i => $val) {
315+
if (isset($data->$repeathiddenname)) {
316+
for ($i = 0; $i < $data->$repeathiddenname; ++$i) {
327317
if (isset($deletehidden[$i])) {
328318
continue;
329319
}
330-
$defs[$i]->$formname = $val;
320+
$obj = new \stdClass();
321+
$defs[$i] = $obj;
322+
}
323+
324+
foreach ($columndefs as $columndef) {
325+
$names = $this->get_element_names($columndef);
326+
$formname = $names->ulelementname;
327+
foreach ($data->$formname as $i => $val) {
328+
if (isset($deletehidden[$i])) {
329+
continue;
330+
}
331+
$defs[$i]->$formname = $val;
332+
}
331333
}
332334
}
333335

0 commit comments

Comments
 (0)