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

[EZP-31664] Fix undefined index notice in legacy content mapper when previewing draft #3038

Open
wants to merge 1 commit into
base: 7.5
Choose a base branch
from

Conversation

emodric
Copy link
Contributor

@emodric emodric commented May 29, 2020

Question Answer
JIRA issue EZP-31664
Bug/Improvement yes
New feature no
Target version 7.5
BC breaks no
Tests pass yes
Doc needed no

In some cases (I have yet to find the pattern), I get the following notice when previewing a draft:

image

This fixes the issue.

TODO:

  • Implement feature / fix a bug.
  • Implement tests.
  • Fix new code according to Coding Standards ($ composer fix-cs).
  • Ask for Code Review.

@emodric emodric changed the title [EZP-31664] Fix undefined notice in legacy content mapper when previewing draft [EZP-31664] Fix undefined index notice in legacy content mapper when previewing draft May 29, 2020
@emodric
Copy link
Contributor Author

emodric commented May 29, 2020

Found the "cause". It happens when draft is created in legacy admin interface. Diving into why this happens with legacy could prove to be quite a big rabbit hole, so I'm more inclined to fix this here :)

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

Successfully merging this pull request may close these issues.

1 participant