Fix for issue 911 found on MSI project - Cannot read property source_…

[phoenix128]

…code of undefined

Dynamic data rows were failing due to a read after delete condition
Rows were removed just before the information update. An undefined javascript error was triggered.

This issue has ben found in MSI, but it comes from core implementation.

Fixed Issues (if relevant)

1. Cannot read property source_code of undefined inventory#911: Cannot read property source_code of undefined

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[VladimirZaets]

Hi, @phoenix128 , thank you for collaboration.
But this fix looks like concatenation of circumstances. According to design of dynamic-rows-grid component we use insertData as single source of data, so shouldn't to be important which operation will be called first.
If _super method will be called first, his will call parent deleteRecord method that will change only recordData property and insertData property shouldn't be affected.

[magento-engcom-team]

Hi @phoenix128. Thank you for your contribution.
Changes from your Pull Request will be available with the upcoming 2.3.0 release.