ref #1226 マスターデータ管理を実装 #1227

Merged
merged 6 commits into from Dec 2, 2015

Projects

None yet

3 participants

@nobuhiko
Contributor
  • magic_quotes_gpc の対応も含む
  • 外部キーに使われているレコードを削除するとシステムエラーになる
  • mtb_zipを開こうとするとエラーになる(rankがないので)
nobuhiko added some commits Nov 23, 2015
@nobuhiko nobuhiko ref #1226 マスターデータ管理を実装
- magic_quotes_gpc の対応
0c52737
@nobuhiko nobuhiko magic_quotes_gpcの対応は別途する d94c730
@nobuhiko nobuhiko fixed scrutinizer
4906d21
@k-yamamura
Contributor

ご対応ありがとうございます。

2点修正をお願いしたいのですが、

  • マスターデータが1件も登録されていない場合、そのマスターデータを開いたらシステムエラーが発生します。
  • 直接DBのレコードをメンテナンスされたと仮定して誤って同じrankが指定されていた場合、
    1件しか画面上には表示されません。
id name rank
1 種別A 0
2 種別B 1
3 種別C 2
4 種別D 1

というレコードが仮に存在していた場合、画面上には

id name rank
1 種別A 0
3 種別C 2
4 種別D 1

しか表示されません。
あえてこのように表示されているようでしたら仕様ですと仰ってください。

お忙しいところ大変申し訳ございませんがご対応お願い致します。

nobuhiko added some commits Nov 28, 2015
@nobuhiko nobuhiko mtb_zipのような例外のテーブルを除去する 1d82f0a
@nobuhiko nobuhiko try catchの追加
88925b9
@nobuhiko
Contributor

@k-yamamura 確認ありがとうございます

マスターデータが1件も登録されていない場合、そのマスターデータを開いたらシステムエラーが発生します。

これが確認出来ませんでした
それ以外は直せたと思います

@chihiro-adachi
Contributor

@k-yamamura

マスターデータが1件も登録されていない場合、そのマスターデータを開いたらシステムエラーが発生します。

こちら以下環境で試しましたが、システムエラーは発生しませんでした。

php 5.6.8
mysql 5.6.24

※mtb_dbとmtb_coutryのデータを全件削除してためしました。

@k-yamamura
Contributor

@nobuhiko
ご対応ありがとうございます。

マスタデータが存在しないときにエラーが発生する件ですがindex_dev.phpをつけると、

ContextErrorException in MasterdataController.php line 56:
Notice: Undefined variable: key

とNoticeエラーが発生します。

index_dev.phpをつけなければ問題なく画面が表示されるのですが、
お手数ではありますがindex_dev.phpでもエラーが発生しないように対応してもらえないでしょうか。

以上よろしくお願い致します。

@nobuhiko nobuhiko fix EC-CUBE#1227 (comment)
c257504
@k-yamamura k-yamamura merged commit faced67 into EC-CUBE:master Dec 2, 2015

2 of 3 checks passed

Scrutinizer Failure condition met
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@k-yamamura k-yamamura added this to the 3.0.7 milestone Dec 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment