Skip to content

Commit

Permalink
Merge pull request #513 from keymanapp/fix/512-languageName-property
Browse files Browse the repository at this point in the history
fix: use property_exists instead of array_key_exists
  • Loading branch information
mcdurdin authored Nov 28, 2024
2 parents 51b3463 + b549ceb commit 7ffa897
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions _includes/includes/ui/keyboard-details.php
Original file line number Diff line number Diff line change
Expand Up @@ -599,13 +599,13 @@ protected static function WriteKeyboardDetails() {
echo " <a id='expand-languages' href='#expand-languages'>Expand $count more &gt;&gt;</a>";
echo "<a id='collapse-languages' href='#collapse-languages'>&lt;&lt; Collapse</a> <span class='expand-languages'>";
}
if (array_key_exists('languageName', $detail)) {
if (property_exists($detail, 'languageName')) {
echo
"<a href='/keyboards?q=l:id:".htmlspecialchars(rawurlencode($bcp47)).
"' title='".htmlspecialchars($bcp47).": ".htmlspecialchars($detail->displayName)."'>" .
(!strcasecmp($bcp47, self::$bcp47) ? "<mark>".htmlspecialchars($detail->languageName)."</mark>" : htmlspecialchars($detail->languageName)).
"</a> ";
}
}
$n++;
}
if($n >= 3) {
Expand Down

0 comments on commit 7ffa897

Please sign in to comment.