ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 11 years ago

Closed 11 years ago

#3082 closed task (fixed)

携帯版のみを利用していたメンバーが iPhone へ機種変更した場合、 SNS を利用できなくなってしまうバグ修正を取り込む

Reported by: masabon Owned by: masabon
Priority: minor Milestone: OpenPNE2.12.6
Component: 指定しない Version: 2.10.x & 2.12.x
Keywords: OpenPNE2.10.12 Cc:

Description

Ticket #2991の取り込み

Attachments (2)

20081121_o_regist_pc_address.png (20.6 KB) - added by kiwa 11 years ago.
20081121_o_regist_pc_address_end.png (21.7 KB) - added by kiwa 11 years ago.

Download all attachments as: .zip

Change History (25)

comment:1 Changed 11 years ago by masabon

Owner: changed from nobody to masabon
Status: newassigned

comment:2 Changed 11 years ago by masabon

Keywords: OpenPNE2.10.12 確認中 added; OpenPNE2.10.12 removed

以下のリビジョンで取り込みました。

確認お願いします。

comment:3 Changed 11 years ago by shingo

◆2.12について ・・・以下の点について再確認してください。
※2.13の r8448 がうまくとりこめて無いような気がします。

  • public_html/modules/portal/skin/005_openpne_blue/skin.css
  • public_html/modules/portal/skin/002_openpne_beige/skin.css
  • public_html/modules/portal/skin/003_openpne_yellow/skin.css
  • public_html/modules/portal/skin/007_openpne_purple/skin.css
  • public_html/modules/portal/skin/006_openpne_navy/skin.css
  • public_html/modules/portal/skin/004_openpne_green/skin.css
  • public_html/modules/portal/skin/001_openpne_red/skin.css

(trunk)

-.snsLogin div#buttonLoginArea { top:170px;}
-.snsLogin div#buttonRegiArea { top:202px;}

(今回の修正)

+.snsLogin div#buttonLoginArea { top:187px;}
+.snsLogin div#buttonRegiArea { top:217px;}
  • public_html/modules/portal/css/master.css
    -#invite div.snsLogin div#loginSecurity { top:209px;}
    -#free div.snsLogin div#loginSecurity { top:239px;}
    -.snsLogin div#buttonLoginArea { top:179px;}
    -.snsLogin div#buttonRegiArea { top:209px;}
    +.snsLogin div#pcAddressRegistArea { top:164px; width:200px; text-align:center;}
    +.snsLogin div#loginSecurity { left:0; width:268px; height:9px; background:url("../img/login_sec_black.gif") no-repeat center center; font-size:0; line-height:0; text-indent:-3000px;}
    +#invite div.snsLogin div#loginSecurity { top:227px;}
    +#free div.snsLogin div#loginSecurity { top:257px;}
    +.snsLogin div#buttonLoginArea { top:187px;}
    +.snsLogin div#buttonRegiArea { top:217px;}
    
    +.snsLogin div#pcAddressRegistArea a,
    
  • webapp/modules/portal/templates/inc_portal_Login_Form.tpl 以下の部分不要
    +           <div id="pcAddressRegistArea"><a href="({t_url m=pc a=page_o_regist_pc_address})">&gt;PCメールアドレス登録</a></div>
    
  • webapp/modules/pc/templates/o_login.tpl 以下の部分不要
     +<br /><a href="({t_url m=pc a=page_o_regist_pc_address})">PCメールアドレス登録</a>
    
  • webapp/modules/pc/templates/o_regist_pc_address.tpl
    ({if $smarty.const.OPENPNE_USE_CAPTCHA})のif文がない
    

comment:4 Changed 11 years ago by shingo

Keywords: OpenPNE2.10.12 差し戻し added; OpenPNE2.10.12 確認中 removed

comment:5 Changed 11 years ago by masabon

Keywords: OpenPNE2.10.12 確認中 added; OpenPNE2.10.12 差し戻し removed

以下のリビジョンで修正しました。 上記取り込み後、下記の取り込みを行うと正しい取り込みになります。

確認お願いします。

comment:6 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.12 テスト待ち added; OpenPNE2.10.12 確認中 removed

Changed 11 years ago by kiwa

Changed 11 years ago by kiwa

comment:7 Changed 11 years ago by kiwa

Keywords: 差し戻し added; テスト待ち removed

2.10のUIに対する指摘です。

o_regist_pc_address

あまり優先度は高くないですが、「※メールアドレスは公開されません。」の上下の空白は余計ではないでしょうか。

o_regist_pc_address_end

こちらもあまり優先度は高くないですが、行間や文章の上下の空白が広すぎる気がします。


動作については問題ありませんでした。

comment:8 Changed 11 years ago by ebihara

Owner: changed from masabon to ebihara
Status: assignednew

comment:7 の修正に関しては海老原が引き受けます。

comment:9 Changed 11 years ago by ebihara

Keywords: 確認中 added; 差し戻し removed

comment:7 について以下のリビジョンで修正しました。ご確認ください。

comment:10 Changed 11 years ago by ebihara

Owner: changed from ebihara to masabon

comment:11 Changed 11 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

2.10の以下のファイルにXHTMLの書式が混じっています。

webapp/modules/pc/templates/fh_diary.tpl
790:<img class="v_spacer_l" alt="dummy" src="./skin/dummy.gif"/>

webapp/modules/pc/templates/o_help_login_error.tpl
44:<img class="v_spacer_l" alt="dummy" src="./skin/dummy.gif"/>
68:<p>携帯から登録された方がPC版を利用するにはPCメールアドレスの登録が必要です。<br />
89:<img class="v_spacer_l" alt="dummy" src="./skin/dummy.gif"/>

webapp/modules/pc/templates/o_regist_pc_address.tpl
95:<input type="text" class="input_text" name="ktai_address" value="({$requests.ktai_address})" size="40" />
123:<input type="password" class="input_password" name="password" value="" size="20" />
153:<input type="text" class="input_text" name="pc_address" value="({$requests.pc_address})" size="40" />
184:<input type="text" class="input_text" name="pc_address2" value="" size="40" />
222:<input type="text" class="input_text" name="captcha" value="" size="30" />
243:<input type="submit" class="input_submit" value=" 送 信 " />
267:<img class="v_spacer_l" alt="dummy" src="./skin/dummy.gif"/>

comment:12 Changed 11 years ago by ebihara

Keywords: 確認中 added; 差し戻し removed

r9342comment:11 の指摘事項について修正しました。

comment:13 Changed 11 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

「class="input_text"」「class="input_password"」は、2.12以降のclassです。2.10では「class="text"」が使われているようです。

webapp/modules/pc/templates/o_regist_pc_address.tpl
95:<input type="text" class="input_text" name="ktai_address" value="({$requests.ktai_address})" size="40">
123:<input type="password" class="input_password" name="password" value="" size="20">
153:<input type="text" class="input_text" name="pc_address" value="({$requests.pc_address})" size="40">
184:<input type="text" class="input_text" name="pc_address2" value="" size="40">
222:<input type="text" class="input_text" name="captcha" value="" size="30">
243:<input type="submit" class="input_submit" value=" 送 信 ">

comment:14 Changed 11 years ago by ebihara

Keywords: 確認中 added; 差し戻し removed

comment:13の指摘事項について r9349 で修正しました。

comment:15 Changed 11 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

2.10では、「type="password"」の場合、「class="password"」ではなく「class="text"」が使われています。

comment:16 Changed 11 years ago by ogawa

Keywords: 確認中 added; 差し戻し removed

comment:15 について、r9359 で修正しました。

comment:17 Changed 11 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

comment:18 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: newclosed

確認しました。問題ありません。

comment:19 Changed 11 years ago by kiwa

Keywords: 差し戻し added
Resolution: fixed
Status: closedreopened

config.php.sampleの追加行について、カンマの後ろにスペースを入れてください。

'page_o_regist_pc_address','do_o_regist_pc_address','page_o_regist_pc_address_end', 

comment:20 Changed 11 years ago by ogawa

Keywords: 確認中 added; 差し戻し removed

comment:19 について、以下のリビジョンで config.php.sample を修正しました。

comment:21 Changed 11 years ago by ogawa

portal をSSL対象に入れるとレビュー画像表示で警告が表示されるため、除外します。

comment:22 Changed 11 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:23 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: reopenedclosed

確認しました。問題ありません。

Note: See TracTickets for help on using tickets.