Bug(バグ) #1787
完了[仕様バグ] サイドバナーで切り替える言語設定が保持されない
100%
説明
概要¶
サイドバナーで切り替える言語の設定について、一度切り替えるとセッションが有効な間でしか設定が保持されない。
また、設定変更画面の「言語とタイムゾーンの設定」とも連携できていない。
切り替えた言語設定は保持されるようにし、また「言語とタイムゾーンの設定」とも連携できるようにする。
Maki Takahashi さんが13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/commit/feec441267b408114f7140f90c53e806ce60c08e
にて修正いたしました。
「言語とタイムゾーンの設定」画面にいる状態で、サイドバナーから言語設定を切り替えた際に、
「言語とタイムゾーンの設定」のフォームの内容も切り替わること
ログアウト後、セッションを削除した状態でもサイドバナーで切り替えた言語設定が保持されていることを確認いたしました。
Naoya Tozuka さんが13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
コーディング規約で「return 文の直前には可読性向上のために空行を入れるべきです」と規定されていますので、
http://www.openpne.jp/coding-standards-ja/
お手数ですが
lib/user/opSecurityUser.class.php
の修正箇所の return 文の前に空行を入れて頂けますでしょうか。
Maki Takahashi さんが13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
https://github.com/openpne/OpenPNE3/commit/780f88bc91d29988b4b8b5c8381aa80d1b540a1c
にて、フィードバックいただいた点を修正(return文前に改行追加)しました。
再度、動作確認を行い動作に問題が無いことを確認いたしました。
Naoya Tozuka さんが13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
確認しました。レビューOKです。
Maki Takahashi さんが13年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Rejected(差し戻し) に変更
- 進捗率 を 70 から 50 に変更
BPチケットが差し戻されたため、BP元チケットも差し戻します。
Maki Takahashi さんが13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
2efc2e77 にてBPチケットのテストで指摘された、
「設定変更画面」での変更内容がサイドバナーの言語変更フォームに反映されない点を修正いたしました。
指摘内容が解消されていることをPC版にて確認いたしました。
Kousuke Ebihara さんが約13年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更