バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

機能要望 > フロント機能 > 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。

フロント機能

新規スレッドを追加する

| 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
tao_s
投稿日時: 2011/3/16 20:47
対応状況: −−−
仙人
登録日: 2008/8/20
居住地: 東京
投稿: 799
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
ご確認ありがとうございます。

仰る通り、急いで作ったので、いろいろぬるいところはあると思います。
取り急ぎ、最低限XSS等の脆弱性や顧客情報の漏洩だけ無い様に作ってあります。

csvファイルの取り扱い等はもう少し改善します。


----------------
EC-CUBEカスタマイズ相談してください。
緊急のEC-CUBEの障害対応
EC-CUBEカスタマイズブログ

y424
投稿日時: 2011/3/16 22:01
対応状況: −−−
一人前
登録日: 2007/7/13
居住地: saitama
投稿: 81
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
ソースありがとうございます。
ようやく停電から復旧したので試してみます。


それと、2.4.4のダウンロードを公式本で載っていて導入を試みたのですが旨く行かず断念しました・・・


早速、ダウンロードのヤツを探してきます。




まったく別件ですが、ガソリンの件。
ご存知かと思いますがかなりのガソリンスタンドで給油の制限や上列をしていますが、1週間ぐらいでは多少通常に戻る兆しがあるみたいです。
さっき、今回のコンビナート火災は出光とシェルは関係ないんですね。
報告までに


----------------
------------------
管理店舗
2.11.1
  

最新が好きなので知識なくチケット?をファイルごと上書きでよく壊すことがある。

テストは最新版を

動作環境
サーバー名 coreserver.jp
CPU Intel X

flealog
投稿日時: 2011/3/16 22:01
対応状況: −−−
仙人
登録日: 2008/6/10
居住地:
投稿: 485
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
引用:

patapataさんは書きました:
環境がない為、確認が取れておりませんが・・・
ソースを流し読みした限り、不具合ではありませんが、ファイル周りの処理が少々甘いようにおもわれます。

donation.csv
このファイルを読み取り専用にした場合、おそらくポイントだけ引かれ、
$fp = fopen(DATA_PATH.'downloads/donation.csv','a+');
にて、$fpがfalseで返ってき、続くfwriteで失敗するかと。
同様に排他制御がない為、処理がぶつかった場合も情報をロストしてしまうように思われます。
(donation.csvをデータベース代わりに使っている為)

またその場合、ポイントを何に使ったか?
それを記録される場所もない?

なにぶんソースでの流し読みなので、上記指摘が正しいかわかりませんが、どなたか確認していただければ幸いです。

----追記----
また、当スレッドでは記載してありますが、Wikiや公式ページには
donation.csvにてデータ管理している旨の記載がない為、
一般のサイトオーナー様が、誤って削除・上書きしてしまう恐れがあります。
なにぶん簡易モジュールである為、仕方がないこととは思いますが、通常管理ではない場合は、仕様に際しその旨を正しく共有しあわなければならないかと私は思います。
readmeつけるなりでもよいかと・・・


patapata様

ご指摘ありがとうごいます。
wikiを編集し、仕様の説明と注意を掲載いたしました。

http://www47.atwiki.jp/eccubedonation/

今後も何かお気づきの点がございましたら、ご指摘をお願いいたします。
よろしくお願いいたします。


----------------
EC-CUBE3っぽいテンプレート
GitHub で公開中
https://github.com/bluestylejp

お問い合わせ番号CSV登録&メール送信プラグイン
http://urx.red/qDT6

patapata
投稿日時: 2011/3/16 22:08
対応状況: −−−
仙人
登録日: 2010/7/7
居住地: 神奈川県
投稿: 502
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
もう一点。
こちらは自信ありませぬが、
管理側の方にて、全てWhere文にdel_flgの記述があることから、
寄付を頂いたお客様がその後、退会した場合。

お名前表示で・・・
エラーを起こすもしくは、画面表示とcsvの内容が一致しなくなる
ように思われます。

※なにぶん環境がない為、指摘自体に誤りがあるかもしれません。

修正の際、あわせてみていただければ幸いです。
nanasess
投稿日時: 2011/3/16 22:11
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2314
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
引用:

patapataさんは書きました:

同様に排他制御がない為、処理がぶつかった場合も情報をロストしてしまうように思われます。


http://php.net/manual/ja/function.fwrite.php の注意書きを読むと,

「fwrite() をコールする前にリソースを flock() する必要がない」と書かれていますが, いかがでしょうか?

厳密な挙動には, あまり詳しくないので, はずしていたらすみません.
patapata
投稿日時: 2011/3/16 22:26
対応状況: −−−
仙人
登録日: 2010/7/7
居住地: 神奈川県
投稿: 502
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
>flealog様
ご対応ありがとうございます。
簡易修正としては・・・
$fp = fopen(DATA_PATH.'downloads/donation.csv','a+');

の後に$fpがflaseかどうかのチェック。
flaseの場合の分岐を加え、site.logに以下を記載。
date("Y-m-d G:i:s").",".$objCustomer->getValue('customer_id').",".$point."\n"
等でよいかと。

PHPのファイルロックは、不勉強の為やり方よくわかりませぬ。
申し訳ない。
patapata
投稿日時: 2011/3/16 22:59
対応状況: −−−
仙人
登録日: 2010/7/7
居住地: 神奈川県
投稿: 502
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
>nanasess様
「flockする必要がない」と書かれていますが、「アトミックである」とも書かれてもいます。
利用者が少なければ、まずぶつからないでしょう。
しかし多ければ違います。
ファイル操作において厳密なやりとりをする際は、必ずロックしなければなりません。

現象確認方法としては、該当処理を数万回転させ、ぶつけてみれば
そのうちの何件かはロストするかと。

またそこまでする必要があるか?
という方もおられるかと思いますが、今回預かるのは「お客様の善意の気持ち」です。
そして設置してるオーナー様も善意の気持ちで設置しているかと思われます。
開発側としてはそれをむげにしない為に、なるべく厳密に行うべきかなと思います。
popo
投稿日時: 2011/3/17 4:28
対応状況: −−−
長老
登録日: 2008/10/1
居住地:
投稿: 189
ポイント寄付後の退会した場合について
ポイント寄付後に退会した場合についてですが、試してみた結果は管理画面では退会者の名前は空白になります。
日時、会員IDや寄付ポイント数は表示されます。
またCSVファイルでも確認してみましたが、内容が一致しなくなる事はありませんでした。
退会した場合でも、CSVファイルには会員IDや寄付ポイント数は記載されていました。

ポイント寄付後の退会があった場合でも、異常は無い様です。


2.3.0
PHPバージョン
PHP 4.4.9
DBバージョン
PostgreSQL 7.3.4
nanasess
投稿日時: 2011/3/17 9:20
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2314
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
引用:

patapataさんは書きました:

またそこまでする必要があるか?
という方もおられるかと思いますが、今回預かるのは「お客様の善意の気持ち」です。
そして設置してるオーナー様も善意の気持ちで設置しているかと思われます。
開発側としてはそれをむげにしない為に、なるべく厳密に行うべきかなと思います。


今回は, 既存のカスタマイズに, 極力干渉しないよう, ファイルベースにしましたが, そこまでするのであれば, DB にしてしまった方が良さげですね.
y424
投稿日時: 2011/3/17 9:37
対応状況: −−−
一人前
登録日: 2007/7/13
居住地: saitama
投稿: 81
Re: 東北地方太平洋沖地震へポイントを寄付できるブロックを作りました。
結局現時点ではダウンロードしたものを使っているので問題ないんでしょうか?


----------------
------------------
管理店舗
2.11.1
  

最新が好きなので知識なくチケット?をファイルごと上書きでよく壊すことがある。

テストは最新版を

動作環境
サーバー名 coreserver.jp
CPU Intel X

« 1 ... 3 4 5 (6) 7 8 9 10 »
| 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


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

統計情報

総メンバー数は89,504名です
総投稿数は110,106件です

投稿数ランキング

1
seasoft
7367
2
468
3217
3
AMUAMU
2712
4
nanasess
2314
5
umebius
2085
6
yuh
1819
7
h_tanaka
1656
8
red
1570
9
mcontact
1307
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.