Make code php 7.3 lint-compatible by paxal · Pull Request #7325 · doctrine/orm

paxal · 2018-07-26T12:57:03Z

Current v2.6.2 raised a warning when ran on php 7.3 :

$ php7.3 -v ; find . -name '*.php' -exec php7.3 -l {} \;| grep -v '^No syntax errors'
PHP 7.3.0alpha4 (cli) (built: Jul 23 2018 11:02:37) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.0-dev, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.3.0alpha4, Copyright (c) 1999-2018, by Zend Technologies
PHP Warning:  "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in ./lib/Doctrine/ORM/UnitOfWork.php on line 2718
PHP Warning:  "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in ./lib/Doctrine/ORM/UnitOfWork.php on line 2747

I changed the continue into break (meaning of continue; inside a switch), although continue 2 seems to be equivalent as there is no code after the switch statement.

Majkl578

LGTM
continue and break inside a switch behave identically and in both cases are used as early-exit here.

Majkl578 · 2018-08-02T12:25:39Z

Note that the test suite was failing hard on 7.3 as well. :)

1) Doctrine\Tests\ORM\Persisters\BasicEntityPersisterCompositeTypeParametersTest::testExpandParametersWillExpandCompositeEntityKeys
"continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"?

/www/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php:2718
/www/doctrine/orm/lib/Doctrine/ORM/EntityManager.php:162
/www/doctrine/orm/tests/Doctrine/Tests/Mocks/EntityManagerMock.php:81
/www/doctrine/orm/tests/Doctrine/Tests/OrmTestCase.php:163
/www/doctrine/orm/tests/Doctrine/Tests/ORM/Persisters/BasicEntityPersisterCompositeTypeParametersTest.php:31

etc.

paxal · 2018-08-06T06:04:25Z

Should I provide a patch for 2.7 as well, or will the same patch be applied as well ?

andreybolonin · 2018-09-06T05:14:57Z

What about to release 2.6.3 with this bugfix ?

Majkl578 · 2018-09-23T02:40:12Z

Doesn't seem to be relevant to master, so not cherry-picking there.

gudinoemiliano · 2018-10-10T13:06:37Z

Anyone knows when this issue-fix will be available in a release? Thanks!

doctrine/orm#7325

Ocramius · 2018-10-15T13:52:02Z

@gudinoemiliano most likely after 7.3.0 is released

drzraf · 2018-11-12T17:27:52Z

It not possible to run composer update --scripts on PHP 7.3 until the fixed release is actually published on packagist (since composer action are what trigger the bug).
Quite open and quite blocking.

Ocramius · 2018-11-12T17:35:38Z

Closed, merged, handled. Will be released AFTER PHP 7.3 is out, as mentioned in #7325 (comment)

teohhanhui · 2018-11-20T17:48:10Z

It means projects that depend on Doctrine cannot test their builds with PHP 7.3 before it's released. Not really an ideal situation...

lcobucci · 2018-11-21T00:34:06Z

@gudinoemiliano @drzraf @teohhanhui we've just managed to release v2.6.3, have fun 👍

@seferov

v2.6.3 [![Build Status](https://travis-ci.org/doctrine/doctrine2.svg?branch=v2.6.3)](https://travis-ci.org/doctrine/doctrine2) This release provides fixes for many things, specially: - Regression in commit order calculation - BC-break in `EntityManager#find()` using optimistic lock outside of transaction - PHP 7.3 compatibility issues -------------------------------------------- - Total issues resolved: **8** - Total pull requests resolved: **26** - Total contributors: **26** Documentation ------------- - [7472: fix incorrect phpdoc typehint](doctrine#7472) thanks to @seferov - [7465: Fixes tiny typo in the 'Working with DateTime instances' documentation](doctrine#7465) thanks to @unguul - [7444: Fixed URLs of doctrine-mapping.xsd in docs](doctrine#7444) thanks to @Naitsirch - [7441: $hydrationMode throughout can be a string as well as int (for custom modes)](doctrine#7441) thanks to @asgrim - [7435: Fix a typo on Documentation](doctrine#7435) thanks to @oguzdumanoglu - [7434: Removed FAQ paragraph stating public variables are disallowed](doctrine#7434) thanks to @Naitsirch and @flaushi - [7423: Update association-mapping.rst](doctrine#7423) thanks to @ThomasLandauer - [7421: JIRA to Github issues on Limitations and Known Issues](doctrine#7421) thanks to @seferov - [7412: Some formatting improvements](doctrine#7412) thanks to @ThomasLandauer - [7411: Autoload error when following the Getting Started Guide](doctrine#7411) thanks to @ThomasLandauer - [7401: &doctrine#91;docs&doctrine#93; Fix docblock in `inheritance-mapping.rst`](doctrine#7401) thanks to @bobdenotter - [7397: Update getting-started.rst](doctrine#7397) thanks to @eibt - [7394: Class 'Doctrine\Common\Persistence\Mapping\Driver\AnnotationDriver' not found](doctrine#7394) thanks to @ekosynth - [7378: Typo fix](doctrine#7378) thanks to @BenMorel - [7377: Fix query andX doctype](doctrine#7377) thanks to @sserbin - [7374: Deprecation message in documentation for YAML](doctrine#7374) thanks to @SenseException and @iltar - [7360: Document getPartialReference() properly](doctrine#7360) thanks to @lcobucci Bug --- - [7471: Fix parameter value processing for objects with unloaded metadata](doctrine#7471) thanks to @alcaeus - [7367: Fix for BC break in 2.6.2 when calling EM::find() with LockMode::OPTIMISTIC outside of a TX](doctrine#7367) thanks to @timdev - [7328: Handle removed parameters by tree walker in Paginator](doctrine#7328) thanks to @plfort - [7325: Make code php 7.3 lint-compatible](doctrine#7325) thanks to @paxal - [7317: &doctrine#91;XML&doctrine#93; Fix default value of many-to-many order-by to ASC](doctrine#7317) thanks to @alexdenvir - [7260: Fix the handling of circular references in the commit order calculator](doctrine#7260) thanks to @stof - [6830: fix applying column options on foreign key columns](doctrine#6830) thanks to @Tobion Improvement ----------- - [7428: CI: Test against PHP 7.3](doctrine#7428) thanks to @Majkl578 - [7363: Fix compatibility with phan](doctrine#7363) thanks to @philippe-unitiz - [7345: Correct DOMDocument constructor in test](doctrine#7345) thanks to @guilliamxavier - [7307: Fix remaining usages of deprecated ClassLoader and Inflector from doctrine/common](doctrine#7307) thanks to @Majkl578 and @simonwelsh

CI on PHP 7.3 is running into a nasty deprecation notice that was fixed upstream. See doctrine/orm#7325

There were some fixes for php 7.3 in doctrine/orm#7325

Fixes issue with 7.3 doctrine/orm#7325

Make code php 7.3 lint-compatible

07fc401

Majkl578 approved these changes Aug 2, 2018

View reviewed changes

Majkl578 added the Bug label Aug 2, 2018

Majkl578 added this to the 2.6.3 milestone Aug 2, 2018

Majkl578 merged commit f1143f5 into doctrine:2.6 Aug 2, 2018

Majkl578 added the Pick to master label Aug 2, 2018

Majkl578 self-assigned this Aug 2, 2018

paxal deleted the php73_compatible branch August 5, 2018 16:30

Majkl578 mentioned this pull request Sep 18, 2018

PHP 7.3 regression/issue #7402

Closed

Majkl578 removed the Pick to master label Sep 23, 2018

mlocati mentioned this pull request Sep 29, 2018

concrete5 version 8 with PHP version 7.3 concretecms/concretecms#7138

Closed

BenMorel added a commit to brick/geo that referenced this pull request Oct 14, 2018

Skip Doctrine tests for PHP 7.3

22cbcaf

doctrine/orm#7325

Ocramius mentioned this pull request Oct 16, 2018

fix PHP 7.3 Warning: "continue" targeting switch is equivalent to "br… #7431

Closed

antograssiot mentioned this pull request Oct 30, 2018

add php 7.3 to travis api-platform/core#2290

Closed

lcobucci added Pick to master and removed Pick to master labels Nov 21, 2018

nanasess mentioned this pull request Dec 12, 2018

Add PHP7.3 to Travis EC-CUBE/ec-cube#4045

Merged

6 tasks

tanasecosminromeo added a commit to tanasecosminromeo/doctrine2 that referenced this pull request Dec 13, 2018

Make code php 7.3 lint-compatible doctrine#7325

9a37b55

This was referenced Jan 1, 2019

[PHP 7.3] [Doctrine] Warning: "continue" targeting switch is equivalent to "break". doctrine/DoctrineORMModule#579

Closed

[PHP 7.3] [Doctrine] Warning: "continue" targeting switch is equivalent to "break". DASPRiD/container-interop-doctrine#39

Closed

localheinz mentioned this pull request Jan 8, 2019

Enhancement: Add PHP 7.3 to Travis build matrix HelloGrayson/factory-girl-php#47

Merged

2 tasks

EtienneBruines mentioned this pull request Jan 22, 2019

PHP 7.2+ - Doctrine to 2.6.3 shopware5/shopware#1989

Closed

3 tasks

yceruto mentioned this pull request Jun 1, 2019

Updated the Travis config EasyCorp/EasyAdminBundle#2742

Closed

stollr pushed a commit to stollr/doctrine-orm that referenced this pull request Oct 1, 2019

Ported PR doctrine#7325 (making code php 7.3 lint-compatible) to v2.5

f4175a3

stollr mentioned this pull request Oct 1, 2019

Ported PR #7325 (making code php 7.3 lint-compatible) to v2.5 #7843

Closed

RCheesley mentioned this pull request Feb 17, 2020

Too many PHP Warning related to Switch - Continue PHP statement mautic/mautic#7843

Closed

AkenRoberts added a commit to doctrine-extensions/DoctrineExtensions that referenced this pull request Aug 16, 2020

Bump doctrine/orm dev dependency version slightly

202a420

CI on PHP 7.3 is running into a nasty deprecation notice that was fixed upstream. See doctrine/orm#7325

franmomu added a commit to franmomu/AliceDataFixtures that referenced this pull request Oct 2, 2020

Bump doctrine/orm to 2.6.3

f569448

There were some fixes for php 7.3 in doctrine/orm#7325

franmomu added a commit to franmomu/AliceDataFixtures that referenced this pull request Oct 2, 2020

Add conflict doctrine/orm with < 2.6.3

9cd07ea

Fixes issue with 7.3 doctrine/orm#7325

franmomu mentioned this pull request Oct 2, 2020

Fix PHPUnit deprecations theofidry/AliceDataFixtures#150

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Make code php 7.3 lint-compatible#7325

Make code php 7.3 lint-compatible#7325
Majkl578 merged 1 commit intodoctrine:2.6from
paxal:php73_compatible

paxal commented Jul 26, 2018

Uh oh!

Majkl578 left a comment

Uh oh!

Majkl578 commented Aug 2, 2018

Uh oh!

paxal commented Aug 6, 2018

Uh oh!

andreybolonin commented Sep 6, 2018

Uh oh!

Majkl578 commented Sep 23, 2018

Uh oh!

gudinoemiliano commented Oct 10, 2018 •

edited

Loading

Uh oh!

Ocramius commented Oct 15, 2018

Uh oh!

drzraf commented Nov 12, 2018 •

edited

Loading

Uh oh!

Ocramius commented Nov 12, 2018

Uh oh!

teohhanhui commented Nov 20, 2018

Uh oh!

lcobucci commented Nov 21, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

Conversation

paxal commented Jul 26, 2018

Uh oh!

Majkl578 left a comment

Choose a reason for hiding this comment

Uh oh!

Majkl578 commented Aug 2, 2018

Uh oh!

paxal commented Aug 6, 2018

Uh oh!

andreybolonin commented Sep 6, 2018

Uh oh!

Majkl578 commented Sep 23, 2018

Uh oh!

gudinoemiliano commented Oct 10, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Ocramius commented Oct 15, 2018

Uh oh!

drzraf commented Nov 12, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Ocramius commented Nov 12, 2018

Uh oh!

teohhanhui commented Nov 20, 2018

Uh oh!

lcobucci commented Nov 21, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

gudinoemiliano commented Oct 10, 2018 •

edited

Loading

drzraf commented Nov 12, 2018 •

edited

Loading