Issue Details (XML | Word | Printable)

Key: JAROL-1
Type: Bug Bug
Status: Open Open
Priority: Trivial Trivial
Assignee: Unassigned
Reporter: Jason Thrasher
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
JCaptcha Roller intégration

update to Roller 2.3 package names and features

Created: 22/Oct/06 05:12 AM   Updated: 10/Sep/09 09:33 PM
Component/s: None
Affects Version/s: None
Fix Version/s: None

Environment: Linux, Tomcat 5.5.17, Sun JDK 1.5, Roller v2.3


 Description  « Hide
Roller's 2.3 release included package name changes, and a minor infrastructure change that cause JCaptcha integration to fail. Two classes need minor patches to fix it. I've outlined the patch here: http://www.coachthrasher.com/roller/page/blog?entry=roller_2_3_upgrade_notes but figure JIRA is the proper place for this info. So...

The old Roller package name is:
org.roller.*
The new, Apache incubator name is:
org.apache.roller.*

edit:
com.octo.captcha.module.roller.JCaptchaCommentAuthenticator.java

change this:
String challengeUrl = context.get("ctxPath") + "/jcaptcha.do";
to this:
String challengeUrl = org.apache.roller.presentation.RollerContext.getRollerContext().getAbsoluteContextUrl() + "/jcaptcha.do";
and update imports as needed to reflect the org.apache.roller package name

edit:
com.octo.captcha.service.captchastore.EhcacheCaptchaStore.java

change this (line 163):
} catch (IOException e) {
to this:
} catch (Exception e) {
to fix a compile error... actually, that was my hack and the proper exception should be caught, if any.

Last, the documentation needs to be updated here:
http://forge.octo.com/jcaptcha/confluence/display/general/JRoller+installation
Instead of editing the web.xml file, Roller 2.3 users need to create this:
WEB-INF/classes/roller-custom.properties
with this line:
comment.authenticator.classname=com.octo.captcha.module.roller.JCaptchaCommentAuthenticator


Sort Order: Ascending order - Click to sort in descending order
Andrew Norman added a comment - 10/Sep/09 09:33 PM
actually the documentation also needs to be updated update the struts-plugins.xml file for the plugins entry and struts-actions for the action path entry.