Updating POD documentation for Bugzilla/Error to suggest the use of

'abort' after passing a few minutes looking for it. rs=joel

1 files changed, 9 insertions, 4 deletions

diff --git a/Bugzilla/Error.pm b/Bugzilla/Error.pm
index be7f87add..548cbb24c 100644
--- a/Bugzilla/Error.pm
+++ b/Bugzilla/Error.pm
@@ -136,6 +136,10 @@ Bugzilla::Error - Error handling utilities for Bugzilla
 
   ThrowUserError("error_tag",
                  { foo => 'bar' });
+ 
+  # supplying "abort" to ensure tables are unlocked
+  ThrowUserError("another_error_tag",
+                 { foo => 'bar' }, 'abort');
 
 =head1 DESCRIPTION
 
@@ -154,10 +158,11 @@ of variables as a second argument. These are used by the
 I<global/user-error.html.tmpl> template to format the error, using the passed
 in variables as required.
 
-An optional third argument may be supplied. If present (and defined), then the
-error handling code will unlock the database tables. In the long term, this
-argument will go away, to be replaced by transactional C<rollback> calls. There
-is no timeframe for doing so, however.
+An optional third argument may be supplied. If present, the error
+handling code will unlock the database tables: it is a Bugzilla standard
+to provide the string "abort" as the argument value. In the long term,
+this argument will go away, to be replaced by transactional C<rollback>
+calls. There is no timeframe for doing so, however.
 
 =item C<ThrowCodeError>