Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

送料を0にできる不具合 #54

Closed
ArtemisHaD opened this issue Jun 2, 2015 · 5 comments
Closed

送料を0にできる不具合 #54

ArtemisHaD opened this issue Jun 2, 2015 · 5 comments
Labels
Milestone

Comments

@ArtemisHaD
Copy link

やり方
送付先の都道府県を設定するときにページのソースをいじり,
北海道=1, 青森=2,...と割り当ててある都道府県のIDを0に設定し送信する.

こうすることで処理が適切に行われず送料が0円に設定されるようです.

@Yangsin Yangsin added the bug label Sep 22, 2015
@ghost
Copy link

ghost commented Jan 6, 2016

$objFormParam->addParam('都道府県', $prefix . 'pref', INT_LEN, 'n', array('EXIST_CHECK', 'NUM_CHECK', 'PREF_CHECK'));

のような感じでPREF_CHECKメソッドを新規に追加してmtb_prefの最大最小値の範囲に収まっているかどうかを検査しようと思いますが、どうでしょうか

@nanasess
Copy link
Contributor

nanasess commented Jan 7, 2016

最大最小値のチェックよりも、マスタの存在チェックの方が良さそうですが、いかがでしょう。
海外向けに 「99: その他」などを追加カスタマイズする場合もありますので。

@ghost
Copy link

ghost commented Jan 7, 2016

@nanasess
マスタの存在チェックで直しました、どうでしょうか

@nanasess
Copy link
Contributor

nanasess commented Jan 8, 2016

@migarashi47 ありがとうございます!後ほど検証して、問題なければ採用したいと思います。

@ghost ghost mentioned this issue Mar 5, 2016
ryo-endo added a commit that referenced this issue Oct 26, 2017
@ryo-endo
Copy link
Contributor

修正の取り込み完了しました。

@ryo-endo ryo-endo added this to the eccube-2.13.6 milestone Oct 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants