Project

General

Profile

Bug(バグ) #2321

受信・送信済みメッセージ一覧画面の「日付」に下書きの作成日時が表示される

Added by Kiwa Sakai over 10 years ago. Updated over 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2011-07-29
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
[QA]バグ通知済:
No
3.8 で発生するか:
Unknown (未調査)

Description

受信メッセージ一覧 ( /message/receiveList )・送信済みメッセージ一覧 ( /message/sendList ) に表示される「日付」について、
一度下書きをしたメッセージの場合、実際の送受信日時ではなく下書き作成日時が表示される。


Related issues

Related to opMessagePlugin - Bug(バグ) #2322: 受信・送信済みメッセージ一覧のソートが送受信日時の降順になっていない Fixed(完了) 2011-07-29

Associated revisions

Revision e12a68d0 (diff)
Added by Maki Takahashi almost 10 years ago

(fixes #2321 #2322) changed to update the column "CreatedAt" when sent from the draft

Revision b912555f (diff)
Added by Maki Takahashi almost 10 years ago

(fixes #2321 #2322) fixed for cording standard

History

#1 Updated by Maki Takahashi almost 10 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Maki Takahashi
  • Target version set to 1.0.0
  • 3.6 で発生するか set to Yes (はい)
  • 3.4 で発生するか set to Yes (はい)

#2 Updated by Maki Takahashi almost 10 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 e12a68d0d6d1e1b5f7baf888315bf6e71ef281e5 で適用されました。

#3 Updated by Yuya Watanabe almost 10 years ago

レビューしました.

本チケットの修正部分は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 

#4 Updated by Yuya Watanabe almost 10 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

#5 Updated by Maki Takahashi almost 10 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 b912555f902c09606fcd5fab3c2c3191b22941f6 で適用されました。

#6 Updated by Yuya Watanabe almost 10 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです.

#7 Updated by Yuma Sakata over 9 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

Also available in: Atom PDF