diff options
author | dizzy <diosmosis@users.noreply.github.com> | 2021-08-20 01:01:12 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-08-20 01:01:12 +0300 |
commit | 18d04c139a8a28e08d394c9afbf2b15ec8162ec0 (patch) | |
tree | eebb6659a43414c302e61f520043699232151821 /plugins/UserCountry | |
parent | 9d26506c1d005f1c480325b0e5f2a5046a96f9dd (diff) |
When aggregating reports, inflate one expanded datatable at a time (#17817)
* datatable should always be a map here
* rewrite datatable aggregation to inflate a single expanded table at a time, rather than all at once
* when adding a new row, make sure we do not share row instances between datatables (as this will mean sharing subtable instances as well)
* support deleting datatables between specific range of IDs
* fix datatable aggregation code that expands one datatable at a time
* fix parameter issue
* fix one type of test failure
* undo last change, variation in precision is expected in column values
* remove plugin submodule update
* debugging change
* change
* debug changes
* undo some debug changes, metadata changes are expected
* small refactor
* decimal and actions metadata changes expected
* lat long changes are for duplicates, expected
* correctly set metadata on created table
* fix test
* update expected screenshots
* revert debugging changes
* change modifier back
* update more tests
* fix UI test
Diffstat (limited to 'plugins/UserCountry')
-rw-r--r-- | plugins/UserCountry/tests/System/expected/test_multi_sites__UserCountry.getCity_month.xml | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/plugins/UserCountry/tests/System/expected/test_multi_sites__UserCountry.getCity_month.xml b/plugins/UserCountry/tests/System/expected/test_multi_sites__UserCountry.getCity_month.xml index b1c0065ea7..f9422d9715 100644 --- a/plugins/UserCountry/tests/System/expected/test_multi_sites__UserCountry.getCity_month.xml +++ b/plugins/UserCountry/tests/System/expected/test_multi_sites__UserCountry.getCity_month.xml @@ -91,8 +91,8 @@ <revenue>30</revenue> <sum_daily_nb_uniq_visitors>2</sum_daily_nb_uniq_visitors> <sum_daily_nb_users>0</sum_daily_nb_users> - <lat>124.456</lat> - <long>22.231</long> + <lat>123.456</lat> + <long>21.321</long> <segment>city==Stratford-upon-Avon;regionCode==WAR;countryCode==gb</segment> <city_name>Stratford-upon-Avon</city_name> <region>WAR</region> @@ -448,8 +448,8 @@ <revenue>30</revenue> <sum_daily_nb_uniq_visitors>2</sum_daily_nb_uniq_visitors> <sum_daily_nb_users>0</sum_daily_nb_users> - <lat>124.456</lat> - <long>22.231</long> + <lat>123.456</lat> + <long>21.321</long> <segment>city==Stratford-upon-Avon;regionCode==WAR;countryCode==gb</segment> <city_name>Stratford-upon-Avon</city_name> <region>WAR</region> |