Item10578: publishskin no longer accepts lists of skins

pencil
Priority: Urgent
Current State: Closed
Released In: n/a
Target Release: minor
Applies To: Extension
Component: PublishPlugin
Branches:
Reported By: DiabJerius
Waiting For:
Last Change By: CrawfordCurrie
Version 2.2.0 no longer accepts multiple skins. For example,

PUBLISH_PUBLISHSKIN = publish, pattern

fails validation with the message
Invalid publishskin: 'publish,pattern'

The cause is at line 29 of Publisher.pm:
     28     inclusions       => { default  => '.*', validator => \&_wildcard2RE },
     29     publishskin      => { validator => \&_validateWord },
     30     relativedir      => { default => '', validator => \&_validateRelPath },

The validation should be done using _validateList, instead of _validateWord.

-- DiabJerius - 31 Mar 2011

Thanks, it's so much easier when someone attaches a patch!

-- CrawfordCurrie - 25 May 2011
 

ItemTemplate edit

Summary publishskin no longer accepts lists of skins
ReportedBy DiabJerius
Codebase 1.1.2
SVN Range
AppliesTo Extension
Component PublishPlugin
Priority Urgent
CurrentState Closed
WaitingFor
Checkins PublishPlugin:6e2b89632b52
TargetRelease minor
ReleasedIn n/a
Topic revision: r5 - 25 May 2011, CrawfordCurrie
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy