This is an automated notification sent by LCG Savannah.
It relates to:
task #13957, project CDS Invenio
==============================================================================
LATEST MODIFICATIONS of task #13957:
==============================================================================
Update of task #13957 (project cdsware):
Status: None => Done
Percent Complete: 0% => 100%
_______________________________________________________
Follow-up Comment #2:
Done in my public branch new-errors-new-url-handlers
==============================================================================
OVERVIEW of task #13957:
==============================================================================
URL:
<http://savannah.cern.ch/task/?13957>
Summary: MiscUtil: obsolete /error facility
Project: CDS Invenio
Submitted by: simko
Submitted on: 2010-02-16 14:53
Should Start On: 2010-02-16 00:00
Should be Finished on: 2010-02-16 00:00
Category: MiscUtil
Priority: 5 - Normal
Status: Done
Privacy: Public
Percent Complete: 100%
Assigned to: jmartinm
Open/Closed: Open
Discussion Lock: Any
Effort: 0.00
_______________________________________________________
The /error URL is aliased on some systems (e.g. on RHEL) in a
system-wide place like httpd.conf. This clashes with our own /error
pages. So we would have to indicate people installing Invenio on such
systems to unalias /error in httpd.conf so that it would not clash.
This is easy. However, since the times register_exception() sends
emails automatically, the error sending usually does not require
interactive human input. So most of /error calls can be obsoleted.
We can therefore try to remove /error pages completely from the
codebase, and replace their usage via register_exception(), wherever
it is possible.
If we still need to gather some manual user feedback, then we should
rather use another URL with a textarea form that would be added to
register_exception() information, liberating /error, in order to avoid
this minor unalias nuisance. Maybe `/help/feedback' (exception to
webdoc pages) or something of that kind would be nice to use for this
purpose.
_______________________________________________________
Follow-up Comments:
-------------------------------------------------------
Date: 2010-04-23 13:42 By: Javier Martin Montull <jmartinm>
Done in my public branch new-errors-new-url-handlers
-------------------------------------------------------
Date: 2010-04-19 13:01 By: Tibor Simko <simko>
Basically, one has to replace the old create_error_box() with the new
register_exception() stuff, that would be sending emails tacitly, and
then we can delete all notion of /error in the codebase everywhere.
_______________________________________________________
Carbon-Copy List:
CC Address | Comment
------------------------------------+-----------------------------
5732 | -COM-
1576 | -SUB-
==============================================================================
This item URL is:
<http://savannah.cern.ch/task/?13957>
_______________________________________________
Message sent via/by LCG Savannah
http://savannah.cern.ch/
