Skip to content
This repository was archived by the owner on Jan 11, 2021. It is now read-only.

Removed @required doc tags#59

Open
darinda wants to merge 2 commits intonetgen:masterfrom
styleflasher:master
Open

Removed @required doc tags#59
darinda wants to merge 2 commits intonetgen:masterfrom
styleflasher:master

Conversation

@darinda
Copy link

@darinda darinda commented Nov 2, 2017

This is a replay of the following commit:

ezsystems/ezpublish-kernel@37e5858?diff=unified

It fixes a bug where the following message appears:

[Semantical Error] The annotation "@required" in property eZ\Publish\Core\FieldType\Image\Value::$id was never imported. Did you maybe forget to add a "use" statement for this annotation?

@emodric
Copy link
Member

emodric commented Nov 2, 2017

Hi @darinda! Thanks for this. I can merge this, however, it seems that there's an extra commit here that somehow escaped into your branch. You can rebase on top of current master to get rid of it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants