プロジェクト

全般

プロフィール

Bug(バグ) #2461

携帯版プロフィール画面で「自己紹介」の項目名が表示されない

Kiwa Sakai12年以上前に追加. 8年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-10-03
期日:
進捗率:

100%

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

説明

Overview (現象)

携帯版プロフィール画面 (/member/profile)にて、自己紹介の項目名が表示されない。


「よ ろ し く ね」という内容が表示されている項目が自己紹介。

現象確認バージョン

OpenPNE 3.6RC2
OpenPNE 3.7.0-dev (master)

Causes (原因)

下記部分においてフォームのタイプがtextareaかつpresetだった場合にcaptionの追加がなされていない.この条件に該当するものが「自己紹介」だけとなるため「自己紹介」のみが表示されない状態となっている.

apps/mobile_frontend/modules/member/templates/profileSuccess.php 59行目〜

 59   if ('textarea' === $profile->getFormType())
 60   {
 61     $value = op_auto_link_text_for_mobile((string)$profile);
 62   } 
 63   elseif ($profile->getProfile()->isPreset())
 64   { 
 65     $presetConfig = $profile->getProfile()->getPresetConfig();
 66     $caption = __($presetConfig['Caption']);
 67     if ('country_select' === $profile->getFormType())
 68     { 
 69       $value = __($culture->getCountry((string)$profile));
 70     } 
 71     elseif ('op_preset_birthday' === $profile->getName())
 72     { 
 73       $value = op_format_date((string)$profile, 'XShortDateJa');
 74     } 
 75     else
 76     {
 77       $value = __((string)$profile);
 78     }
 79   } 

Way to fix (修正内容)

diff --git a/apps/mobile_frontend/modules/member/templates/profileSuccess.php b/apps/mobile_frontend/modules/member/templates/profileSuccess.php
index 5f0d9c6..b018ffb 100644
--- a/apps/mobile_frontend/modules/member/templates/profileSuccess.php
+++ b/apps/mobile_frontend/modules/member/templates/profileSuccess.php
@@ -56,11 +56,7 @@ foreach ($member->getProfiles(true) as $profile)
     continue;
   }

-  if ('textarea' === $profile->getFormType())
-  {
-    $value = op_auto_link_text_for_mobile((string)$profile);
-  }
-  elseif ($profile->getProfile()->isPreset())
+  if ($profile->getProfile()->isPreset())
   {
     $presetConfig = $profile->getProfile()->getPresetConfig();
     $caption = __($presetConfig['Caption']);
@@ -78,6 +74,11 @@ foreach ($member->getProfiles(true) as $profile)
     }
   }

+  if ('textarea' === $profile->getFormType())
+  {
+    $value = op_auto_link_text_for_mobile($value);
+  }
+
   if ($member->getId() == $sf_user->getMemberId())
   {
     if ($profile->getPublicFlag() == ProfileTable::PUBLIC_FLAG_FRIEND)

mobile_frontend-member-profile.png 表示 (5.16 KB) Kiwa Sakai, 2011-10-03 18:01


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #3099: 携帯版プロフィール画面で「自己紹介」の項目名が表示されない Fixed(完了) 2011-10-03
関連している OpenPNE 3 - Backport(バックポート) #3214: 携帯版プロフィール画面で「自己紹介」の項目名が表示されない Fixed(完了) 2011-10-03

関係しているリビジョン

リビジョン d5d34afc (差分)
Yuya Watanabe12年以上前に追加

(fixes #2461) fixed to display self introduction item

履歴

#1 Yuya Watanabe12年以上前に更新

  • 説明 を更新 (diff)
  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Yuya Watanabe にセット
  • 対象バージョンOpenPNE 3.7.0 にセット

原因

下記部分においてフォームのタイプがtextareaかつpresetだった場合にcaptionの追加がなされていない.この条件に該当するものが「自己紹介」だけとなるため「自己紹介」のみが表示されない状態となっている.

apps/mobile_frontend/modules/member/templates/profileSuccess.php 59行目〜

 59   if ('textarea' === $profile->getFormType())
 60   {
 61     $value = op_auto_link_text_for_mobile((string)$profile);
 62   } 
 63   elseif ($profile->getProfile()->isPreset())
 64   { 
 65     $presetConfig = $profile->getProfile()->getPresetConfig();
 66     $caption = __($presetConfig['Caption']);
 67     if ('country_select' === $profile->getFormType())
 68     { 
 69       $value = __($culture->getCountry((string)$profile));
 70     } 
 71     elseif ('op_preset_birthday' === $profile->getName())
 72     { 
 73       $value = op_format_date((string)$profile, 'XShortDateJa');
 74     } 
 75     else
 76     {
 77       $value = __((string)$profile);
 78     }
 79   } 

問題が発生した経緯

#1160の修正コミットとなる 1e9b885468 によりtextareaだった場合にpresetかつtextareaだった場合を考慮されなかった.

修正方針

textareaかつpresetである場合にもcaptionを表示するように修正する.

#2 wa ta12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#3 Yuya Watanabe12年以上前に更新

  • 説明 を更新 (diff)

#4 Kousuke Ebihara12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#5 Yuma Sakata12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

#6 kaoru n8年以上前に更新

  • 3.6 で発生するかUnknown (未調査) にセット
  • 3.8 で発生するかUnknown (未調査) にセット

他の形式にエクスポート: Atom PDF