OpenPNE3 Plugins » opMessagePlugin

レビューしました．

本チケットの修正部分はOKですが，同ファイル内のコーディング規約違反および処理の無駄な部分が気になりました．量が多いため本チケットで一緒くたに扱うかどうかは不明ですが一旦差し戻しにします．

参考として，気になった部分を下記に列挙します．

if とブロック開始記号が同じ行に存在する．変数名にアンダースコアを用いている．

 56   public function getIsReceiver($member_id)
 57   {
 58     $message = Doctrine::getTable('MessageSendList')->getMessageByReferences($member_id, $this->getId());
 59     if ($message && $this->getIsSend()) {
 60       return 1;
 61     } else {
 62       return 0;
 63     }
 64   }

return の前の行に改行がない．

 70   public function getSendList()
 71   {
 72     $objs = Doctrine::getTable('MessageSendList')->getMessageSendList($this->getId());
 73     return $objs;
 74   }

使われない変数 $cnt に代入が行われている．比較の値が前にない．return の前に改行がない．

 80   public function getSendTo()
 81   {
 82     $objs = $this->getSendList();
 83     if ($cnt = count($objs) == 0) {
 84       return null;
 85     }
 86     return $objs[0]->getMember();
 87   }
 88 

return 前に改行がない．

 93   public function getMessageFiles(Doctrine_Query $q = null)
 94   {
 95     if (is_null($q))
 96     {
 97       $a = Doctrine::getTable('MessageFile')
 98         ->orderBy('id ASC');
 99     }
100     $files = parent::getMessageFile($q);
101     return $files;
102   }

比較の値が前にない．

155   public function getDecoratedMessageBody()
156   {
157     $type = $this->getMessageType()->type_name;
158     if ($type === 'message')
159     {
160       return $this->body;
161     }
162