Ticket #2158 (closed バグ指摘: 修正済)

Opened 11 years ago

Last modified 11 years ago

現在のカゴの「画像がありません」の商品写真を押すと空白ページにジャンプする

Reported by: shutta Owned by: shutta
Priority: Milestone: EC-CUBE2.13.0
Component: フロント Version: 2.12.3
Keywords: Cc:
修正済み: yes

Description

コミュニティにて、下記投稿あり。

現在のカゴの「画像がありません」の商品写真を押すと空白ページにジャンプする
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=12224&forum=8

EC-CUBEバージョン 2.12.3
PHPバージョン PHP 5.2.17
DBバージョン MySQL 5.5.28-log
WEBサーバー Apache/2.2.23

【再現方法】
1.商品を画像なしで登録する。
2.1の商品をフロントページで買い物カゴに入れる。
3.買い物カゴで、「画像がありません」という画像をクリック
4.真っ白い画面が表示される。

【原因】
「画像がありません」の判別を行う部分にて、hrefありのaタグが埋め込まれてしまっているため。

【修正箇所】
data/Smarty/templates/default/cart/index.tpl
128行目~133行目(あたり)



<a class="expansion" target="_blank"
<!--{if $item.productsClass.main_image|strlen >= 1}--> href="<!--{$smarty.const.IMAGE_SAVE_URLPATH}--><!--{$item.productsClass.main_image|sfNoImageMainList|h}-->"
<!--{/if}-->
>
<img src="<!--{$smarty.const.ROOT_URLPATH}-->resize_image.php?image=<!--{$item.productsClass.main_list_image|sfNoImageMainList|h}-->&width=65&height=65" alt="<!--{$item.productsClass.name|h}-->" />
</a>

修正例

<!--{if $item.productsClass.main_image|strlen >= 1}-->
<a class="expansion" target="_blank" href="<!--{$smarty.const.IMAGE_SAVE_URLPATH}--><!--{$item.productsClass.main_image|sfNoImageMainList|h}-->">
<!--{/if}-->
<img src="<!--{$smarty.const.ROOT_URLPATH}-->resize_image.php?image=<!--{$item.productsClass.main_list_image|sfNoImageMainList|h}-->&width=65&height=65" alt="<!--{$item.productsClass.name|h}-->" />
<!--{if $item.productsClass.main_image|strlen >= 1}-->
</a>
<!--{/if}-->


このように修正することで、修正されると思います。
時期修正にて標準でなおして頂けると助かります。

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=12224&forum=8&post_id=56310

ブラウザにより、挙動が異なる様です。

【Mac OS X 10.8】
Chrome バージョン 24.0.1312.57
Firefox 19.0

【Windows 8】
Chrome バージョン 24.01312.57

こちらでは、空白ページにジャンプしますが

【Windows 8
IE 10.0.9200.16484

こちらでは、バツマークがフローティングしている状態となりました。


どちらにせよ、フローティングにも、空白にもしたくないので、aタグで囲む必要がなければ、囲わない方が良い気がします

Change History

comment:1 Changed 11 years ago by shutta

  • Owner changed from somebody to shutta
  • 修正済み set
  • Status changed from new to assigned

r22579 にて修正。

comment:2 Changed 11 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済

ご対応ありがとうございました。

クローズさせて頂きます。

Note: See TracTickets for help on using tickets.