| Comment | File | Size | Author |
|---|---|---|---|
| #70 | 2320269_69.patch | 8.62 KB | chx |
| #68 | 2320269_68.patch | 15.28 KB | chx |
| #64 | 2320269_63.patch | 9.04 KB | chx |
| #62 | 2320269_62.patch | 8.68 KB | chx |
| #60 | access_for_redirect.patch | 2.18 KB | tim.plunkett |
| Comment | File | Size | Author |
|---|---|---|---|
| #70 | 2320269_69.patch | 8.62 KB | chx |
| #68 | 2320269_68.patch | 15.28 KB | chx |
| #64 | 2320269_63.patch | 9.04 KB | chx |
| #62 | 2320269_62.patch | 8.68 KB | chx |
| #60 | access_for_redirect.patch | 2.18 KB | tim.plunkett |
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