Project

General

Profile

Bug(バグ) #2817

コミュニティホーム画面でコミュニティ書き込み通知メールのフォームの「Save」が翻訳されない

Added by Yuya Watanabe over 9 years ago. Updated 6 months ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2012-02-14
Due date:
% Done:

100%

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

Description

概要

プラグインとして opCommunityTopic のみをインストールした状態で,コミュニティホーム画面( /community/コミュニティID )でコミュニティ書き込み通知メールのフォームが翻訳されず,「Save」と表示される.

他のプラグインが入っている状態では「確定」と表示される.

確認バージョン

OpenPNE 3.6.2 + opCommunityTopicPlugin 1.0.3

備考

opCommunityTopicPlugin 0.9.9 では #1368 「コミュニティ書き込み通知メールに対応」が実装されていないためフォームが表示されず,問題は発生していない.

参考画像

!https://redmine.openpne.jp/attachments/413/スクリーンショット%20.png!

スクリーンショット .png View (22.7 KB) Yuya Watanabe, 2012-03-26 17:01


Subtasks

Backport(バックポート) #4397: コミュニティホーム画面でコミュニティ書き込み通知メールのフォームの「Save」が翻訳されないFixed(完了)kaoru n


Related issues

Related to OpenPNE 3 - Backport(バックポート) #2923: コミュニティホーム画面でコミュニティ書き込み通知メールのフォームの「Save」が翻訳されない Fixed(完了) 2012-02-14
Related to OpenPNE 3 - Backport(バックポート) #2924: コミュニティホーム画面でコミュニティ書き込み通知メールのフォームの「Save」が翻訳されない Fixed(完了) 2012-02-14
Copied to OpenPNE 3 - Backport(バックポート) #3644: コミュニティホーム画面でコミュニティ書き込み通知メールのフォームの「Save」が翻訳されない Invalid(無効) 2012-02-14

Associated revisions

Revision cdaf394f (diff)
Added by Yuya Watanabe about 8 years ago

add dictionary item to translate 'Save' into Japanese in pc_frontend (fixes #2817)

History

#1 Updated by Yuya Watanabe over 9 years ago

  • Description updated (diff)

#2 Updated by Kousuke Ebihara over 9 years ago

  • Target version changed from 1.0.x to 1.0.4

#3 Updated by Yuya Watanabe about 9 years ago

#4 Updated by Yuya Watanabe about 9 years ago

  • Description updated (diff)

#5 Updated by Yuya Watanabe about 9 years ago

  • Description updated (diff)

#6 Updated by Yuya Watanabe about 9 years ago

  • Subject changed from コミュニティホーム画面でコミュニティ書き込み通知メールのフォームの「保存」が翻訳されない to コミュニティホーム画面でコミュニティ書き込み通知メールのフォームの「Save」が翻訳されない

#7 Updated by Yuya Watanabe about 9 years ago

携帯版では「確定」と表示されることを確認しました.

このコード部は lib/form/opConfigCommunityTopicNotificationMailForm.class.php で定義され,apps/mobile_frontend/modules/community/config/view.yml で表示されるように決定されています.これはコア側の機能を利用していると見なせるため,本体側の問題であると思われます.

これは opCommunityTopicPlugin 側には含まれていません.下記が opCommunityTopicPlugin のディレクトリ直下で検索を行った結果です.

 $ ack 確定
apps/pc_backend/i18n/messages.ja.xml
215:        <target>確定</target>

また, OpenPNE ディレクトリ直下で検索を行った結果は以下のとおりです.これを見ると携帯版のみ定義されていることがわかります.

 $ ack 確定
apps/pc_backend/modules/design/templates/mobileColorConfigSuccess.php
38:<p class="caution" id="c01">※プリセットカラーの呼出し後、かならず「色変更を確定する」を押し配色の設定を確定してください。</p>
61:<p class="caution" id="c03">※色のプレビューが変更されてもそのままでは色設定は反映されません。必ず確定してください。</p>
76:<p class="textBtn" id="c04"><input type="submit" value="色変更を確定する" /></p>

apps/pc_backend/i18n/messages.ja.xml
272:        <target>設定を確定する</target>

apps/mobile_frontend/i18n/messages.ja.xml
47:        <target>確定</target>

#8 Updated by Yuya Watanabe about 9 years ago

明確な原因は調査しきれていませんが,少なくとも opCommunityTopicPlugin の問題ではなさそうなのでプロジェクトを 一旦コア側に移動させます.

#9 Updated by Yuya Watanabe about 9 years ago

  • Project changed from opCommunityTopicPlugin to OpenPNE 3
  • Target version deleted (1.0.4)

#10 Updated by Yuma Sakata about 9 years ago

  • Target version set to OpenPNE 3.9.0-old

#11 Updated by pnetan   about 9 years ago

  • Target version deleted (OpenPNE 3.9.0-old)

#12 Updated by ichikawa tatsuya over 8 years ago

本体側i18nに「確定」の項目を追加

diff --git apps/pc_frontend/i18n/messages.ja.xml apps/pc_frontend/i18n/messages.ja.xml
index bd16057..a55c84a 100644
--- apps/pc_frontend/i18n/messages.ja.xml
+++ apps/pc_frontend/i18n/messages.ja.xml
@@ -1415,6 +1415,10 @@
         <source>Reset</source>
         <target>リセット</target>
       </trans-unit>
+      <trans-unit id="">
+        <source>Save</source>
+        <target>確定</target>
+      </trans-unit>
     </body>
   </file>
 </xliff>

#13 Updated by ichikawa tatsuya over 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to ichikawa tatsuya
  • 3.8 で発生するか set to Unknown (未調査)

#14 Updated by ichikawa tatsuya over 8 years ago

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

#15 Updated by Yuya Watanabe about 8 years ago

  • Target version set to OpenPNE 3.9.0-old

#16 Updated by Yuya Watanabe about 8 years ago

更新履歴 cdaf394f9034465bebc1359dfa50b8ab9c3cccda で適用されました。

#17 Updated by Yuya Watanabe about 8 years ago

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

#18 Updated by Shinichi Urabe almost 7 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.8.12

レビューが終わっているようなので今回のバージョンに含みます

#19 Updated by 誠二 天重 almost 7 years ago

#20 Updated by 誠二 天重 almost 7 years ago

  • Target version changed from OpenPNE 3.8.12 to OpenPNE 3.9.0-old

#22 Updated by isao sano about 4 years ago

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

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#23 Updated by isao sano about 4 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

#24 Updated by isao sano about 4 years ago

OpenPNE 3.9.0 + opCommunityTopicPlugin 1.0.3 だと再現します。
OpenPNE 3.9.0 + opCommunityTopicPlugin-1.1.2.1 では再現しません。

#25 Updated by kaoru n over 1 year ago

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

#26 Updated by kaoru n 10 months ago

  • Target version changed from OpenPNE 3.10.x to OpenPNE 3.11.x

#27 Updated by kaoru n 7 months ago

opCommunityTopicPlugin-1.0.x はすでにメンテナンスを行っていないので対象外とします

#28 Updated by kaoru n 7 months ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#2817-16 にて適用済みのコメントがありますが、ソース上にはないので同内容を追加します。

#29 Updated by kaoru n 7 months ago

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

https://github.com/openpne/OpenPNE3/pull/619
にてプルリクエストしました

#30 Updated by Rimpei Ogawa 7 months ago

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

#31 Updated by isao sano 6 months ago

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

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

#32 Updated by kaoru n 6 months ago

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

マージしました

Also available in: Atom PDF