Item15052: Support a cover plus skin as the publishskin

pencil
Priority: Normal
Current State: Being Worked On
Released In: n/a
Target Release: n/a
Applies To: Extension
Component: PublishPlugin
Branches:
Reported By: BramVanOosterhout
Waiting For: BramVanOosterhout
Last Change By: BramVanOosterhout
-- BramVanOosterhout - 20 Nov 2021

The publishskin parameter insists on a single value. covers that modify a skin (for instance mycover,pattern are rejected.

Publisher.pm line 77-81
   publishskin => {
        default   => 'basic_publish',
        validator => \&validateWord,
        desc      => 'Publish Skin'
    },

Foswiki::Func::readTemplate supports a comma separated list for the skin parameter, so the logic in Publisher.pm line 544-546 would support multiple values.
    $this->{skin_template} =
      Foswiki::Func::readTemplate( $this->{opt}->{template},
        $this->{opt}->{publishskin} );

I am changing line 77-81 to:
 publishskin => {
        default   => 'basic_publish',
        validator => sub {
                validateList( @_, \&validateWord );
                },
        desc      => 'Publish Skin'
    },

to support the use of a cover over an existing skin as the publishskin.

 
Topic revision: r1 - 20 Nov 2021, BramVanOosterhout
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