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
>
>
>
>