lagotto / alm-report

ALM Reports
http://almreports.plos.org/
MIT License
8 stars 3 forks source link

Handle session without dois #42

Closed mfenner closed 10 years ago

mfenner commented 10 years ago

Instead of raising an error we should redirect to the home page for a new search when a session doesn't contain at least one DOI. This closes #34.

jure commented 10 years ago

Since generate is the only action that needs dois in the session it's OK like this, otherwise it would make sense to have a before filter. It would just add cruft in this case though.

Looks good, merging.