Closed (fixed)
Project:
Drupal 7 to Modern Drupal Module Upgrader
Component:
Code
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
10 Sep 2014 at 15:55 UTC
Updated:
16 Sep 2014 at 03:02 UTC
Jump to comment: Most recent
Comments
Comment #1
phenaproximaThis is critical, but not a demo blocker for the Diff module.
Comment #2
phenaproximaThis seems to be working now for parameters mapped to paths (as long as they're explicitly mapped in the arguments array of the D7 route). If that was Greek to you, don't worry: it's a physical mental note for me.
You can verify this by converting the Diff module, then looking at the methods in the default controller. With the exception of diffsOverview() -- which doesn't explicitly map its parameter to the path, and is therefore skipped due to the ambiguity (we COULD account for that edge case, but why bother...it's an edge case) -- they'll have the proper rewrites applied.
Comment #3
phenaproxima