Item10585: topiclist parameter validation cannot handle subwebs and topic names with hyphens.

Version 2.20

I'm generating a list of topics from a search across multiple subwebs. The list returned by the search looks sort of like this:

Public.3C273-Wings-2001-01-30, Public/SAOTrace.Make-world, Public/SAOTrace.ManualInstall

This is being passed to the "topiclist" parameter. Unfortunately the validation fails because the regexp in _validateList in Publisher.pm doesn't allow the "-" or "/" characters:

     68 sub _validateList {
     69     my $v = shift;
     70     if ($v =~ /^([\w,. ]*)$/ ) {
     71         return $1;
     72     }
     73     my $k = shift;
     74     die "Invalid $k: '$v'";
     75 }

It should be able to handle any permitted topic name.

-- DiabJerius - 02 Apr 2011

I've attached a patch which creates a new _validateTopicList function to handle this problem

-- DiabJerius - 02 Apr 2011

Came to this via another path and ended up with a slightly different fix.

Commenting is disabled while not logged in

-- CrawfordCurrie - 25 May 2011