バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

質問 > 管理機能 > 売上の集計数字が違う

管理機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
ririkaru
投稿日時: 2017/3/31 11:25
対応状況: −−−
半人前
登録日: 2016/10/17
居住地:
投稿: 11
売上の集計数字が違う
▼テンプレート
[EC-CUBE]3.0.14 アップデート
[レンタルサーバ] CPI
[OS] CentOS
[PHP] 7.0.15
[データベース] postgresql
[導入プラグインの有無] 売上集計プラグイン



いつもお世話になっております。

新たに修正したい箇所がございます。


現在、管理画面ホームにて、

売上状況という項目にて、
月の売上、本日の売上、昨日の売上と表示されていますが、

こちらの集計データは、
「キャンセル」というステータスも含まれてしまっています。
キャンセルに関しては、売上の数字から抜きたいのですが、
方法はありますでしょうか?

また、売上集計プラグインも使用しており、
売上分析として活用させて頂いております。

ですが、こちらのプラグインも「キャンセル」ステータスも含まれてしまっています。

恐れ入りますが、ご教示の程よろしくお願い致します。
umebius
投稿日時: 2017/3/31 11:31
対応状況: −−−
登録日: 2016/7/22
居住地:
投稿: 2085
Re: 売上の集計数字が違う
Eccube/Controller/Admin/AdminController.phpのindexメソッドで
下記のようになっているのでキャンセルは抜かれているはずなのですが。

プラグインの影響か、ステータスのID変更されたりしました?

(config.ymlの値を編集したということも)


        /**
         * 売り上げ状況
         */
        $excludes = array();
        $excludes[] = $app['config']['order_processing'];
        $excludes[] = $app['config']['order_cancel'];
        $excludes[] = $app['config']['order_pending'];


----------------
EC-CUBEカスタマイズ・高速化・脆弱性対策・SEO ご相談ください。

2系・3系から4系へのバージョンアップ実績豊富

umebius
投稿日時: 2017/3/31 11:35
対応状況: −−−
登録日: 2016/7/22
居住地:
投稿: 2085
Re: 売上の集計数字が違う
        $app['eccube.event.dispatcher']->dispatch(EccubeEvents::ADMIN_ADMIM_INDEX_SALES, $event);
        $excludes = $event->getArgument('excludes');




同じファイルの上記コードのあとでdump($excludes)をしてみて、キャンセルステータスのIDが含まれているかどうか確認してみてください。

キャンセルステータスのIDはマスタデータで観れると思います。


----------------
EC-CUBEカスタマイズ・高速化・脆弱性対策・SEO ご相談ください。

2系・3系から4系へのバージョンアップ実績豊富

ririkaru
投稿日時: 2017/3/31 11:50
対応状況: −−−
半人前
登録日: 2016/10/17
居住地:
投稿: 11
Re: 売上の集計数字が違う
早急なご教示誠にありがとうございます。

確かにステータスID変えておりました。。。。

dumpしたところやはり含まれておりませんでしたので、

NOT IN (:excludes,4)


※4がキャンセルのステータスです

にて対応しましたが、取り急ぎはこのような形でも問題ないでしょうか?
今後ステータスIDを変更することはございません。


もしくは、今後のアップデート等の為に、
初期値自体を変えた方がいいのでしょうか?
もしその場合、どこで定義されているのか見つけられません。。
恐れ入りますが、教えていただけないでしょうか・・・?

よろしくお願い致します。
umebius
投稿日時: 2017/3/31 13:28
対応状況: −−−
登録日: 2016/7/22
居住地:
投稿: 2085
Re: 売上の集計数字が違う

NOT IN (:excludes,4)

この書き方ってできるのですか?

$excludes[] = 4;で良いと思いますが
正常に動くのであれば何でも大丈夫です。

app/config/eccube/config.ymlに
order_cancel: 4
と追記すれば大丈夫です。

初期値
order_back_order: 4
order_cancel: 3
order_deliv: 5
order_new: 1
order_pay_wait: 2
order_pending: 7
order_pre_end: 6
order_processing: 8


----------------
EC-CUBEカスタマイズ・高速化・脆弱性対策・SEO ご相談ください。

2系・3系から4系へのバージョンアップ実績豊富

ririkaru
投稿日時: 2017/3/31 13:43
対応状況: −−−
半人前
登録日: 2016/10/17
居住地:
投稿: 11
Re: 売上の集計数字が違う
umebius様

ありがとうございます。

やってみたらできてしまいました。INを使ってるので、[,]でくぎれば取り急ぎは出来るかもと思いやってみました。

ですが、umebiusがおっしゃるように
config.ymlを修正し対応したいと思います。

今後アップデートの度に対応するのは骨が折れるので、
なるべくsrc内は触らないようにしたかったのです。。


本当に助かりました。
ありがとうございます。
umebius
投稿日時: 2017/3/31 14:14
対応状況: −−−
登録日: 2016/7/22
居住地:
投稿: 2085
Re: 売上の集計数字が違う

IN (:excludes,4) が動作するというのは知りませんでしたし、試そうと思ったこともありませんでした。

ありがとうございます、勉強になりました。


----------------
EC-CUBEカスタマイズ・高速化・脆弱性対策・SEO ご相談ください。

2系・3系から4系へのバージョンアップ実績豊富

スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


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

統計情報

総メンバー数は88,820名です
総投稿数は109,982件です

投稿数ランキング

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
1290
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.