Closed (won't fix)
Project:
Drupal core
Version:
8.6.x-dev
Component:
other
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2014 at 23:51 UTC
Updated:
5 May 2019 at 09:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
chx commentedComment #3
chx commentedComment #5
chx commentedComment #7
chx commentedComment #9
chx commentedComment #10
chx commentedComment #12
chx commentedComment #14
chx commentedComment #15
chx commentedComment #17
benjy commentedComment #19
chx commentedfor now, CartesianAssociativeProduct is in Drupal\Component\Utility let's hope it gets merged upstream.
Comment #21
chx commentedComment #23
chx commentedfilterRoutesPerRequest was a) JOINing on
route_filterinstead of{route_filter}b) why JOIN, the point is to be fast and everything is in one table. I discovered a much bigger bug though, that is in HEAD, attaching a patch.If 23_1 is a no-go then we need to figure a better way to figure out what to filter. I guess
Comment #24
chx commentedComment #27
benjy commentedComment #29
benjy commentedComment #30
benjy commentedComment #32
benjy commentedComment #34
benjy commentedComment #36
benjy commentedComment #37
chx commentedRemoved hoa in favor of my own optimized (not only smaller but faster) CartesianProduct with a test, changed the fetching in getRoutesByPath to only fetch for one path only.
Comment #39
chx commentedComment #40
chx commentedComment #42
benjy commentedComment #44
benjy commentedComment #46
chx commentedComment #47
chx commentedComment #48
benjy commentedComment #50
znerol commentedComment #52
benjy commentedComment #53
benjy commentedComment #55
chx commentedComment #56
tim.plunkettComment #58
tim.plunkettComment #60
tim.plunkettComment #62
chx commentedComment #64
chx commentedComment #66
chx commentedComment #67
chx commentedComment #68
chx commentedComment #70
chx commentedComment #82
dpiDead