プロジェクト

全般

プロフィール

Bug(バグ) #1777

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

Kiwa Sakai13年以上前に追加. 12年以上前に更新.

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

100%

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

説明

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


関連するチケット

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

関係しているリビジョン

リビジョン 43d0f68d (差分)
Maki Takahashiほぼ13年前に追加

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

リビジョン d53e1dd9 (差分)
Maki Takahashiほぼ13年前に追加

fixed for coding standard (refs #1777)

リビジョン 107034b7 (差分)
Maki Takahashiほぼ13年前に追加

fixed for cording standard (refs #1777)

converted double quotation to single quotation

履歴

#1 Kousuke Ebihara13年以上前に更新

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

#2 Kousuke Ebihara13年以上前に更新

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

#3 Kousuke Ebihara13年以上前に更新

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

#11 isao sanoほぼ13年前に更新

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

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

#12 Maki Takahashi12年以上前に更新

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

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