Bug(バグ) #1777
メッセージをゴミ箱から削除するときに確認画面がない [仕様バグ]
100%
説明
メッセージをゴミ箱から削除する動作に、削除確認画面がはさまれない。
関連するチケット
関係しているリビジョン
added confirmation pages for delete actions (dust box) (refs #1777)
fixed for coding standard (refs #1777)
fixed for cording standard (refs #1777)
converted double quotation to single quotation
履歴
#1 Kousuke Ebihara が13年以上前に更新
- 優先度 を Normal(通常) から High(高め) に変更
#2 Kousuke Ebihara が13年以上前に更新
- [QA]バグ通知済 を いいえ にセット
#3 Kousuke Ebihara が13年以上前に更新
- [QA]バグ通知済 を いいえ から はい に変更
#4 Maki Takahashi がほぼ13年前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Maki Takahashi にセット
#5 Maki Takahashi がほぼ13年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
https://github.com/martini2002jp/opMessagePlugin/commit/43d0f68d1c320672a20870b159fdcd01a96fcd20
にて修正いたしました。
PC、モバイル版において、受信/送信/下書きについては一覧表示/詳細表示から削除する際に
(下書きのみ詳細表示から削除機能がありません)確認画面が表示されず、ゴミ箱に移動することを確認しました。
ゴミ箱からの削除については一覧表示/詳細表示ともに確認画面が表示されること
確認画面から元の画面(一覧、詳細)に戻れること
確認画面から削除を選択した場合に、正しく削除がなされていることを確認しました。
#6 Naoya Tozuka がほぼ13年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
修正点確認しました。
lib/action/opMessagePluginMessageActions.class.php :
コーディング規約に関する指摘になりますが、変数と値を比較する際は最初に値を置き、場合によっては型チェックも行ってください。
http://www.openpne.jp/coding-standards-ja/
#7 Maki Takahashi がほぼ13年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
https://github.com/martini2002jp/opMessagePlugin/commit/d53e1dd9b1d9c030242a86fc73bc7e81c206c89a
にてフィードバックいただいた点を修正しました。
再度、動作確認を行い期待される動作をしていることを確認しました。
#8 Naoya Tozuka がほぼ13年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
あと一点すみません。文字列リテラルをシングルクオートで囲んで頂けるとコーディング規約に準拠した形になります。
http://www.openpne.jp/coding-standards-ja/
#9 Maki Takahashi がほぼ13年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
https://github.com/martini2002jp/opMessagePlugin/commit/107034b7854072c899231f71b18574656e300de7
にて、ダブルクオートをシングルクオートに変更しました。
#10 Naoya Tozuka がほぼ13年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
お手数おかけしました。レビューOKです。
#12 Maki Takahashi が12年以上前に更新
- 対象バージョン を 0.9.2 にセット