Ticket #1329 (closed defect: fixed)

Opened 14 months ago

Last modified 10 months ago

Null Pointer Exception on browseVolume when no existing volumes

Reported by: npeterson Owned by: npeterson
Priority: medium Milestone:
Component: ambra Version: 0.9.3
Keywords: Cc:

Description

NPE on PLoS Biology because there are no existing volumes. Maybe we should trap nicely in browseVolume?

2009-06-08 11:36:37,175 ERROR UnhandledException?(PLoSBiology)> [http-8080-Processor14 org.topazproject.ambra.UnhandledException?] java.lang.NullPointerException?

at

org.topazproject.ambra.article.service.BrowseService?.getIssueInfo2(Brows eService.java:302)

at

org.topazproject.ambra.article.service.BrowseService?.access$500(BrowseSe? rvice.java:72)

at

org.topazproject.ambra.article.service.BrowseService?$4.lookup(BrowseServ? ice.java:288)

at

org.topazproject.ambra.article.service.BrowseService?$4.lookup(BrowseServ? ice.java:287)

at

org.topazproject.ambra.cache.EhcacheProvider?$1.execute(EhcacheProvider?.j ava:140)

at

org.topazproject.ambra.cache.Cache$SynchronizedLookup?.execute(Cache.java :231)

at

org.topazproject.ambra.cache.EhcacheProvider?.get(EhcacheProvider?.java:13 0) ... ...

Change History

Changed 14 months ago by ssterling

  • status changed from new to assigned
  • owner changed from ssterling to wtoconnor

Changed 14 months ago by npeterson

  • owner changed from wtoconnor to ssterling

Changed 14 months ago by rcave

  • milestone changed from 0.9.3_rc2 to 0.9.3_rc3

Changed 13 months ago by ssterling

  • status changed from assigned to verify
  • owner changed from ssterling to npeterson
  • resolution set to fixed

(In [7759])For the Browse Volume action, eliminated the null pointer exception when there is no volume.

It is not anticipated that this action will be accessed for any Journal which has no volumes and no issues so, in either of those cases, a relatively sparse page is displayed. The user no longer sees the Site Error page.

Also revamped the logic for default display of volume and/or issue for both the Browse Volume and Browse Issue actions.

Addresses # 1329

Changed 12 months ago by npeterson

  • milestone 0.9.3_rc3 deleted

Changed 10 months ago by npeterson

  • status changed from verify to closed
Note: See TracTickets for help on using tickets.