MDL-70066 badges: change usages of array_key_exists to isset

This commit is contained in:
Simey Lameze 2020-10-30 07:17:43 +08:00
parent a3f72b3972
commit 06a4390060
2 changed files with 14 additions and 14 deletions

View file

@ -104,16 +104,16 @@ class assertion_exporter extends exporter {
global $DB; global $DB;
$result = []; $result = [];
if (array_key_exists('related_badge', $this->data)) { if (property_exists($this->data, 'related_badge')) {
$exporter = new badgeclass_exporter($this->data['related_badge'], $this->related); $exporter = new badgeclass_exporter($this->data->related_badge, $this->related);
$result['badge'] = $exporter->export($output); $result['badge'] = $exporter->export($output);
} }
if (array_key_exists('related_recipient', $this->data)) { if (property_exists($this->data, 'related_recipient')) {
$exporter = new recipient_exporter($this->data['related_recipient'], $this->related); $exporter = new recipient_exporter($this->data->related_recipient, $this->related);
$result['recipient'] = $exporter->export($output); $result['recipient'] = $exporter->export($output);
} }
if (array_key_exists('related_verify', $this->data)) { if (property_exists($this->data, 'related_verify')) {
$exporter = new verification_exporter($this->data['related_verify'], $this->related); $exporter = new verification_exporter($this->data->related_verify, $this->related);
$result['verification'] = $exporter->export($output); $result['verification'] = $exporter->export($output);
} }
return $result; return $result;

View file

@ -54,28 +54,28 @@ class badgeclass_exporter extends exporter {
$isarray = true; $isarray = true;
} }
$prefixed = 'related_' . $one; $prefixed = 'related_' . $one;
if (array_key_exists($one, $data) && !array_key_exists($one, $related)) { if (property_exists($data, $one) && !array_key_exists($one, $related)) {
if ($isarray) { if ($isarray) {
$newrelated = []; $newrelated = [];
foreach ($data[$one] as $item) { foreach ($data->$one as $item) {
$newrelated[] = (object) $item; $newrelated[] = (object) $item;
} }
$related[$one] = $newrelated; $related[$one] = $newrelated;
} else { } else {
$related[$one] = (object) $data[$one]; $related[$one] = (object) $data->$one;
} }
unset($data[$one]); unset($data->$one);
} else if (array_key_exists($prefixed, $data) && !array_key_exists($one, $related)) { } else if (property_exists($data, $prefixed) && !array_key_exists($one, $related)) {
if ($isarray) { if ($isarray) {
$newrelated = []; $newrelated = [];
foreach ($data[$prefixed] as $item) { foreach ($data->$prefixed as $item) {
$newrelated[] = (object) $item; $newrelated[] = (object) $item;
} }
$related[$one] = $newrelated; $related[$one] = $newrelated;
} else { } else {
$related[$one] = (object) $data[$prefixed]; $related[$one] = (object) $data->$prefixed;
} }
unset($data[$prefixed]); unset($data->$prefixed);
} else if (!array_key_exists($one, $related)) { } else if (!array_key_exists($one, $related)) {
$related[$one] = null; $related[$one] = null;
} }