Skip to content

Fixed Ui Export Search Result Iterator for zero keys #3546

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
Closed

Fixed Ui Export Search Result Iterator for zero keys #3546

wants to merge 1 commit into from

Conversation

sivaschenko
Copy link
Member

Grid collections with id starting from zero are not exported to xml, because first iteration valid check (bool)$this->key() will return false (as key is 0).

From key() php function documentation: If the internal pointer points beyond the end of the elements list or the array is empty, key returns null.

So check for is_null is most appropriate here.

 - Fixed Ui Export Search Result Iterator for zero keys
@NadiyaS NadiyaS added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Feb 26, 2016
@NadiyaS
Copy link
Contributor

NadiyaS commented Feb 26, 2016

Hi @sivaschenko,
thanks for contributing.
Internal ticket was created MAGETWO-49817 to process your PR

@sivaschenko
Copy link
Member Author

Thank you @NadiyaS !

@sivaschenko
Copy link
Member Author

Was fixed in mainline. Not actual anymore

@sivaschenko sivaschenko deleted the ui-export-iterator-fix branch March 23, 2016 13:28
magento-engcom-team pushed a commit that referenced this pull request Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants