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

Fix #267: if no config specified all behaviors are enabled #298

Merged
merged 1 commit into from
Mar 9, 2016

Conversation

akerouanton
Copy link
Contributor

No description provided.

@docteurklein
Copy link
Contributor

fixes #297, #292, #286, #281, #272, #270

docteurklein added a commit that referenced this pull request Mar 9, 2016
Fix #267: if no config specified all behaviors are enabled
@docteurklein docteurklein merged commit 29c1606 into KnpLabs:master Mar 9, 2016
@akerouanton akerouanton deleted the fix/default-config branch March 9, 2016 11:15
@ghost
Copy link

ghost commented Mar 9, 2016

Error after update to 1.3.1
capture

@ghost
Copy link

ghost commented Mar 9, 2016

v1.3.2 didn't solve issue #297
capture1

@docteurklein
Copy link
Contributor

@YvesBlancquaert thanks for the quick feedback!
I guess we need to figure out what's the problem first, before tagging anything :)

Could you check if the TranslatableSubscriber class is even correctly called?
It seems that it does not automagically map the IDs anymore (https://github.com/KnpLabs/DoctrineBehaviors/blob/master/src/ORM/Translatable/TranslatableSubscriber.php#L93).

Can you debug this part and look if mapId is called?

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

Successfully merging this pull request may close these issues.

2 participants