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

Opened 9 years ago

Last modified 9 years ago

Re: v2.13.1 管理画面>受注管理にて受注数量の変更ができない

Reported by: kimoto Owned by: kimoto
Priority: Milestone: EC-CUBE2.13.2
Component: 管理画面 Version: 2.13.1
Keywords: Cc:
修正済み: yes

Description

・お届け先は1口 ・お届け先は、注文者自身ではなく、追加登録した住所 の場合、発生します。

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=14396&forum=9&post_id=64667#forumpost64667

不具合ですね。edit.tpl(v2.13.1の修正を加えていないソースコードです。)
    function quantityCopyForSingleShipping(product_index){
        var product_index = parseInt(product_index);
        var input_quantity = $('input[name^="quantity[' + product_index + ']"]').val();
        $('input[name^="shipment_quantity[0][' + product_index + ']"]').val(input_quantity);
    }
この部分の「shipment_quantity[0]」が「0」の決め打ちになっているのが原因ですね。追記:以外と修正は単純なようです。「0」-->「<!--{$top_shipping_id}-->」にすれば良いようです。つまり、edit.tplの修正案を以下のようにします。
    function quantityCopyForSingleShipping(product_index, shipment_product_index){
        var product_index = parseInt(product_index);
        var input_quantity = $('input[name^="quantity[' + product_index + ']"]').val();
        $('input[name^="shipment_quantity[<!--{$top_shipping_id}-->][' + shipment_product_index + ']"]').val(input_quantity);
    }
きれいではないですが、取り合えず。quantityCopyForSingleShippingを呼ぶ側で、<!--{$top_shipping_id}-->を引数に追加し、quantityCopyForSingleShippingは、その引数を使用する方がきれいではありますが。

Change History

comment:1 Changed 9 years ago by kimoto

  • Owner changed from somebody to kimoto

comment:2 Changed 9 years ago by kimoto

  • 修正済み set

r23358 にて修正

お届け先は1口 ・お届け先は、注文者自身ではなく、追加登録した住所 の場合 受注商品の数量が変更できない問題です

comment:3 Changed 9 years ago by h_yoshimoto

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

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

Note: See TracTickets for help on using tickets.