Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

管理機能>商品選択ポップアップ画面で規格2のonchangeに存在しないfnCheckStock()が使われている #25

Closed
clicktx opened this issue Jan 22, 2015 · 2 comments
Milestone

Comments

@clicktx
Copy link
Contributor

clicktx commented Jan 22, 2015

http://svn.ec-cube.net/open_trac/changeset/23448 で削除されていると思われるfnCheckStock()が templates/admin/order/product_select.tpl で呼び出されている。

<dd>
    <select name="classcategory_id2" id="<!--{$class2}-->" style="<!--{$arrErr[$class2]|sfGetErrorColor}-->" onchange="fnCheckStock(this.form);"></select>
    <!--{if $arrErr[$class2] != ""}-->
    <br /><span class="attention">※ <!--{$tpl_class_name2[$id]}-->を入力して下さい。</span>
    <!--{/if}-->
</dd>
@clicktx
Copy link
Contributor Author

clicktx commented Jan 22, 2015

そもそもeccube.jsでchangeイベントとして書かれているので、規格1にある onchange="fnSetClassCategories(this.form);"も不要っぽい。

ただし、product_select.tpl 内にある function fnSetClassCategories()自体は必要。ページ描写時に規格2を設定している。

@ryo-endo ryo-endo added this to the 2.17.0 milestone Oct 26, 2017
@so-amuamu so-amuamu added bug:Middle and removed bug labels Jul 31, 2018
@so-amuamu
Copy link

本件、残っているので直す

kiy0taka added a commit that referenced this issue Aug 2, 2018
fix #25 不要なイベントを削除
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants