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

NativeFileSessionHandlerの引数へSessionのsave_pathを設定 #4572

Merged
merged 1 commit into from Jun 12, 2020

Conversation

h-kida
Copy link
Contributor

@h-kida h-kida commented Jun 12, 2020

概要(Overview・Refs Issue)

セッションの保存先が変更される問題を修正
#4571

テスト(Test)

システム情報より、session.save_pathが意図したパスへ変わることを確認

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか

@ryo-endo ryo-endo requested a review from nanasess June 12, 2020 07:27
@okazy
Copy link
Contributor

okazy commented Jun 12, 2020

ローカルで session.save_path が変更されていることを確認しました。
var/sessions/prod または var/sessions/dev にsessionが保存されてることを確認しました。

image

@nanasess
Copy link
Contributor

AppVeyor が落ちてるのはガチャなので無視してよさそうです

@okazy okazy linked an issue Jun 12, 2020 that may be closed by this pull request
@okazy
Copy link
Contributor

okazy commented Jun 12, 2020

UnitTestはTravisが通っているので問題ないかと思います。
E2ETestでエラーとなってしまっているので念のため再実行しておきます。

Codeception (ubuntu-18.04, 7.3, pgsql, admin01)

There was 1 error:

---------
1) EA04OrderCest: Ea0405-uc01-t01(& uc01-t02) 受注登録
 Test  codeception/acceptance/EA04OrderCest.php:order_受注登録
                                                                                                                                                                                                                                                                                                                         
  [Facebook\WebDriver\Exception\UnexpectedAlertOpenException] unexpected alert open: {Alert text : 選択されていません}
  (Session info: chrome=83.0.4103.97)
  (Driver info: chromedriver=83.0.4103.39 (ccbf011cb2d2b19b506d844400483861342c20cd-refs/branch-heads/4103@{#416}),platform=Linux 5.3.0-1022-azure x86_64)  
                                                                                                                                                                                                                                                                                                                         

Scenario Steps:

 29. $I->click("#form1 > div.c-conversionArea > div > d...") at codeception/_support/Page/Admin/OrderEditPage.php:174
 28. $I->click("#searchProductModalList > table > tbody...") at codeception/_support/Page/Admin/OrderEditPage.php:167
 27. $I->waitForElementVisible("#searchProductModalList...") at codeception/_support/Page/Admin/OrderEditPage.php:159
 26. $I->click("#searchProductModalButton") at codeception/_support/Page/Admin/OrderEditPage.php:158
 25. $I->fillField({"id":"admin_search_pro...},"チェリーアイスサンド") at codeception/_support/Page/Admin/OrderEditPage.php:157
 24. $I->waitForElementVisible({"id":"addProduct"}) at codeception/_support/Page/Admin/OrderEditPage.php:156

@ryo-endo ryo-endo added this to the 4.0.4 milestone Jun 12, 2020
@okazy okazy added the bug:High label Jun 12, 2020
@okazy okazy merged commit aa7f803 into EC-CUBE:4.0 Jun 12, 2020
@ryo-endo
Copy link
Contributor

こちらデグレのため4.0.4に取り込みます。
@h-kida ご対応ありがとうございました!

@okazy
Copy link
Contributor

okazy commented Jun 12, 2020

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

nanasess added a commit to nanasess/ec-cube that referenced this pull request Jun 15, 2020
ryo-endo added a commit that referenced this pull request Jun 29, 2020
#4572 でセッションの挙動が修正されて不要になった記述を削除
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

セッションの保存先が変わってしまっている
5 participants