-
Notifications
You must be signed in to change notification settings - Fork 2
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
SW-659 Add separate route for export debug information #253
SW-659 Add separate route for export debug information #253
Conversation
$mainProduct = clone $result->first(); | ||
$children = $mainProduct->getChildren(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not using a cloned instance of the product lead to the children being recursively added. So it was possible to do this:
$result->first()->getChildren()->last()->getChildren()->last()->getChildren()->last()->getChildren()->last()
Issue came up, because the JsonResponse
class complained about recursion during JSON parsing:
Before:
- Requested Variant
- Sibling 1
- Sibling 2
- Main Variant
- Sibling 1 / Child 1
- Sibling 2 / Child 2
- Main variant
- .... (and so on)
Now:
- Requested Variant
- Sibling 1
- Sibling 2
- Main Variant
- Sibling 1 / Child 1
- Sibling 2 / Child 2
…_debug_information' into SW-659_separate_route_for_export_debug_information
Closing in favor of #285 |
No description provided.