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

TransactionListener の追加 #1632

Merged
merged 4 commits into from Aug 9, 2016

Conversation

Projects
None yet
3 participants
@nanasess
Contributor

nanasess commented Jul 25, 2016

  • #1518
  • トランザクションがネストしないよう設定
  • PHPUnit 実行時はトランザクションがネストしてしまい、不要なロールバックが発生するため TransactionListener でトランザクション管理しないよう修正
  • PHPUnit 実行時は \Eccube\Application::testMode = true になるよう設定
  • 論理削除された Member を参照しないよう修正
    • TransactionListener を有効にすると何故かエラーになるため
  • FormServiceProvider を登録した場合は eccube.service.plugin インスタンスを生成するよう修正
    • TransactionListener を有効にすると何故かエラーになるため

chihiro-adachi and others added some commits Jun 14, 2016

トランザクションがネストしないよう設定
- PHPUnit 実行時はトランザクションがネストしてしまい、不要なロールバック
  が発生するために TransactionListener でトランザクション管理しないよう修正
- PHPUnit 実行時は \Eccube\Application::testMode = true になるよう設定
- 論理削除された Member を参照しないよう修正
  - TransactionListener を有効にすると何故かエラーになるため
- FormServiceProvider を登録した場合は eccube.service.plugin インスタン
  スを生成するよう修正
  - TransactionListener を有効にすると何故かエラーになるため
@nanasess

This comment has been minimized.

Show comment
Hide comment
@nanasess

nanasess Jul 26, 2016

Contributor

#1604 をマージしないと、時折テストに失敗します

Contributor

nanasess commented Jul 26, 2016

#1604 をマージしないと、時折テストに失敗します

@ryo-endo ryo-endo added the bug label Jul 28, 2016

@ryo-endo ryo-endo added this to the 3.0.11 milestone Jul 28, 2016

@ryo-endo ryo-endo merged commit d5345cf into EC-CUBE:master Aug 9, 2016

3 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ryo-endo

This comment has been minimized.

Show comment
Hide comment
@ryo-endo

ryo-endo Aug 9, 2016

Contributor

取り込みました。PullRequestありがとうございました。

Contributor

ryo-endo commented Aug 9, 2016

取り込みました。PullRequestありがとうございました。

@nanasess nanasess deleted the nanasess:improve/transaction branch Sep 26, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment