機能要望 > その他 > 【リリース】EC-CUBE2.12.0 α版をリリースいたしました |
その他
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
ECCUORE |
投稿日時: 2012/3/23 15:37
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました version-2_12-dev 21670 で試しても同じ結果でした。
|
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アクセス時に同時に行わせたい処理」に関してプラグインで行うのは難しいと考えてよろしいでしょうか。 (可能な場合もあるが)
|
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 の実装がコーディングガイドラインに沿ってないので, 修正していった方が良さげですね. ただ, こういったケースにすべて対応するため, キーとなるIDをすべてメンバ変数にするのは, ちょっと気持ち悪い気が... 現状の実装で何とかするならば, プラグイン側でオーバーライドしてしまうという強引な方法もあると思います. |
ECCUORE |
投稿日時: 2012/3/23 18:05
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました >現状の実装で何とかするならば, プラグイン側でオーバーライドしてしまうという強引な方法もあると思います.
参考に御伺いしたいのですが、どのような実装になりますでしょうか?
|
ECCUORE |
投稿日時: 2012/3/23 18:08
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました CategoryContentsプラグインにあるprefilterTransformですが、管理画面ではdevice_type_idが渡ってこないから、switchのdefaultも有効にしていると考えてよろしいでしょうか。
そもそも、渡ってこないのが2.12.0系のバグなのでしょうか。
|
ECCUORE |
投稿日時: 2012/3/23 18:44
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: 【リリース】EC-CUBE2.12.0 α版をリリースいたしました IPアドレス記録プラグインを作ってみました。
チケット1692に添付してあります。 本スレッドに書いてあること以外で気になるのはdtb_csvへのインサート時のNoについてです。 最新のNoを取ってやるべきかと思いますが、現在はテストプラグインなので固定で書いてます。 プラグイン作成する場合は結構多用すると思うので、標準で用意して頂けると助かります。
|
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 α版をリリースいたしました >サンプルコードを書いてみようと思ったのですが, 完璧に忙殺されておりまして...(汗)
御回答ありがとうございます。 大変お忙しいと思いますので、サンプルコードまで書いて頂かなくても大丈夫です。 指針として、この処理でこう上書きしろ とかという アドバイスさえいただければと思って参考までにお伺いしたまでです。 よろしくお願いします
|
« 1 (2) 3 » |
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |