Skip to content

Doctrine event listener could persist a null value #8541

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

Closed
wants to merge 1 commit into from

Conversation

matt9mg
Copy link
Contributor

@matt9mg matt9mg commented Oct 23, 2017

In the current docs, say the entity name is updated and we have not uploaded a new file, it looks like the fileName would be null meaning if the entity already has a brochure it would be removed?

In the current docs if say the product name is updated it looks like the fileName would be null meaning if a product alread has a brichure it would be removed?
@xabbuh xabbuh added this to the 2.7 milestone Oct 24, 2017
@xabbuh
Copy link
Member

xabbuh commented Oct 27, 2017

Thank you @matt9mg.

xabbuh added a commit that referenced this pull request Oct 27, 2017
This PR was submitted for the 3.3 branch but it was merged into the 2.7 branch instead (closes #8541).

Discussion
----------

Doctrine event listener could persist a null value

In the current docs, say the entity name is updated and we have not uploaded a new file, it looks like the fileName would be null meaning if the entity already has a brochure it would be removed?

Commits
-------

713f382 Doctrine event listener could persist a null value
@xabbuh xabbuh closed this Oct 27, 2017
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.

3 participants