[ 
https://jira.duraspace.org/browse/DS-690?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=19179#action_19179
 ] 

Kim Shepherd commented on DS-690:
---------------------------------

Hi Andrea, thanks for the comment -- I have patch that I'll attach shortly. It 
was a bit tricky to ensure that it 

* Doesn't interfere with the old stats reports that are already associated with 
/jspui/statistics 
* Looks and feels the same as xmlui paths, eg. /jspui/handle/foo/bar/statistics 
* Can still work the legacy way, eg. /jspui/displaystats?handle=foo/bar

But I think I've sussed it! ;) Would be great if you could help me test it.

-k.

> Tidy up URL mapping for DisplayStatisticsServlet (JSPUI servlet that handles 
> solr statistics)
> ---------------------------------------------------------------------------------------------
>
>                 Key: DS-690
>                 URL: https://jira.duraspace.org/browse/DS-690
>             Project: DSpace
>          Issue Type: Improvement
>          Components: JSPUI
>            Reporter: Kim Shepherd
>            Assignee: Kim Shepherd
>             Fix For: 1.8.0
>
>
> The XMLUI URL mapping for solr statistics is nice and tidy, eg.
> ${contextPath}/handle/123456789/1/statistics
> Whereas the JSPUI URL mapping is not as nice to look at, nor is it as 
> functional:
> ${contextPath}/displaystats?handle=123456789/1
> It would help with crawler/sitemap maintenance, path-based authZ, DSO 
> resolution, etc. if it was in line with the XMLUI version. My fault. (Mark 
> did point this out to me at one stage, too..)
> I'd like to submit a patch for this in time for 1.7, but I wonder whether 
> there could be some fallout with researchers, admins, users who have already 
> bookmarked older stats pages -- either way, it's better to get the change in 
> earlier then later, but I should also make a note in the documentation and 
> maybe advertise the change in mailing lists ahead of time.
> I'll put 1.7.0 as a fix version and will work on a patch, but will wait for 
> IRC vote/comments before committing code or resolving this issue.
> Cheers,
> Kim

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://jira.duraspace.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
What You Don't Know About Data Connectivity CAN Hurt You
This paper provides an overview of data connectivity, details
its effect on application quality, and explores various alternative
solutions. http://p.sf.net/sfu/progress-d2d
_______________________________________________
Dspace-devel mailing list
Dspace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-devel

Reply via email to