プロジェクト

全般

プロフィール

Backport(バックポート) #1035

Displaying error messages in SNS configuration page is not clear (SNS 設定でのエラーメッセージの表示が明確ではない)

tu nguyen ngocほぼ14年前に追加. ほぼ14年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-01-15
期日:
進捗率:

100%


説明

Overview (現象)

SNS configuration page displays error message to specific field. But this has following problems (SNS 設定ページはエラーメッセージを特定のフィールドに出力する。しかし、これには以下の問題がある)

  • Error messages are black (エラーメッセージの色が黒で出力される)
  • No mentions about failed to save (保存に失敗していることに対する言及がない)

Way to fix (修正内容)

  • Error messages are now red (エラーメッセージを赤で表示するようにした)
  • "Failed to save." messages are now in above the page (保存に失敗した旨を知らせるエラーメッセージをページ上部に表示するようにした)

Original report (元の報告)

http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=5331 より転記

管理ログイン→SNS設定→一般設定で、
デイリーニュース受信頻度の設定のチェックをすべて解除して保存すると保存されてないのですがその時他に変更した設定も反映されないので設定を変えたつもりになってしまう可能性があります。

例)CAPTCHA 使用設定で使用するから使用しないに変更とデイリーニュースをすべてチェックはずす変更を同時にした場合にCAPTCHAの設定変更も保存されていない。

関連するチケット

関連している OpenPNE 3 - Bug(バグ) #502: Displaying error messages in SNS configuration page is not clear (SNS 設定でのエラーメッセージの表示が明確ではない) Fixed(完了) 2010-01-15
関連している OpenPNE 3 - Backport(バックポート) #1062: SNS設定でエラーの際、Flashセッションが次のページに維持される Fixed(完了) 2010-05-12

関係しているリビジョン

リビジョン dc3495fe (差分)
Shinichi Urabeほぼ14年前に追加

fixed sns/config doesn't error messages clearly (fixes #1035, BP from #502)

Conflicts:

apps/pc_backend/i18n/messages.ja.xml
web/css/backend.css

履歴

#1 Shinichi Urabeほぼ14年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 dc3495fedd65387fad5e45a1720069ded75abec7 で適用されました。

#2 Mutsumi Imamuraほぼ14年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

SNS設定ページでエラーメッセージが表示された状態で、他のページ(例:メール設定など)のリンクをクリックし遷移してもエラーメッセージが表示されたままになります。

#3 Mutsumi Imamuraほぼ14年前に更新

  • 担当者Shinichi Urabe にセット

#4 Shinichi Urabeほぼ14年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

Imamura Mutsumi は書きました:

SNS設定ページでエラーメッセージが表示された状態で、他のページ(例:メール設定など)のリンクをクリックし遷移してもエラーメッセージが表示されたままになります。

#1062 で対応しましょう

#5 Rimpei Ogawaほぼ14年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更

setFlash() の第3引数の $persist を false に設定するだけなので、可能であれば #1062 も修正してリリースした方がよいと思います。

flash の問題以外はOKです。

#6 Shinichi Urabeほぼ14年前に更新

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

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