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 - This page was cached on 21 Feb 2017 - 23:35.

The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License