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

PHP7.1 対応 #1957

Closed
trebla-on opened this issue Dec 6, 2016 · 4 comments
Closed

PHP7.1 対応 #1957

trebla-on opened this issue Dec 6, 2016 · 4 comments
Labels
enhancement 機能追加
Milestone

Comments

@trebla-on
Copy link
Contributor

■問題:ユニットテストは失敗する
Eccube\Tests\Repository\TaxRuleRepositoryTest::testGetByRuleWithCountry
Failed asserting that 533 matches expected 532.

■説明:そのテストの中に「Eccube\Entity\TaxRule::compareTo()」を使ってます、メッソドの中に
DateTimeを比べてます、PHP7.1からmicrosecondsも含めています。
if ($this->getApplyDate() == $Target->getApplyDate()) {
PHP7.1のドキュメントhこちらです
http://php.net/manual/en/migration71.incompatible.php#migration71.incompatible.datetime-microseconds

■対応方法:フォーマトした日付を比べる
if ($this->getApplyDate() == $Target->getApplyDate()) {

if ($this->getApplyDate()->format('YmdHis') == $Target->getApplyDate()->format('YmdHis')) {

@Yangsin Yangsin changed the title ユニットテストFAILの調査と結果(PHP7.1) PHP7.1 対応 Dec 6, 2016
@Yangsin Yangsin added the enhancement 機能追加 label Dec 6, 2016
@Yangsin Yangsin added this to the 3.0.x milestone Dec 6, 2016
@Yangsin Yangsin modified the milestones: 3.0.14, 3.0.x Jan 6, 2017
@Yangsin
Copy link

Yangsin commented Jan 6, 2017

7.1環境で動作確認を行えればシステム要件に追加したいと思います。

@t-nagahashi
Copy link
Contributor

以下2種類の確認をもって、動作確認とします。
・codeceptionにて確認
・時期バージョンアップの動作確認テストにて、手動テスト

@nanasess
Copy link
Contributor

nanasess commented Feb 3, 2017

EC-CUBE API で以下の対応が必要です。
EC-CUBE/eccube-api#62

@t-nagahashi
Copy link
Contributor

codeception, 受入テストがOKとなりましたので、
3.0.14以降、PHP7.1をシステム要件に追加したいと思います。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能追加
Projects
None yet
Development

No branches or pull requests

4 participants