バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

バグ報告 > 管理機能 > 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない

管理機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
taji
投稿日時: 2011/7/4 20:35
対応状況: −−−
半人前
登録日: 2011/5/30
居住地:
投稿: 20
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
nanasess様

修正ありがとうございました。

引用:

別の原因なのですね...


PHP5.3.xだと再現する本件のバグを追っていて、なぜPHP5.2.xでは起きないのかと調査していて見つかったのが、isBlank()の件です。
どちらにしても不具合が改修された事に違いないです。

完全に調べ切れてないので、書き込むのをちょっと躊躇したのですが、、
引用:

/data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php
756: if (isset($changed_no)) {
757: $arrValues[$changed_no] = $arrProduct[$key];
758: } else {
759: $arrValues[] = $arrProduct[$key];
760: }

の759行目の$arrValues[]の添え字が指定されていないのが怪しいんじゃないかと思ってます。
MPV
投稿日時: 2011/7/5 11:08
対応状況: −−−
仙人
登録日: 2011/3/27
居住地:
投稿: 309
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
taji様
nanasess様

お世話になります。

またも、あまりお役に立てないかもしれませんが
以下のような確認を行いました。

なお、nanasess様にご登録いただきました
http://svn.ec-cube.net/open_trac/changeset/21014
は反映済みの状態です。

引用:
完全に調べ切れてないので、書き込むのをちょっと躊躇したのですが、、
引用:

/data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php
756: if (isset($changed_no)) {
757: $arrValues[$changed_no] = $arrProduct[$key];
758: } else {
759: $arrValues[] = $arrProduct[$key];
760: }

の759行目の$arrValues[]の添え字が指定されていないのが怪しいんじゃないかと思ってます。


今回の事象発生時の$changed_noを
SC_Utils::sfPrintR( $changed_no );
で確認いたしましたところ、NULLでしたので
taji様の仰られる通り、elseの方に
処理が流れているようです。

ですので、これもtaji様の仰られる通りなのですが
引用:
$arrValues[]の添え字が指定されていないのが怪しいんじゃないかと思ってます。

を踏まえて、以下のように設定してみました。

   if (isset($changed_no)) {
     $arrValues[$changed_no] = $arrProduct[$key];
   } else {
     $arrValues[$key+1] = $arrProduct[$key];
   }

これで確認したところ、張り付き現象がなくなりました。
ただ、当方の確認も最低限ですし、また変な勘違いをしている
可能性もありますので、ご見解を頂ければ幸いです。

どうぞよろしくお願いいたします。<(_ _)>
taji
投稿日時: 2011/7/5 11:18
対応状況: −−−
半人前
登録日: 2011/5/30
居住地:
投稿: 20
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
MPV様

私もMPV様と同様に考えて、下記のような変更を考えました。

引用:

if (isset($changed_no)) {
$arrValues[$changed_no] = $arrProduct[$key];
} else {
$productAddNo = 0;
if(is_array($arrExistsProductClassIds)) {
$productAddNo = count($arrExistsProductClassIds);
}
$arrValues[$productAddNo] = $arrProduct[$key];
}


この辺りの処理に詳しい方のご意見など伺えると助かります。
MPV
投稿日時: 2011/7/6 20:52
対応状況: −−−
仙人
登録日: 2011/3/27
居住地:
投稿: 309
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
taji様

確認が遅くなりまして申し訳ございませんでした。

当方の修正では穴だらけで、商品を追加しても
カウント数が変わらないので追加時にうまく動作しません。
真にお恥ずかしい限りです・・・。

taji様の修正内容でしたら、複数追加時でも
きちんと修正されますし、規格内容の張り付きも
回避できていました。

当方はtaji様の修正内容が正しいと思いますが
よろしければnanasess様、コミッターの方々に
動作をご確認頂き、修正内容をコミットして
頂ければと思っております。

お忙しいところ、大変恐縮ですが
nanasess様、コミッターの方々
どうぞよろしくお願いいたします。<(_ _)>
MPV
投稿日時: 2011/7/16 22:05
対応状況: −−−
仙人
登録日: 2011/3/27
居住地:
投稿: 309
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
その後、ご確認を頂けていないようなので
再度自己レスいたします。

http://svn.ec-cube.net/open_trac/ticket/1398

にて、nanasess様に発行して頂きましたチケット
(SC_Utils.phpのisBlank関数の修正対応)とは
別事象として、taji様に解決方法をご回答頂きました
規格情報が受注管理画面で張り付く事象につきまして、
(もしかして当方がステータスを解決済みに
してしまったせいかもしれませんが・・・)
お忙しいところ大変恐縮なのですが、チケットを
(解決方法も書いてありますので)発行して
頂ければ幸いです。

どうぞよろしくお願いいたします。<(_ _)>
nanasess
投稿日時: 2011/7/18 11:39
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2313
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
対応が遅くなり申し訳ございません.

再現性の確認をしようと思いながら, 忙殺されていました(汗)
とり急ぎ, チケットのみ登録しておきました.

http://svn.ec-cube.net/open_trac/ticket/1412
MPV
投稿日時: 2011/7/18 22:10
対応状況: −−−
仙人
登録日: 2011/3/27
居住地:
投稿: 309
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
nanasess様

お忙しいところご対応頂きまして
ありがとうございます。

当方こそ、せいぜい声を上げるくらいしかできず、
申し訳ありません・・・。

いつもnanasess様はじめ、コミッターの方々には
親切にご対応頂きまして、本当に感謝しております。

重ねまして、ありがとうございます。<(_ _)>

nanasess
投稿日時: 2011/8/3 20:54
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2313
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
大変遅くなりましたが, 事象を確認しましたので, 修正をコミットしておきました.

http://svn.ec-cube.net/open_trac/changeset/21117

ちなみに, 下記 Seasoft 様の修正でも解消されたようです.
http://svn.ec-cube.net/open_trac/changeset/21099

MPV 様, taji 様ありがとうございました.
MPV
投稿日時: 2011/8/4 22:58
対応状況: −−−
仙人
登録日: 2011/3/27
居住地:
投稿: 309
Re: 受注管理画面で商品の変更を行っても、規格情報がうまく反映されない
nanasess様

お世話になっております。<(_ _)>
お礼が遅くなりまして、こちらこそ申し訳ありませんでした。

ご確認、及びコミットして頂きまして
ありがとうございました。

引用:

ちなみに, 下記 Seasoft 様の修正でも解消されたようです.
http://svn.ec-cube.net/open_trac/changeset/21099


お教え頂きましてありがとうございます。
明日、こちらの修正内容も取り込もうと思います。
こちらの方はチケットを見ると、2.11.2に乗りそうですね。

話がややそれますが、2.11.2のリリース日が
やや気になっておりまして・・・。
(お盆前リリースだと、お盆期間中はショップも
休業なので導入作業がしやすいなあと個人的に
思っております。)
脱線して失礼いたしました。

重ねまして、ご対応ありがとうございました。
« 1 2 (3)
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


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

統計情報

総メンバー数は88,930名です
総投稿数は110,012件です

投稿数ランキング

1
seasoft
7367
2
468
3217
3
AMUAMU
2712
4
nanasess
2313
5
umebius
2085
6
yuh
1819
7
h_tanaka
1646
8
red
1570
9
mcontact
1296
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
799
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.