From 06a43900602d78f9b8d37892ff7f076b1a68dfe9 Mon Sep 17 00:00:00 2001 From: Simey Lameze Date: Fri, 30 Oct 2020 07:17:43 +0800 Subject: [PATCH] MDL-70066 badges: change usages of array_key_exists to isset --- badges/classes/external/assertion_exporter.php | 12 ++++++------ badges/classes/external/badgeclass_exporter.php | 16 ++++++++-------- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/badges/classes/external/assertion_exporter.php b/badges/classes/external/assertion_exporter.php index 4e1e393773f..06b523803e3 100644 --- a/badges/classes/external/assertion_exporter.php +++ b/badges/classes/external/assertion_exporter.php @@ -104,16 +104,16 @@ class assertion_exporter extends exporter { global $DB; $result = []; - if (array_key_exists('related_badge', $this->data)) { - $exporter = new badgeclass_exporter($this->data['related_badge'], $this->related); + if (property_exists($this->data, 'related_badge')) { + $exporter = new badgeclass_exporter($this->data->related_badge, $this->related); $result['badge'] = $exporter->export($output); } - if (array_key_exists('related_recipient', $this->data)) { - $exporter = new recipient_exporter($this->data['related_recipient'], $this->related); + if (property_exists($this->data, 'related_recipient')) { + $exporter = new recipient_exporter($this->data->related_recipient, $this->related); $result['recipient'] = $exporter->export($output); } - if (array_key_exists('related_verify', $this->data)) { - $exporter = new verification_exporter($this->data['related_verify'], $this->related); + if (property_exists($this->data, 'related_verify')) { + $exporter = new verification_exporter($this->data->related_verify, $this->related); $result['verification'] = $exporter->export($output); } return $result; diff --git a/badges/classes/external/badgeclass_exporter.php b/badges/classes/external/badgeclass_exporter.php index f16d7628b73..4d4880fbad1 100644 --- a/badges/classes/external/badgeclass_exporter.php +++ b/badges/classes/external/badgeclass_exporter.php @@ -54,28 +54,28 @@ class badgeclass_exporter extends exporter { $isarray = true; } $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) { $newrelated = []; - foreach ($data[$one] as $item) { + foreach ($data->$one as $item) { $newrelated[] = (object) $item; } $related[$one] = $newrelated; } else { - $related[$one] = (object) $data[$one]; + $related[$one] = (object) $data->$one; } - unset($data[$one]); - } else if (array_key_exists($prefixed, $data) && !array_key_exists($one, $related)) { + unset($data->$one); + } else if (property_exists($data, $prefixed) && !array_key_exists($one, $related)) { if ($isarray) { $newrelated = []; - foreach ($data[$prefixed] as $item) { + foreach ($data->$prefixed as $item) { $newrelated[] = (object) $item; } $related[$one] = $newrelated; } else { - $related[$one] = (object) $data[$prefixed]; + $related[$one] = (object) $data->$prefixed; } - unset($data[$prefixed]); + unset($data->$prefixed); } else if (!array_key_exists($one, $related)) { $related[$one] = null; }