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

Richard Rodgers commented on DS-1082:
-------------------------------------

Thinking on it further, maybe a better approach would be to define a 'default 
embargo period' (configurable?) that would be applied when the terms were 
invalid. That way, the cautionary principle would be honored (it would be 
safely embargoed at least for a while) - perhaps combined with a notification 
to give admins a chance to correct. Just pondering...
                
> Invalid embargo causes item archival to crash.
> ----------------------------------------------
>
>                 Key: DS-1082
>                 URL: https://jira.duraspace.org/browse/DS-1082
>             Project: DSpace
>          Issue Type: Bug
>            Reporter: Kevin Van de Velde
>            Assignee: Kevin Van de Velde
>             Fix For: 1.8.1
>
>         Attachments: embargo-fix.patch
>
>
> When an item is archived & the embargo date is invalid (either an invalid 
> date has been entered as embargo date or the embargo date has already passed) 
> the item cannot be archived since the embargo manager will throw an invalid 
> argument exception.
> In the attached patch is a possible solution for this problem (catching the 
> exception & logging the error), however I wonder should an email be sent to 
> alert the admin that an invalid embargo was entered ?

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

        

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
Dspace-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dspace-devel

Reply via email to