[Pmwiki-users] Suggested change to pgcust.php

[Patrick R. Michaud]

On Wed, May 14, 2003 at 09:51:13AM +1200, John Rankin wrote:
> Proposal:
> change the line that reads
>     $group = FmtPageName('$Group',$pagename);
> 
> to
>     if (preg_match("/^($GroupNamePattern)[.\\/]?\$/",$pagename,$match)) {
>         $group = $match[1];
>     } else {
>         $group = FmtPageName('$Group',$pagename);
>     }


This seems to be a reasonable change.  I'll put it in the next release.
Thanks for the suggestion.

Should I just go ahead and do the same thing for titles, as in

   if (preg_match("/^($GroupNamePattern)([.\\/]($PageTitlePattern))?/",
       $pagename,$match)) {
     $group = $match[1];
     $title = $match[3];
   } 

and skip the FmtPageName altogether?  

Pm