Project

General

Profile

Bug(バグ) #1777

メッセージをゴミ箱から削除するときに確認画面がない [仕様バグ]

Added by Kiwa Sakai about 9 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2010-11-10
Due date:
% Done:

100%

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

Description

メッセージをゴミ箱から削除する動作に、削除確認画面がはさまれない。


Related issues

Related to opMessagePlugin - Backport(バックポート) #2105: メッセージをゴミ箱から削除するときに確認画面がない [仕様バグ] Fixed(完了) 2011-05-17

Associated revisions

Revision 43d0f68d (diff)
Added by Maki Takahashi over 8 years ago

added confirmation pages for delete actions (dust box) (refs #1777)

Revision d53e1dd9 (diff)
Added by Maki Takahashi over 8 years ago

fixed for coding standard (refs #1777)

Revision 107034b7 (diff)
Added by Maki Takahashi over 8 years ago

fixed for cording standard (refs #1777)

converted double quotation to single quotation

History

#1 Updated by Kousuke Ebihara about 9 years ago

  • Priority changed from Normal(通常) to High(高め)

#2 Updated by Kousuke Ebihara about 9 years ago

  • [QA]バグ通知済 set to No

#3 Updated by Kousuke Ebihara about 9 years ago

  • [QA]バグ通知済 changed from No to Yes

#4 Updated by Maki Takahashi over 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Maki Takahashi

#5 Updated by Maki Takahashi over 8 years ago

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

https://github.com/martini2002jp/opMessagePlugin/commit/43d0f68d1c320672a20870b159fdcd01a96fcd20
にて修正いたしました。

PC、モバイル版において、受信/送信/下書きについては一覧表示/詳細表示から削除する際に
(下書きのみ詳細表示から削除機能がありません)確認画面が表示されず、ゴミ箱に移動することを確認しました。
ゴミ箱からの削除については一覧表示/詳細表示ともに確認画面が表示されること
確認画面から元の画面(一覧、詳細)に戻れること
確認画面から削除を選択した場合に、正しく削除がなされていることを確認しました。

#6 Updated by Naoya Tozuka over 8 years ago

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

修正点確認しました。

lib/action/opMessagePluginMessageActions.class.php :
コーディング規約に関する指摘になりますが、変数と値を比較する際は最初に値を置き、場合によっては型チェックも行ってください。
http://www.openpne.jp/coding-standards-ja/

#7 Updated by Maki Takahashi over 8 years ago

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

https://github.com/martini2002jp/opMessagePlugin/commit/d53e1dd9b1d9c030242a86fc73bc7e81c206c89a
にてフィードバックいただいた点を修正しました。

再度、動作確認を行い期待される動作をしていることを確認しました。

#8 Updated by Naoya Tozuka over 8 years ago

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

あと一点すみません。文字列リテラルをシングルクオートで囲んで頂けるとコーディング規約に準拠した形になります。
http://www.openpne.jp/coding-standards-ja/

#9 Updated by Maki Takahashi over 8 years ago

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

https://github.com/martini2002jp/opMessagePlugin/commit/107034b7854072c899231f71b18574656e300de7
にて、ダブルクオートをシングルクオートに変更しました。

#10 Updated by Naoya Tozuka over 8 years ago

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

お手数おかけしました。レビューOKです。

#11 Updated by isao sano over 8 years ago

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

テスト完了いたしました。
問題ありませんでしたので、Fixedいたします。

#12 Updated by Maki Takahashi over 8 years ago

  • Target version set to 0.9.2

Also available in: Atom PDF