@diosmosis opened this Pull Request on October 21st 2018 Member

Changes:

  • Fix original format tests that fail on mysqli due to it returning ints by casting Site properties to int.
  • Make sure table sort in join tables is stable by using DFS instead of weighting tables. Join dependencies are parsed/assumed and dependent table/table-aliases get ordered first.
  • Couple other minor test fixes.
@diosmosis commented on October 21st 2018 Member

Refs #13025

@tsteur commented on November 12th 2018 Member

It's a complex PR/topic and don't understand all the code / hard to review so focused on looking at tests. Ran the system tests for most premium features and they seem to pass. Also checked custom reports generates similar queries that still work (some SQLs changed but are still valid). As nothing seems to break I reckon it's good to merge (in 3.8).

@diosmosis commented on November 12th 2018 Member

I'll also go through the code and try to make it more expressive.

Powered by GitHub Issue Mirror