click here for details... Sakai Executive Director Position Search now open
Issue Details (XML | Word | Printable)

Key: SAK-11214
Type: Task Task
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Nuno Fernandes
Votes: 0
Watchers: 3
Operations

If you were logged in you would be able to see more operations.
Sakai

Allow enable/disable of wiki reads events through sakai.properties

Created: 21-Aug-2007 03:23   Updated: 27-Oct-2009 13:40
Component/s: Rwiki
Affects Version/s: 2.4.0, 2.4.1, 2.4.x, 2.5.0, 2.5.2
Fix Version/s: 2.6.0

Time Tracking:
Not Specified

Environment: Any
Issue Links:
Relate
 

2.6.x Status: None
2.5.x Status: Won't Fix
2.4.x Status: None


 Description  « Hide
Add more control of the setting "trackReads" present in /rwiki-tool/tool/src/webapp/WEB-INF/commandComponents.xml, allowing it to be set from sakai.properties.

Related: SAK-6851

 All   Comments   Work Log   Change History   Subversion Commits   git Commits      Sort Order: Ascending order - Click to sort in descending order
Ian Boston added a comment - 23-Apr-2008 05:16
Fixed added

wiki.trackreads=true

for sakai.properties



Ian Boston added a comment - 23-Apr-2008 05:18
Please make certain this gets into the documentation

thanks

Owen McGrath added a comment - 19-Jun-2008 16:32
I've been testing this in our 2.5 deployment and am getting the impression so far that the sakai.properties setting doesn't do anything.

 I'm finding that the reads only get logged if the property is set in the commandComponents.xml file.



Nuno Fernandes added a comment - 20-Jun-2008 00:45
Owen, please note that this is fixed in trunk only (for 2.5 this must be set in commandComponents.xml). Is it failing for trunk, also? Thanks

Owen McGrath added a comment - 21-Jun-2008 11:11
Sorry, my mistake. We don't this fix yet.

Owen McGrath added a comment - 13-Aug-2008 13:19
Any chance this code be added to 2.5.x?

Owen McGrath added a comment - 13-Aug-2008 13:20
Could this be added to 2.5.x?

David Horwitz added a comment - 15-Aug-2008 04:04
Is the current behaviour preserved by this change? i.e. will including it in 2-5-x affect OTB behaviour for people on the branch

Stephen Marquard added a comment - 15-Aug-2008 05:06
From looking at the code this defaults to false, which is the current position in 2-5-x (i.e. we're not getting wiki.read events in our 2-5-x build), so it seems to retain the current behaviour.

Megan May added a comment - 22-Sep-2008 05:16
While this doesn't change the default behavior, this seems to be a change that doesn't address a bug.

Jean-François Lévêque added a comment - 03-Dec-2008 06:22
Setting as 2.5.x Status as Won't Fix based on the "no new features" policy and Megan's comment.

Could you provide a 2.5.x patch for Owen, Nuno?

Nuno Fernandes added a comment - 03-Dec-2008 07:07
I applied the patch generated from this issue commit to our production server replica (2.5.x) with "wiki.trackreads=true" in sakai.properties but, when I access a wiki page, "wiki.read" is not logged to SAKAI_EVENT and:

- for existing wiki pages, an email is automatically sent as in the case a page has been modified;
- on new (non-existing) pages, the stacktrace below is sent to catalina.out.


WARN: postEvent, notifyObservers(), event: 0:wiki.read@/wiki/site/test-site-28-12-2005/outra pagina.[m, 3] (2008-12-03 14:49:21,772 http-8080-Processor23_org.sakaiproject.event.impl.ClusterEventTracking)
java.lang.NumberFormatException: null
at java.lang.Long.parseLong(Long.java:372)
at java.lang.Long.parseLong(Long.java:461)
at uk.ac.cam.caret.sakai.rwiki.component.service.impl.SiteEmailNotificationRWiki.getMessageContent(SiteEmailNotificationRWiki.java:243)
at uk.ac.cam.caret.sakai.rwiki.component.service.impl.SiteEmailNotificationRWiki.getPlainMessage(SiteEmailNotificationRWiki.java:179)
at uk.ac.cam.caret.sakai.rwiki.component.service.impl.SiteEmailNotificationRWiki.plainTextContent(SiteEmailNotificationRWiki.java:263)
at org.sakaiproject.util.EmailNotification.notify(EmailNotification.java:198)
at org.sakaiproject.event.impl.BaseNotificationService$BaseNotification.notify(BaseNotificationService.java:1094)
at org.sakaiproject.event.impl.BaseNotificationService.update(BaseNotificationService.java:607)
at java.util.Observable.notifyObservers(Observable.java:142)
at org.sakaiproject.event.impl.BaseEventTrackingService.notifyObservers(BaseEventTrackingService.java:119)
at org.sakaiproject.event.impl.ClusterEventTracking.postEvent(ClusterEventTracking.java:255)
at org.sakaiproject.event.impl.BaseEventTrackingService.post(BaseEventTrackingService.java:239)
at uk.ac.cam.caret.sakai.rwiki.tool.service.impl.CommandServiceImpl$DefaultCommand.execute(CommandServiceImpl.java:114)
at uk.ac.cam.caret.sakai.rwiki.tool.RWikiServlet.execute(RWikiServlet.java:186)
at uk.ac.cam.caret.sakai.rwiki.tool.RWikiServlet.doGet(RWikiServlet.java:117)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:690)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:803)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:269)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:188)
at org.sakaiproject.util.RequestFilter.doFilter(RequestFilter.java:555)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:215)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:188)
at org.apache.catalina.core.ApplicationDispatcher.invoke(ApplicationDispatcher.java:691)
at org.apache.catalina.core.ApplicationDispatcher.processRequest(ApplicationDispatcher.java:469)
at org.apache.catalina.core.ApplicationDispatcher.doForward(ApplicationDispatcher.java:364)
at org.apache.catalina.core.ApplicationDispatcher.forward(ApplicationDispatcher.java:301)
at org.sakaiproject.tool.impl.ActiveToolComponent$MyActiveTool.forward(ActiveToolComponent.java:459)
at org.sakaiproject.portal.charon.SkinnableCharonPortal.forwardTool(SkinnableCharonPortal.java:1344)
at org.sakaiproject.portal.charon.handlers.ToolHandler.doTool(ToolHandler.java:163)
at org.sakaiproject.portal.charon.handlers.ToolHandler.doGet(ToolHandler.java:86)
at org.sakaiproject.portal.charon.SkinnableCharonPortal.doGet(SkinnableCharonPortal.java:892)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:690)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:803)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:269)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:188)
at org.sakaiproject.util.RequestFilter.doFilter(RequestFilter.java:592)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:215)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:188)
at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:210)
at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:174)
at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:127)
at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:117)
at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:108)
at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:151)
at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:870)
at org.apache.coyote.http11.Http11BaseProtocol$Http11ConnectionHandler.processConnection(Http11BaseProtocol.java:665)
at org.apache.tomcat.util.net.PoolTcpEndpoint.processSocket(PoolTcpEndpoint.java:528)
at org.apache.tomcat.util.net.LeaderFollowerWorkerThread.runIt(LeaderFollowerWorkerThread.java:81)
at org.apache.tomcat.util.threads.ThreadPool$ControlRunnable.run(ThreadPool.java:685)
at java.lang.Thread.run(Thread.java:595)


Jean-François Lévêque added a comment - 04-Dec-2008 01:23
Sorry, Nuno. I thought you might already have tried it.

Does anybody else have an idea?

Has anybody tried wiki.trackreads=true with 2.6?

Nuno Fernandes added a comment - 04-Dec-2008 01:36
wow....

Same issue, with one difference: "wiki.read" is actually logged to SAKAI_EVENT in all cases.
So, if the page exists, accessing it in wiki causes a notification email to be sent; if the page is new (does not exist), the stacktrace above is presented.

Should this issue be re-opened or a new one created?!?!




Jean-François Lévêque added a comment - 04-Dec-2008 01:40
If your last comment is the result of 2.6 testing, reopening is a good idea.

Nuno Fernandes added a comment - 04-Dec-2008 02:07
This is the result of trunk (updated) testing - I assume this is the case of 2.6.x as well. I currently don't have a 2.6.x checked out - I will check it out and test this against 2.6.x.

Stephen Marquard added a comment - 04-Dec-2008 05:14
New JIRA created for the above issues: SAK-11214