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.

Sign up for GitHub

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

Jump to bottom

PHP FATAL ERROR use FOS\MessageBundle\Model\ThreadMetadata #287

Closed
aliel opened this issue Sep 2, 2016 · 1 comment
Closed

PHP FATAL ERROR use FOS\MessageBundle\Model\ThreadMetadata #287

aliel opened this issue Sep 2, 2016 · 1 comment
Assignees
@tgalopin

Comments

@aliel
Copy link

aliel commented Sep 2, 2016

Hi

with last merge, i have an error =>
PHP Fatal error: Cannot use FOS\MessageBundle\Model\ThreadMetadata as ThreadMetadata because the name is already in use in MessageBundle/Entity/Thread.php on line 12

Suspected commit => #286
d33c18a#diff-a365cf1835fe7fc524c52bdfcc452db8R10

env : PHP 7.0.10
@tgalopin
Copy link
Contributor

tgalopin commented Sep 3, 2016

That's indeed the issue. I'm going to fix it in a few minutes.

Thanks!

@tgalopin tgalopin closed this as completed Sep 3, 2016
tgalopin added a commit that referenced this issue Sep 3, 2016
@tgalopin
Fix #287 and improve CS/readability of various parts of the code
2ee5268
tgalopin added a commit that referenced this issue Sep 3, 2016
@tgalopin
Fix #287 and improve CS/readability of various parts of the code
f63498c
tgalopin added a commit that referenced this issue Sep 3, 2016
@tgalopin
Fix #287 and improve CS/readability of various parts of the code
88c3dea
tgalopin added a commit that referenced this issue Sep 4, 2016
@tgalopin
Merge pull request #288 from FriendsOfSymfony/fix-287
cdfee5e 
Fix #287 and improve CS/readability of various parts of the code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tgalopin tgalopin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aliel @tgalopin