プロジェクト

全般

プロフィール

Bug(バグ) #1787

[仕様バグ] サイドバナーで切り替える言語設定が保持されない

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

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

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

説明

概要

サイドバナーで切り替える言語の設定について、一度切り替えるとセッションが有効な間でしか設定が保持されない。
また、設定変更画面の「言語とタイムゾーンの設定」とも連携できていない。

切り替えた言語設定は保持されるようにし、また「言語とタイムゾーンの設定」とも連携できるようにする。


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #2185: [仕様バグ] サイドバナーで切り替える言語設定が保持されない Fixed(完了) 2011-06-10
関連している OpenPNE 3 - Bug(バグ) #1453: 文言設定で指定の言語がない場合Fatalエラーとなる Fixed(完了) 2010-07-29

関係しているリビジョン

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

fixed the language setting in the side banner save both user session and memberConfig (refs #1787)

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

fixed for coding standard (refs #1787)

リビジョン 856bb28d (差分)
Maki Takahashi12年以上前に追加

fixed the language setting in the side banner save both user session and memberConfig (refs #1787, BP from #2185)

リビジョン a2b60589 (差分)
Maki Takahashi12年以上前に追加

fixed for coding standard (refs #1787, BP from #2185)

リビジョン 2efc2e77 (差分)
Maki Takahashi12年以上前に追加

(refs #1787) fixed the language settin in member config save both memberConfig and user session

履歴

#2 Kousuke Ebihara13年以上前に更新

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

#3 Kousuke Ebihara13年以上前に更新

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

#4 Shingo Yamadaほぼ13年前に更新

  • 担当者Shingo Yamada にセット

#5 Shingo Yamadaほぼ13年前に更新

  • 担当者Shingo Yamada から Maki Takahashi に変更

#6 Maki Takahashiほぼ13年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#7 Maki Takahashiほぼ13年前に更新

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

https://github.com/openpne/OpenPNE3/commit/feec441267b408114f7140f90c53e806ce60c08e
にて修正いたしました。

「言語とタイムゾーンの設定」画面にいる状態で、サイドバナーから言語設定を切り替えた際に、
「言語とタイムゾーンの設定」のフォームの内容も切り替わること
ログアウト後、セッションを削除した状態でもサイドバナーで切り替えた言語設定が保持されていることを確認いたしました。

#8 Naoya Tozukaほぼ13年前に更新

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

コーディング規約で「return 文の直前には可読性向上のために空行を入れるべきです」と規定されていますので、
http://www.openpne.jp/coding-standards-ja/
お手数ですが

lib/user/opSecurityUser.class.php

の修正箇所の return 文の前に空行を入れて頂けますでしょうか。

#9 Maki Takahashiほぼ13年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

#10 Maki Takahashiほぼ13年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

https://github.com/openpne/OpenPNE3/commit/780f88bc91d29988b4b8b5c8381aa80d1b540a1c
にて、フィードバックいただいた点を修正(return文前に改行追加)しました。

再度、動作確認を行い動作に問題が無いことを確認いたしました。

#11 Naoya Tozukaほぼ13年前に更新

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

確認しました。レビューOKです。

#12 Shingo Yamada12年以上前に更新

  • 360対象beta13 にセット

#13 Maki Takahashi12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

BPチケットが差し戻されたため、BP元チケットも差し戻します。

#14 Maki Takahashi12年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

#15 Maki Takahashi12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

2efc2e77 にてBPチケットのテストで指摘された、
「設定変更画面」での変更内容がサイドバナーの言語変更フォームに反映されない点を修正いたしました。

指摘内容が解消されていることをPC版にて確認いたしました。

#16 Kousuke Ebihara12年以上前に更新

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

#17 isao sano12年以上前に更新

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

テスト完了しました。
問題ありませんでしたので、ステータスをFixedに致します。

#18 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

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