バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

その他 > その他 > GoogleAnalytics_pluginのテストについて

その他

新規スレッドを追加する

スレッド表示 | 古いものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
nanasess
投稿日時: 2009/11/9 9:28
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2325
Re: GoogleAnalytics_pluginのテストについて
引用:

seasoftさんは書きました:
管理機能からプラグインのインストール・アンインストールをできるように改訂しました。
http://svn.ec-cube.net/open_trac/changeset/18370


ありがとうございます!
まだ精査できておりませんが, とり急ぎお礼まで.
seasoft
投稿日時: 2009/11/7 9:26
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7369
Re: GoogleAnalytics_pluginのテストについて
管理機能からプラグインのインストール・アンインストールをできるように改訂しました。
http://svn.ec-cube.net/open_trac/changeset/18370

現状では、
・プラグイン関連のファイル配置は手動で行なう
・インストール・アンインストールで実行するのは固定SQLのみ
というモノです。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

nanasess
投稿日時: 2009/10/25 22:24
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2325
Re: GoogleAnalytics_pluginのテストについて
大変遅くなってすみません.
comu-ver2 にコミットしました.

http://svn.ec-cube.net/open_trac/changeset/18345
nanasess
投稿日時: 2009/10/22 11:23
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2325
Re: GoogleAnalytics_pluginのテストについて
引用:

seasoftさんは書きました:
require_base.php の「include_once(DATA_PATH . "require_plugin.php");」は、他の読み込んでいるファイルと同様に存在して読み込めるはずですから、require_once で良いような気がします。

(プラグイン利用時も通常は /data/require_plugin.php は書き換えずに利用する前提ですよね?)

いかがでしょうか?


そうですね.
こちらは, require_once に修正したいと思います.

ありがとうございます.
seasoft
投稿日時: 2009/10/22 11:09
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7369
Re: GoogleAnalytics_pluginのテストについて
require_base.php の「include_once(DATA_PATH . "require_plugin.php");」は、他の読み込んでいるファイルと同様に存在して読み込めるはずですから、require_once で良いような気がします。

(プラグイン利用時も通常は /data/require_plugin.php は書き換えずに利用する前提ですよね?)

いかがでしょうか?


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

nanasess
投稿日時: 2009/10/22 10:53
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2325
Re: GoogleAnalytics_pluginのテストについて
引用:

seasoftさんは書きました:
引用:


むしろ、
data/require_plugin.php

        if (file_exists($requireFile)) {
            require_once($requireFile);
        }
の方がマズくないですか?
存在はするけど、読み取れないとか、ディレクトリーだった とか。


検証もせず、自分の思い込みだけで書いてます・・・ スイマセン


スミマセン.
こちらも include_once の方が良さげですね.
ご指摘ありがとうございます.

$requireFile の確認は, インストーラできちんとやれるようにしたいですね.
include_once を使用するのは, もしもの時の保険ということで.
seasoft
投稿日時: 2009/10/22 2:09
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7369
Re: GoogleAnalytics_pluginのテストについて
> 万が一, プラグインの読み込みに失敗した場合でも, スクリプトを停止することなく処理を続行するためです.

現状ですと、「プラグインの読み込みに失敗した場合」になっておらず、「/data/require_plugin.php の読み込みに失敗した場合」になりませんか?

むしろ、
data/require_plugin.php

        if (file_exists($requireFile)) {
            require_once($requireFile);
        }
の方がマズくないですか?
存在はするけど、読み取れないとか、ディレクトリーだった とか。


検証もせず、自分の思い込みだけで書いてます・・・ スイマセン


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

nanasess
投稿日時: 2009/10/21 21:53
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2325
Re: GoogleAnalytics_pluginのテストについて
引用:

seasoftさんは書きました:

ちなみに、require_once でなく include_once なのは何か意図があってでしょうか?


万が一, プラグインの読み込みに失敗した場合でも, スクリプトを停止することなく処理を続行するためです.

require_once で失敗すると EC-CUBE 全体が使用不能になってしまいますので.
seasoft
投稿日時: 2009/10/21 15:19
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7369
Re: GoogleAnalytics_pluginのテストについて
> いま, コミット可能な環境にないので, 後ほどコミットしておきます!

ありがとうございます。よろしくお願いします。

ちなみに、require_once でなく include_once なのは何か意図があってでしょうか?


> data/class/pages/upgrade/ 以下のクラスで, 似たようなことをやってるので, 参考になりますかね??

情報ありがとうございます。

あまり査読した覚えの無いディレクトリですので、あとで見てみます。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

nanasess
投稿日時: 2009/10/21 14:46
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2325
Re: GoogleAnalytics_pluginのテストについて
引用:

seasoftさんは書きました:

とりあえず、
// プラグインを読み込む
include_once(DATA_PATH . "require_plugin.php");
をコメントアウトしておくのが無難かななど思いますが、どうでしょうか?


そうですね.
いま, コミット可能な環境にないので, 後ほどコミットしておきます!

引用:

基本的な設計として、どのタイミングで実行するかというのを決めておきたいです。

最終的には、tar アーカイブなどで1ファイルでインストールという方向ですかね?

そうだとすると、管理機能からボタンを押下するなどのタイミングで、ファイル展開して SQL 実行といった感じですかね。


そんな感じで大丈夫だと思います.
data/class/pages/upgrade/ 以下のクラスで, 似たようなことをやってるので, 参考になりますかね??
(1) 2 3 4 5 »
スレッド表示 | 古いものから 前のトピック | 次のトピック | トップ


 



ログイン


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

統計情報

総メンバー数は95,982名です
総投稿数は111,643件です

投稿数ランキング

1
seasoft
7369
2
468
3217
3
AMUAMU
2712
4
nanasess
2325
5
umebius
2085
6
yuh
1893
7
h_tanaka
1850
8
red
1574
9
mcontact
1536
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.