Noticed this issue while testing MDL-29224.

Sam's comment regarding this issue:
The URL module in all branches should include URL's ending with '/' when searching for internal links. Without this check links to something like http://yoursite.local/moodle/course/ will be processed incorrectly and avoid having the onclick/extra fields set.