Re: [Phpwiki-talk] SiteMap.php

[Stefan <son...@ba...>]

Hello Reini,

i didn't understand the code at all so it was a quick fix that works 
without known problems for us.

One thing more in the original code, needle and heystack has to be 
changed i think?
&& !preg_match("/$this->ExcludePages/", $linkpagename))  {

Stefan

Reini Urban schrieb:

>Stefan schrieb:
>  
>
>>SORRY FORGOT TO DELETE ONE ECHO WITHIN THE FILE
>>here is the corrected Version
>>Stefan
>>
>>------
>>I've done some changes to SiteMap.php to eleminate a lot of errors and
>>warnings. The function preg_match was used wrong so i replaced it by a
>>few lines of code and now it works without error. SiteMap is used from
>>IncludeSiteMap.php which now works with page names including white space
>>etc ...
>>    
>>
>
>Sorry, your version is not all correct.
>Your problem is that you assume that
>  $this->ExcludedPages = join("|", $exclude);
>but
>  $this->ExcludedPages = "^(?:" . join("|", $exclude) . ")";
>
>$exclude is a user regular expression, which must be passed to preg_match.
>But $exclude could be PageList also, if taken from a <!plugin-list !>,
>so the real fix will be expand a possible pagelist.
>
>And of course to quote the linkname as plugin arg as your did, thanks!
>  
>