You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jspwiki.apache.org by Juan Pablo Santos Rodríguez <ju...@gmail.com> on 2019/01/29 19:30:34 UTC

[RESULT][VOTE] Release JSPWiki version 2.11.0.M1

Hi,

after 72h, we've 5 +1 (4 from PMCs), so the vote passes. I'll proceed with
the next release steps.

thanks to everybody voting!


best regards,
juan pablo

On Mon, Jan 28, 2019 at 10:51 PM Murray Altheim <mu...@altheim.com>
wrote:

> Hi Juan Pablo,
>
> With your notes and the JIRA issue I'm
>
> +1
>
> Thanks very much,
>
> Murray
>
> > Hi Murray
> >
> > I think it's a valid concern, although I think it's enough (for this
> > release) with the prominent section noting
> > this at https://jspwiki-wiki.apache.org/Wiki.jsp?page=NewIn2.11 This
> link
> > will be sent on the announce
> > mail so downstream users should see it and avoid potential hassles. Also,
> > as per release numbering,
> > changes should be expected when migrating to this version.
> >
> > Lastly, later tonight I'll open a JIRA so this doesn't get lost and gets
> > properly fixed with references to
> > Dirk's suggestions.
> >
> > WDYT?
> >
> > br,
> > juan pablo
> >
> > On Mon, Jan 28, 2019 at 9:39 AM Murray Altheim <mu...@altheim.com>
> > wrote:
> >
> >> Here's a possible solution. In WikiContext there's a method named
> >> setDefaultTemplate(HttpServletRequest) that has a FIXME note, to the
> >> effect that we need to check for the existence of the template
> >> directory.
> >>
> >> If we were to replace the beginning of the WikiContext.java file with:
> >>
> >> public class WikiContext
> >>     implements Cloneable, Command
> >> {
> >>     /**
> >>      * The name used for the default template. The value is {@value}.
> >>      */
> >>     public static final String DEFAULT_TEMPLATE_NAME = "default";
> >>
> >>     ...
> >>     private    String     m_template = DEFAULT_TEMPLATE_NAME;
> >>     ...
> >>
> >> and the beginning of the setDefaultTemplate() method with:
> >>
> >>     protected void setDefaultTemplate( HttpServletRequest request )
> >>     {
> >>         String defaultTemplate = m_engine.getTemplateDir();
> >>
> >>         // check to see if the template directory actually exists
> >>         if ( !templateDirectoryExists( m_engine,  request ) ) {
> >>             defaultTemplate = DEFAULT_TEMPLATE_NAME;
> >>         }
> >>         ...
> >>
> >> and provide this utility method, which returns true if it finds
> >> ViewTemplate.jsp
> >> in the template directory specified in the property file:
> >>
> >>     /**
> >>      * A test to see if the template directory specified in the wiki's
> >> properties actually
> >>      * exists.
> >>      * <p>
> >>      * This checks the existence of the <tt>ViewTemplate.jsp</tt> file,
> >> which exists in every
> >>      * template.
> >>      *
> >>      * @param engine the WikiEngine
> >>      * @param request the HttpServletRequest used to obtain the real
> >> path
> >>      * @return true if the template directory exists on the server
> >>      */
> >>     private boolean templateDirectoryExists( WikiEngine engine,
> >> HttpServletRequest request )
> >>     {
> >>         File templatesDir = new
> >> File(request.getServletContext().getRealPath("/"), "templates");
> >>         File templateDir = new File(templatesDir,
> >> engine.getTemplateDir());
> >>         File viewTemplateJsp = new File(templateDir,
> >> "ViewTemplate.jsp");
> >>         return viewTemplateJsp.exists();
> >>     }
> >>
> >> Would that do?
> >>
> >> Yes, I can either provide the changed file or a patch, but the above is
> >> the gist of it.
> >>
> >> I haven't quite understood the responses so far, i.e., whether or not
> >> this is considered serious enough to warrant not going forward. As I
> >> mentioned my downvote is open to change. The above suggestion could go
> >> in the next version...
> >>
> >> Cheers,
> >>
> >> Murray
> >>
> >>
> ...........................................................................
> >> Murray Altheim <murray18 at altheim dot com>                       = =
> >> ===
> >> http://www.altheim.com/murray/                                     ===
> >> ===
> >>                                                                    = =
> >> ===
> >>      In the evening
> >>      The rice leaves in the garden
> >>      Rustle in the autumn wind
> >>      That blows through my reed hut.
> >>             -- Minamoto no Tsunenobu
> >>
> >>
> >>
> >>
> >
>
>
>
> ...........................................................................
> Murray Altheim <murray18 at altheim dot com>                       = =  ===
> http://www.altheim.com/murray/                                     ===
> ===
>                                                                    = =  ===
>      In the evening
>      The rice leaves in the garden
>      Rustle in the autumn wind
>      That blows through my reed hut.
>             -- Minamoto no Tsunenobu
>
>
>
>