Bug(バグ) #1787
[仕様バグ] サイドバナーで切り替える言語設定が保持されない
100%
説明
概要¶
サイドバナーで切り替える言語の設定について、一度切り替えるとセッションが有効な間でしか設定が保持されない。
また、設定変更画面の「言語とタイムゾーンの設定」とも連携できていない。
切り替えた言語設定は保持されるようにし、また「言語とタイムゾーンの設定」とも連携できるようにする。
関連するチケット
関係しているリビジョン
fixed the language setting in the side banner save both user session and memberConfig (refs #1787)
fixed for coding standard (refs #1787)
(refs #1787) fixed the language settin in member config save both memberConfig and user session
履歴
#2 Kousuke Ebihara が13年以上前に更新
- 対象バージョン を OpenPNE 3.7.0 にセット
#3 Kousuke Ebihara が13年以上前に更新
- 優先度 を 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 Yamada が12年以上前に更新
- 360対象 を beta13 にセット
#13 Maki Takahashi が12年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Rejected(差し戻し) に変更
- 進捗率 を 70 から 50 に変更
BPチケットが差し戻されたため、BP元チケットも差し戻します。
#14 Maki Takahashi が12年以上前に更新
- ステータス を Rejected(差し戻し) から Accepted(着手) に変更
#15 Maki Takahashi が12年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
2efc2e77 にてBPチケットのテストで指摘された、
「設定変更画面」での変更内容がサイドバナーの言語変更フォームに反映されない点を修正いたしました。
指摘内容が解消されていることをPC版にて確認いたしました。
#16 Kousuke Ebihara が12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更