バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

機能要望 > その他 > 【リリース】EC-CUBE2.12.0 α版をリリースいたしました

その他

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
ECCUORE
投稿日時: 2012/3/23 15:37
対応状況: −−−
長老
登録日: 2009/10/22
居住地: 東京
投稿: 248
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
version-2_12-dev 21670 で試しても同じ結果でした。


----------------
EC CUORE 株式会社クオーレ
カスタマイズ御相談下さい。

ECCUORE
投稿日時: 2012/3/23 16:45
対応状況: −−−
長老
登録日: 2009/10/22
居住地: 東京
投稿: 248
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
プラグインを試しに作成中ですが、行き詰まりました。

弊社で無料公開中のモジュール「IPアドレス記録」と同等のプラグインを作成中です。
http://ec.cuore.jp/products/detail87.html

行き詰った点としては、「DBアクセス時に同時に行わせたい処理」をどう実現するかです。

対象フックポイント[lc_page_products_review_action_complete]
lfRegistRecommendDataで書き込まれたレコードにIPをUPDATEするという処理をプラグインで実装する予定でした。
ただ、lfRegistRecommendDataで発行されるreview_idは、関数の変数に格納される為、review_idの判断がプラグイン側で出来ません。

現在シーケンス値と書き込み内容や性別などから推測は出来るものの、少し気持ちが悪いです。

このように、「DBアクセス時に同時に行わせたい処理」に関してプラグインで行うのは難しいと考えてよろしいでしょうか。
(可能な場合もあるが)


----------------
EC CUORE 株式会社クオーレ
カスタマイズ御相談下さい。

yoshimoto
投稿日時: 2012/3/23 17:29
対応状況: −−−
EC-CUBE開発チーム
登録日: 2012/1/13
居住地:
投稿: 16
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
お世話になっております。
株式会社ロックオン吉本です。

確認した所、現状の仕様ではおっしゃられている通りプラグインでの関与は難しいです。

現状のEC-CUBEは当然プラグインが介入する想定でつくられていないので
こういったケースが多々発生するかと思います。

lfRegistRecommendDataの様なキーとなる値を発行し、扱う場合は
プラグインからも値が取れる様に関数自体(本体処理)を修正していくか
単純にフックポインを増やしていく対応が必要になってくるかと思います。

何か良い案があれば頂きたいです。
nanasess
投稿日時: 2012/3/23 17:41
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2303
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
引用:

lfRegistRecommendDataの様なキーとなる値を発行し、扱う場合は
プラグインからも値が取れる様に関数自体(本体処理)を修正していくか
単純にフックポインを増やしていく対応が必要になってくるかと思います。


lfRegistRecommendData の実装がコーディングガイドラインに沿ってないので, 修正していった方が良さげですね.
ただ, こういったケースにすべて対応するため, キーとなるIDをすべてメンバ変数にするのは, ちょっと気持ち悪い気が...

現状の実装で何とかするならば, プラグイン側でオーバーライドしてしまうという強引な方法もあると思います.
ECCUORE
投稿日時: 2012/3/23 18:05
対応状況: −−−
長老
登録日: 2009/10/22
居住地: 東京
投稿: 248
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
>現状の実装で何とかするならば, プラグイン側でオーバーライドしてしまうという強引な方法もあると思います.

参考に御伺いしたいのですが、どのような実装になりますでしょうか?


----------------
EC CUORE 株式会社クオーレ
カスタマイズ御相談下さい。

ECCUORE
投稿日時: 2012/3/23 18:08
対応状況: −−−
長老
登録日: 2009/10/22
居住地: 東京
投稿: 248
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
CategoryContentsプラグインにあるprefilterTransformですが、管理画面ではdevice_type_idが渡ってこないから、switchのdefaultも有効にしていると考えてよろしいでしょうか。

function prefilterTransform(&$source, LC_Page_Ex $objPage, $filename) {
        $objTransform = new SC_Helper_Transform($source);
        $template_dir = PLUGIN_UPLOAD_REALDIR . 'CategoryContents/templates/';
        switch($objPage->arrPageLayout['device_type_id']){
            case DEVICE_TYPE_MOBILE:
            case DEVICE_TYPE_SMARTPHONE:
            case DEVICE_TYPE_PC:
                // 商品一覧画面
                if (strpos($filename, 'products/list.tpl') !== false) {
                    $objTransform->select('h2.title')->insertBefore(file_get_contents($template_dir . 'snip_products_list_add.tpl'));
                }
                break;
            case DEVICE_TYPE_ADMIN:
            default:    ←←←←←←←←←←←←←←←←←←ここ
                // カテゴリ登録画面
                if (strpos($filename, 'products/category.tpl') !== false) {
                    $objTransform->select('div.now_dir')->replaceElement(file_get_contents($template_dir . 'snip.admin_basis_category_add.tpl'));
                }
                break;
        }
        $source = $objTransform->getHTML();
    }


そもそも、渡ってこないのが2.12.0系のバグなのでしょうか。


----------------
EC CUORE 株式会社クオーレ
カスタマイズ御相談下さい。

ECCUORE
投稿日時: 2012/3/23 18:44
対応状況: −−−
長老
登録日: 2009/10/22
居住地: 東京
投稿: 248
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
IPアドレス記録プラグインを作ってみました。
チケット1692に添付してあります。

本スレッドに書いてあること以外で気になるのはdtb_csvへのインサート時のNoについてです。
最新のNoを取ってやるべきかと思いますが、現在はテストプラグインなので固定で書いてます。
プラグイン作成する場合は結構多用すると思うので、標準で用意して頂けると助かります。


----------------
EC CUORE 株式会社クオーレ
カスタマイズ御相談下さい。

yoshimoto
投稿日時: 2012/3/26 10:07
対応状況: −−−
EC-CUBE開発チーム
登録日: 2012/1/13
居住地:
投稿: 16
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
ECCUORE様

>CategoryContentsプラグインにあるprefilterTransformですが、管理画面ではdevice_type_idが渡ってこないから、switchのdefaultも有効にしていると考えてよろしいでしょうか。
はい。ご指摘の様に管理画面ではdevice_type_idが入って来ないのでそういった実装になっています。
また、管理画面でdevice_type_idが空なのは仕様という認識をしています。

サンプルのプラグインありがとうございます!
早速自分の環境でも動かさせて頂きます。

>最新のNoを取ってやるべきかと思いますが、現在はテストプラグインなので固定で書いてます。
>プラグイン作成する場合は結構多用すると思うので、標準で用意して頂けると助かります。
今の実装ではまだまだプラグイン製作者への負荷も高いので、標準でプラグイン用のユーティリティを揃えていきたいですね!
フィードバックありがとうございます。

引き続きよろしくお願いいたします。
nanasess
投稿日時: 2012/3/29 11:42
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2303
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
ECCUOREさんは書きました:
>現状の実装で何とかするならば, プラグイン側でオーバーライドしてしまうという強引な方法もあると思います.

参考に御伺いしたいのですが、どのような実装になりますでしょうか?



サンプルコードを書いてみようと思ったのですが, 完璧に忙殺されておりまして...(汗)
手が遅くて申し訳ございません.
ECCUORE
投稿日時: 2012/3/29 13:02
対応状況: −−−
長老
登録日: 2009/10/22
居住地: 東京
投稿: 248
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました
>サンプルコードを書いてみようと思ったのですが, 完璧に忙殺されておりまして...(汗)

御回答ありがとうございます。

大変お忙しいと思いますので、サンプルコードまで書いて頂かなくても大丈夫です。
指針として、この処理でこう上書きしろ とかという アドバイスさえいただければと思って参考までにお伺いしたまでです。

よろしくお願いします


----------------
EC CUORE 株式会社クオーレ
カスタマイズ御相談下さい。

« 1 (2) 3 »
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


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

統計情報

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

投稿数ランキング

1
seasoft
7365
2
468
3217
3
AMUAMU
2712
4
nanasess
2303
5
umebius
2085
6
yuh
1818
7
h_tanaka
1610
8
red
1568
9
mcontact
1240
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
796
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.