バージョン選択

フォーラム

 どうか教えて下さい！  バグ見つけましたよ！  こんな機能あったらなぁ  プラグインについて  その他

メニュー

 当サイトについて  初めてご利用になる方へ  EC-CUBE開発情報

オンライン状況

47 人のユーザが現在オンラインです。 (26 人のユーザが フォーラム を参照しています。)
登録ユーザ: 3
ゲスト: 44
may88watch 8xbetviet rangerbiru もっと...

サイト内検索
バグ報告 > 管理機能 > v2.12.5 規格2の初期値「選択してください」が表示されない

管理機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
mayaa
投稿日時: 2013/10/2 12:01
対応状況: −−−
新米
登録日: 2013/10/2
居住地:
投稿: 1
v2.12.5 規格2の初期値「選択してください」が表示されない
受注管理 > 検索
検索結果一覧 > 編集
受注商品情報一覧 > 変更
ポップアップの商品選択 > 検索
商品一覧

この順にたどっていき、商品一覧に規格2がある商品が2個以上並ぶと
1個目の商品の規格2には「選択してください」という選択肢があるのに対し、
2個目以降の商品の規格2には選択肢が表示されていません。

http://demo.ec-cube.net/admin/
このデモで商品名「赤キャベツ」で検索すると再現します。
sumida
投稿日時: 2013/10/2 22:15
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
Re: v2.12.5 規格2の初期値「選択してください」が表示されない
とりあえず、原因として、function fnOnLoad()時に実行する関数が２個以上あっても、１個目だけ有効になっているような感じです。

function fnOnLoad()時の関数を２個目を直書きで、１個目の前にいれたら、直書きが有効になりましたが、その次（元の１個目）は、有効になっていません。
sumida
投稿日時: 2013/10/3 11:52
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
Re: v2.12.5 規格2の初期値「選択してください」が表示されない
進展がありました。

javascript部分の関数 fnSetClassCategories 内で呼び出されている fnCheckStock が影響しているようです。
呼び出し部分をコメントにすると、２個目以降も有効になりました。

なお、直接の原因は、まだです。

追記：
少なくとも、fnCheckStockの最終行でストップしており、２個目以降の処理がされていないようです。

再追記：
fnCheckStockの中で、規格１が未選択の場合の処理が抜けているような。
sumida
投稿日時: 2013/10/3 14:25
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
Re: v2.12.5 規格2の初期値「選択してください」が表示されない
結局、解決しきれないのですが、

javascriptの関数 fnCheckStock の処理が、productsClassCategories に設定されているデータに合致していないような。
（特に、「選択してください。」の場合）

保守性を考慮すれば、規格２があるか否かで処理を分けた方が良いと思いました。
sumida
投稿日時: 2013/10/4 18:45
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
Re: v2.12.5 規格2の初期値「選択してください」が表示されない
原因：
規格２が '__unselected'（「選択してください」）の場合、javascriptの関数 fnCheckStock の処理が不正になっていました。
詳細：
productsClassCategoriesは、規格の一覧を保持した配列変数になっています。
'__unselected'の場合、'classcategory_id2'が未定義であったため、関数 fnCheckStock 内で、変数 classcat_id2 に、「選択してください」（form.classcategory_id2.valueの値）が代入されていました。
その結果

classcat2 = eccube.productsClassCategories[product_id][classcat_id1]['#選択してください' ];

という配列の参照になり、以降が動作不良になっていました。

対策：
(1) productsClassCategoriesの内容を追加
data/class/SC_Product.php

            $classCategories = array();
// added
            $classCategories['__unselected']['__unselected']['classcategory_id2'] = '__unselected';
// added
            $classCategories['__unselected']['__unselected']['name'] = '選択してください';
            $classCategories['__unselected']['__unselected']['product_class_id'] = $arrProductClasses[0]['product_class_id'];
            // 商品種別
            $classCategories['__unselected']['__unselected']['product_type'] = $arrProductClasses[0]['product_type_id'];


規格２への割当部分の処理追加
data/Smarty/templates/admin/order/product_select.tpl

    function fnCheckStock(form) {
        product_id = form.product_id.value;
        classcat_id1 = form.classcategory_id1.value;
        classcat_id2 = form.classcategory_id2 ? form.classcategory_id2.value : '';
// added_1
if (classcat_id2 == '__unselected') {
        classcat2 = eccube.productsClassCategories[product_id][classcat_id1][classcat_id2];
} else {
// added_1
        classcat2 = eccube.productsClassCategories[product_id][classcat_id1]['#' + classcat_id2];
// added_2
}
// added_2
        // 商品規格
        eleDynamic = document.getElementById('product_class_id' + product_id);

これでどうでしょう。

追記：
ショップ画面でも、同様の問題がありますが、
上記修正がなくても、動作しています。
（修正した方が良いと思います。）
v2.12.5
html/js/products.js（checkStock）
v2.13.0
html/js/eccube.js（eccube.checkStock）
です。
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


EC-CUBE公式 Amazon Payプラグイン

統計情報

総メンバー数は93,679名です
総投稿数は111,075件です

投稿数ランキング

1
seasoft 		7369
2
468 		3217
3
AMUAMU 		2712
4
nanasess 		2318
5
umebius 		2085
6
yuh 		1857
7
h_tanaka 		1756
8
red 		1574
9
mcontact 		1466
10
tsuji 		958
11
fukap 		907
12
shutta 		835
13
tao_s 		804
14 ramrun 789
15 karin 689
16 sumida 641
17
homan 		633
18 DELIGHT 572
19
patapata 		502
20
flealog 		485


ネットショップの壺

EC-CUBEインテグレートパートナー

Copyright© EC-CUBE CO.,LTD. All Rights Reserved.