Skip to content

[PropertyInfo] conflict for phpdocumentor/reflection-docblock 3.2 #23549

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

Merged

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jul 17, 2017

Q A
Branch? 3.2
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

phpdocumentor/reflection-docblock included a change in release 3.2.0
which required a tag to be followed by a space. This conflicts with our
use of the @Group annotation:

/**
 * @var \DateTime[]
 * @Groups({"a", "b"})
 */
public $collection;

phpdocumentor/reflection-docblock included a change in release 3.2.0
which required a tag to be followed by a space. This conflicts with our
use of the `@Group` annotation:

```php
/**
 * @var \DateTime[]
 * @groups({"a", "b"})
 */
public $collection;
```
@xabbuh
Copy link
Member Author

xabbuh commented Jul 17, 2017

The PR related to this behaviour change is phpDocumentor/ReflectionDocBlock#89 (note the change regular expression). I guess that this should be solved in the the phpdocumentor/reflection-docblock package (e.g. by ignoring "invalid" tags or so).

@xabbuh xabbuh changed the title conflict for phpdocumentor/reflection-docblock 3.2 [PropertyInfo] conflict for phpdocumentor/reflection-docblock 3.2 Jul 17, 2017
@fabpot
Copy link
Member

fabpot commented Jul 17, 2017

@xabbuh Do we need to open an issue on the upstream repo about this BC break?

@fabpot
Copy link
Member

fabpot commented Jul 17, 2017

Thank you @xabbuh.

@fabpot fabpot merged commit 58d49f7 into symfony:3.2 Jul 17, 2017
fabpot added a commit that referenced this pull request Jul 17, 2017
…ock 3.2 (xabbuh)

This PR was merged into the 3.2 branch.

Discussion
----------

[PropertyInfo] conflict for phpdocumentor/reflection-docblock 3.2

| Q             | A
| ------------- | ---
| Branch?       | 3.2
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

`phpdocumentor/reflection-docblock` included a change in release 3.2.0
which required a tag to be followed by a space. This conflicts with our
use of the `@Group` annotation:

```php
/**
 * @var \DateTime[]
 * @groups({"a", "b"})
 */
public $collection;
```

Commits
-------

58d49f7 conflict for phpdocumentor/reflection-docblock 3.2
This was referenced Jul 17, 2017
@xabbuh xabbuh deleted the phpdocumentor-reflection-docblock-conflict branch July 19, 2017 06:55
@xabbuh
Copy link
Member Author

xabbuh commented Jul 19, 2017

I think we should: phpDocumentor/ReflectionDocBlock#108

@esynaps
Copy link

esynaps commented Jul 28, 2017

See PR #107

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

Successfully merging this pull request may close these issues.

4 participants