Bug(バグ) #1777
完了メッセージをゴミ箱から削除するときに確認画面がない [仕様バグ]
100%
説明
メッセージをゴミ箱から削除する動作に、削除確認画面がはさまれない。
Maki Takahashi さんが13年以上前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Maki Takahashi にセット
Maki Takahashi さんが13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
https://github.com/martini2002jp/opMessagePlugin/commit/43d0f68d1c320672a20870b159fdcd01a96fcd20
にて修正いたしました。
PC、モバイル版において、受信/送信/下書きについては一覧表示/詳細表示から削除する際に
(下書きのみ詳細表示から削除機能がありません)確認画面が表示されず、ゴミ箱に移動することを確認しました。
ゴミ箱からの削除については一覧表示/詳細表示ともに確認画面が表示されること
確認画面から元の画面(一覧、詳細)に戻れること
確認画面から削除を選択した場合に、正しく削除がなされていることを確認しました。
Naoya Tozuka さんが13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
修正点確認しました。
lib/action/opMessagePluginMessageActions.class.php :
コーディング規約に関する指摘になりますが、変数と値を比較する際は最初に値を置き、場合によっては型チェックも行ってください。
http://www.openpne.jp/coding-standards-ja/
Maki Takahashi さんが13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
https://github.com/martini2002jp/opMessagePlugin/commit/d53e1dd9b1d9c030242a86fc73bc7e81c206c89a
にてフィードバックいただいた点を修正しました。
再度、動作確認を行い期待される動作をしていることを確認しました。
Naoya Tozuka さんが13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
あと一点すみません。文字列リテラルをシングルクオートで囲んで頂けるとコーディング規約に準拠した形になります。
http://www.openpne.jp/coding-standards-ja/
Maki Takahashi さんが13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
https://github.com/martini2002jp/opMessagePlugin/commit/107034b7854072c899231f71b18574656e300de7
にて、ダブルクオートをシングルクオートに変更しました。
Naoya Tozuka さんが13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
お手数おかけしました。レビューOKです。