Re: [Dspace-tech] Filter-media in 15

[Graham Triggs]

Larry Stone wrote:
 It _should_ never get an NPE over a missing configuration key.  Given
 our sparse testing resources it behooves us to make the code as
 resilient as possible.

I'll disagree slightly - it isn't necessarily wrong for code to blow up. 
In cases like this, it's probably better for the code to blow up, than 
to quietly pretend to work, with only a vigilant pair of eyes on the log 
file realising if it isn't.

But it should blow up with something more useful than an NPE - testing 
the returned value for null, or catching the NPE and rethrowing a more 
descriptive exception.

G
This email has been scanned by Postini.
For more information please visit http://www.postini.com


-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/
___
DSpace-tech mailing list
DSpace-tech@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-tech

Re: [Dspace-tech] multiple subscription notification

[Claudia Jürgen]

Hi Kyle,

Graham Trigss just fixed this bug for 1.5.1.

sunny greetings

Claudia



Kyle Kaliebe schrieb:
 Claudia,
 
 It would be easy enough to change the query in Harvest.harvest to return a
 list based on date accessioned rather than date last modified. However, I
 also leave room for the possibility that the harvest method is used by other
 classes that want the query to be by date modified. Are there any instances
 of this that you know of? To be safe, I will likely just add a new method
 modified from harvest to be used exclusively by the subscription
 notification routines.
 
 Kyle Kaliebe
 Systems Developer/Engineer
 Knowledge Bank
 University Libraries
 Ohio State University
 [EMAIL PROTECTED]
 
 -Original Message-
 From: Claudia Juergen [mailto:[EMAIL PROTECTED] 
 Sent: Friday, July 11, 2008 12:28
 To: Kyle Kaliebe
 Cc: dspace-tech@lists.sourceforge.net
 Subject: Re: [Dspace-tech] multiple subscription notification
 
 Hi Kyle,
 
 this is a known bug, the subscription date check is based on last modified
 not date accessioned, thus if you edit an item it'll be sent again.
 
 cheers
 
 Claudia
 
 Dear Dspace Tech List,



 We are having an issue where our nightly subscription email service is
 sending out new item notifications with repeating items. The most common
 pattern we have seen is that a collection's subscribers will be notified
 about new items twice in the following two days. We also have instances
 where a subscription notification will be sent out with 30+ new items
 even
 though many of the items in the list were submitted years ago and are not
 new. Any thoughts? Thanks.



 Kyle Kaliebe
 Systems Developer/Engineer
 Knowledge Bank
 University Libraries
 Ohio State University
 [EMAIL PROTECTED]



 -
 Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
 Studies have shown that voting for your favorite open source project,
 along with a healthy diet, reduces your potential for chronic lameness
 and boredom. Vote Now at

 http://www.sourceforge.net/community/cca08__
 _
 DSpace-tech mailing list
 DSpace-tech@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/dspace-tech

 
 
 
 
 -
 Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
 Studies have shown that voting for your favorite open source project,
 along with a healthy diet, reduces your potential for chronic lameness
 and boredom. Vote Now at http://www.sourceforge.net/community/cca08
 ___
 DSpace-tech mailing list
 DSpace-tech@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/dspace-tech

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/
___
DSpace-tech mailing list
DSpace-tech@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-tech

[Dspace-tech] Help with user authentication

[Rafael Henkin]

Hi,

 

A few weeks ago I sent an email asking about updating LDAP
fields when the user logged in. I've tried modifying the authenticate
method in LDAPAuthentication.java, with no success. What I've tried:

- if ldap.ldapAuthenticate returns on success, there's the
command context.setCurrentUser(eperson) and it returns SUCCESS.

- in this same block I've added lines to set the eperson
email to the ldap one, update the eperson and commit the context.

This didn't work, as it overrides the email which (by what I read in the
logs) should still be the same old one in order for the login to work.

Any suggestions? 

 

Thanks,
Rafael

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/___
DSpace-tech mailing list
DSpace-tech@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-tech