From b29c86632dffec6c5e063867eb0876f5c2b9d650 Mon Sep 17 00:00:00 2001
From: "mkanat%bugzilla.org" <>
Date: Wed, 18 Nov 2009 07:11:36 +0000
Subject: Bug 521373: Hook: object-before_create

---
 Bugzilla/Hook.pm   | 20 ++++++++++++++++++++
 Bugzilla/Object.pm |  6 ++++++
 2 files changed, 26 insertions(+)

(limited to 'Bugzilla')

diff --git a/Bugzilla/Hook.pm b/Bugzilla/Hook.pm
index 69fa2f654..324576594 100644
--- a/Bugzilla/Hook.pm
+++ b/Bugzilla/Hook.pm
@@ -585,6 +585,26 @@ Params:
 
 =back
 
+=head2 object-before_create
+
+This happens at the beginning of L<Bugzilla::Object/create>.
+
+Params:
+
+=over
+
+=item C<class>
+
+The name of the class that C<create> was called on. You can check this 
+like C<< if ($class->isa('Some::Class')) >> in your code, to perform specific
+tasks before C<create> for only certain classes.
+
+=item C<params>
+
+A hashref. The set of named parameters passed to C<create>.
+
+=back
+
 =head2 page-before_template
 
 This is a simple way to add your own pages to Bugzilla. This hooks C<page.cgi>,
diff --git a/Bugzilla/Object.pm b/Bugzilla/Object.pm
index b04593f89..d99c17a63 100644
--- a/Bugzilla/Object.pm
+++ b/Bugzilla/Object.pm
@@ -24,6 +24,7 @@ use strict;
 package Bugzilla::Object;
 
 use Bugzilla::Constants;
+use Bugzilla::Hook;
 use Bugzilla::Util;
 use Bugzilla::Error;
 
@@ -399,6 +400,11 @@ sub _check_field {
 sub check_required_create_fields {
     my ($class, $params) = @_;
 
+    # This hook happens here so that even subclasses that don't call
+    # SUPER::create are still affected by the hook.
+    Bugzilla::Hook::process('object-before_create', { class => $class,
+                                                      params => $params });
+
     foreach my $field ($class->REQUIRED_CREATE_FIELDS) {
         ThrowCodeError('param_required',
             { function => "${class}->create", param => $field })
-- 
cgit v1.2.3-24-g4f1b