Allowing page->url to be hooked

[pjg]

I've been using pw successfully for a while now and for two of my recent projects I had to rewrite the page->url function. Since rewriting this in the actual core is not great for updates it would be great if the function would generally be made hookable. This allows for a more dynamic way of dealing with links and link rendering while, as far as I can see so far, not interfering with the actual functionality of the pw core.

For the time being I'm more than happy to add my 3 _ to it and with the next update do it again, but it would be useful if it was there anyway.

[Soma]

You would instead hook Page::path. It' used by url and making url also hookable doesn't make sense and maybe even for performance reasons as it can be called many times in a request.

Or you could add your own url method to Page.

[kongondo]

And we have a new member...welcome to PW pjg

[SiNNuT]

$pages->addHookAfter('Page::path', null, 'hookPagePath');
function hookPagePath(HookEvent $e) {
  $page = $e->object;
  if($page->template == 'article') $e->return = "/blog/$page->name/";
}

http://processwire.com/talk/topic/1799-routes-and-rewriting-urls/

[pjg]

For some reason hooking onto path didn't even cross my mind... sounds like the better idea though as url is dependent on path anyway and it would allow me to remove a workaround currently running for the admin pages.