Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Original issue is: on User settings report, after clicking evolution row for desktop vs mobile report user gets notice "No data for this graph" (checked at demo.piwik.org a minute ago). After debuging I found out that in case of Piwik_DataTable_Array after inserting empty rows, object is populated with separate index for fields. When evolution graph tries to filter out unnecesary columns, it passes original label, which is i.e. Desktop. In normal case (when there is no index) filtering is performed using actual labels (translated). On the other side, when this additional index is present ,it is used to filter out not needed rows. This leads to situation when datatable is being filtered to leave only translated label, but search is performed on untranslated index and this leads to returning of empty datatable. I figured out that the best way to avoid this situation is to block adding empty rows, and therefore additional index is not created. Maybe this isn't the best solution possible, but I'm not quite sure what is this additional index used for in other cases and this fixes Mobile vs Desktop evolution row report.