Project

General

Profile

Bug(バグ) #1787

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

Added by Kiwa Sakai almost 9 years ago. Updated almost 4 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2010-11-16
Due date:
% Done:

100%

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

Description

概要

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

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


Related issues

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

Associated revisions

Revision feec4412 (diff)
Added by Maki Takahashi about 8 years ago

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

Revision 780f88bc (diff)
Added by Maki Takahashi about 8 years ago

fixed for coding standard (refs #1787)

Revision 856bb28d (diff)
Added by Maki Takahashi about 8 years ago

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

Revision a2b60589 (diff)
Added by Maki Takahashi about 8 years ago

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

Revision 2efc2e77 (diff)
Added by Maki Takahashi about 8 years ago

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

History

#2 Updated by Kousuke Ebihara almost 9 years ago

  • Target version set to OpenPNE 3.7.0

#3 Updated by Kousuke Ebihara over 8 years ago

  • Priority changed from Normal(通常) to High(高め)

#4 Updated by Shingo Yamada over 8 years ago

  • Assignee set to Shingo Yamada

#5 Updated by Shingo Yamada about 8 years ago

  • Assignee changed from Shingo Yamada to Maki Takahashi

#6 Updated by Maki Takahashi about 8 years ago

  • Status changed from New(新規) to Accepted(着手)

#7 Updated by Maki Takahashi about 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

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

#8 Updated by Naoya Tozuka about 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

lib/user/opSecurityUser.class.php

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

#9 Updated by Maki Takahashi about 8 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

#10 Updated by Maki Takahashi about 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)

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

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

#11 Updated by Naoya Tozuka about 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

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

#12 Updated by Shingo Yamada about 8 years ago

  • 360対象 set to beta13

#13 Updated by Maki Takahashi about 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

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

#14 Updated by Maki Takahashi about 8 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

#15 Updated by Maki Takahashi about 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)

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

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

#16 Updated by Kousuke Ebihara about 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#17 Updated by isao sano almost 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

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

#18 Updated by kaoru n almost 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF