Did a quick test from the war (will set up a dev environment so can do some
more testing!),  here is what i found:

Hard coded stuff :

roller.properties

log4j.appender.roller.File=/Applications/Java/apache-tomcat-8.5.9/logs/roller.log

(maybe switch to log4j2?
<Property name="log-path">${sys:catalina.home}/logs</Property>)

##

Media upload gives this error (but works):

SEVERE: Servlet.service() for servlet [jsp] threw exception
org.apache.jasper.JasperException:
/WEB-INF/jsps/editor/MediaFileAddSuccess.jsp (line: [83], column: [12])
Attribute [id] invalid for tag [iterator] according to TLD

<s:iterator value="newImages" id="newImage">

id="newFile" two of these!

###

Menu preferences/maintenance

Flush cache and Reset Hitcount

>> Method flushCache for action maintenance is not allowed!

Possibly strict-method-invocation="true" is on, so could try to add
allowed-methods to your struts.xml for the class

<action name="maintenance"
Class="org.apache.roller.weblogger.ui.struts2.editor.Maintenance">
            <result name="success" type="tiles">.Maintenance</result>
<allowed-methods>flushCache,reset</allowed-methods>
 </action>

(btw your previous dmi would have worked, so there was no need to change
it!)

##

Cheers Greg

On 29 August 2017 at 02:10, Dave <snoopd...@gmail.com> wrote:

> I've prepared a second release candidate for Roller 5.2.0. The first RC
> depended on Struts 2.3 and this new RC-2 release depends on Struts 2.5.10.1
> and no longer uses Dynamic Method Invocation (DMI). Because of this change,
> a significant number of JSP pages were changed.
>
> I've been testing this RC-2 release locally and on rollerweblogger.org
> for the past week and so far it seems stable.
>
> Apart from the Struts upgrade, Roller 5.2.0 is primarily a bug fix release,
> with lots of fixes from Kohei.
>
> You can find the release candidate here:
>     https://dist.apache.org/repos/dist/dev/roller/roller-5.2/v5.2.0-rc-2
>
> And the list of issues resolved is here:
>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> projectId=12310906&version=12340980
>
> Please take a look at the release and vote within the next 72 hours.
>
> My vote is +1 to release.
>
> Thanks,
> Dave
>

Reply via email to