Project

General

Profile

Bug(バグ) #4264

「全員に公開」と「公開しない」が翻訳されない

Added by Chiharu Nakajima almost 2 years ago. Updated 27 days ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2019-07-16
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

Description

「全員に公開」と「公開しない」が日本語に翻訳されません

#4143 の修正による影響

t4264_01.png View (53.3 KB) kaoru n, 2019-07-16 17:44

t4264_02.png View (50.2 KB) kaoru n, 2019-07-16 17:44


Subtasks

Backport(バックポート) #4315: 「全員に公開」と「公開しない」が翻訳されないFixed(完了)kaoru n

Backport(バックポート) #4316: 「全員に公開」と「公開しない」が翻訳されないFixed(完了)kaoru n

History

#1 Updated by Chiharu Nakajima about 1 year ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE 3.9.x

#2 Updated by kaoru n 5 months ago

  • Description updated (diff)
  • Assignee set to kaoru n

#3 Updated by kaoru n 5 months ago

#4 Updated by kaoru n 5 months ago

  • File deleted (翻訳不備.jpg)

#5 Updated by kaoru n 5 months ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/524
にてプルリクエストを行いました。

#6 Updated by kaoru n 2 months ago

  • Target version changed from OpenPNE 3.9.x to OpenPNE-3.9.6

#7 Updated by Rimpei Ogawa 2 months ago

メモ

"publicFlags" の翻訳ファイルが pc_frontend と mobile_frontend にしかないため、
pc_backend や api から MemberConfigForm が呼ばれると意図しない結果になるのではないかと思います。
(エラーにならず、翻訳されないだけなら今までと挙動が変わらないので問題ないかもしれません)

例えば、
apps/pc_backend/modules/member/lib/ReissuePasswordForm.class.php
などは MemberConfigForm を継承したものになっています。
'age_public_flag', 'profile_page_public_flag' は使われていないので実害はないかもしれません。

#8 Updated by kaoru n 2 months ago

publicFlags.ja.xml が存在しないことでエラーは発生しませんが、念の為に api, pc_backend に publicFlags.ja.xml を追加しておいたほうが良いでしょうか?

#9 Updated by Rimpei Ogawa 2 months ago

publicFlags.ja.xml が存在することに害はないと思いますので、念の為に追加しておく方針にしましょう。

#11 Updated by Rimpei Ogawa 2 months ago

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

レビューOKです。

#12 Updated by isao sano 27 days ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

試験完了しました。問題ありません。

#13 Updated by kaoru n 27 days ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF