プロジェクト

全般

プロフィール

Bug(バグ) #1777

完了

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

Kiwa Sakai さんが約14年前に追加. 約13年前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2010-11-10
期日:
進捗率:

100%

予定工数:
3.6 で発生するか:
Yes
[QA]バグ通知済:
はい
3.8 で発生するか:

説明

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


関連するチケット 1 (0件未完了1件完了)

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

操作

Kousuke Ebihara さんがほぼ14年前に更新

  • 優先度Normal(通常) から High(高め) に変更

Kousuke Ebihara さんがほぼ14年前に更新

  • [QA]バグ通知済いいえ にセット

Kousuke Ebihara さんがほぼ14年前に更新

  • [QA]バグ通知済いいえ から はい に変更

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です。

isao sano さんが13年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

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

Maki Takahashi さんが約13年前に更新

  • 対象バージョン0.9.2 にセット

他の形式にエクスポート: Atom PDF