X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=plugins%2FQnA%2Factions%2Fqnashowquestion.php;h=cc8f98448aa1e259f4d14bb96aba31ad43999cd3;hb=78dc376f017cd7a9ee746e961dd8eefb8941eff2;hp=c30464bbf61aed2efb03f27f17892512910ebf1c;hpb=31b29fde50e9664e1b70064c043879ce87553883;p=quix0rs-gnu-social.git diff --git a/plugins/QnA/actions/qnashowquestion.php b/plugins/QnA/actions/qnashowquestion.php index c30464bbf6..cc8f98448a 100644 --- a/plugins/QnA/actions/qnashowquestion.php +++ b/plugins/QnA/actions/qnashowquestion.php @@ -57,11 +57,11 @@ class QnashowquestionAction extends ShownoticeAction */ function prepare($argarray) { - OwnerDesignAction::prepare($argarray); + Action::prepare($argarray); $this->id = $this->trimmed('id'); - $this->question = QnA_Question::staticGet('id', $this->id); + $this->question = QnA_Question::getKV('id', $this->id); if (empty($this->question)) { // TRANS: Client exception thrown trying to view a non-existing question. @@ -76,7 +76,7 @@ class QnashowquestionAction extends ShownoticeAction throw new ClientException(_m('No such question notice.'), 404); } - $this->user = User::staticGet('id', $this->question->profile_id); + $this->user = User::getKV('id', $this->question->profile_id); if (empty($this->user)) { // TRANS: Client exception thrown trying to view a question of a non-existing user. @@ -90,7 +90,11 @@ class QnashowquestionAction extends ShownoticeAction throw new ServerException(_m('User without a profile.')); } - $this->avatar = $this->profile->getAvatar(AVATAR_PROFILE_SIZE); + try { + $this->avatar = $this->profile->getAvatar(AVATAR_PROFILE_SIZE); + } catch (Exception $e) { + $this->avatar = null; + } return true; } @@ -115,13 +119,7 @@ class QnashowquestionAction extends ShownoticeAction } if (count($answerIds) > 0) { - $notice = new Notice(); - $notice->query( - sprintf( - 'SELECT notice.* FROM notice WHERE notice.id IN (%s)', - implode(',', $answerIds) - ) - ); + $notice = Notice::multiGet('id', $answerIds); $nli = new NoticeList($notice, $this); $nli->show(); @@ -151,9 +149,9 @@ class QnashowquestionAction extends ShownoticeAction */ function title() { - // TRANS: Page title for a question. - // TRANS: %1$s is the nickname of the user who asked the question, %2$s is the question. return sprintf( + // TRANS: Page title for a question. + // TRANS: %1$s is the nickname of the user who asked the question, %2$s is the question. _m('%1$s\'s question: %2$s'), $this->user->nickname, $this->question->title @@ -168,7 +166,6 @@ class QnashowquestionAction extends ShownoticeAction return Action::lastModified(); } - /** * @fixme combine the notice time with question update time */