From 66e373d115707a31506abb62b95425555e94435b Mon Sep 17 00:00:00 2001 From: Mahdi Mokhtari Date: Tue, 5 Jan 2016 15:04:39 +0100 Subject: Bug 1235270: Set submitter_id before calling _check_data() r=LpSolit --- Bugzilla/Attachment.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'Bugzilla/Attachment.pm') diff --git a/Bugzilla/Attachment.pm b/Bugzilla/Attachment.pm index ed43cbe94..1f2bddda5 100644 --- a/Bugzilla/Attachment.pm +++ b/Bugzilla/Attachment.pm @@ -903,6 +903,8 @@ sub create { sub run_create_validators { my ($class, $params) = @_; + $params->{submitter_id} = Bugzilla->user->id || ThrowUserError('invalid_user'); + # Let's validate the attachment content first as it may # alter some other attachment attributes. $params->{data} = $class->_check_data($params); @@ -910,7 +912,6 @@ sub run_create_validators { $params->{creation_ts} ||= Bugzilla->dbh->selectrow_array('SELECT LOCALTIMESTAMP(0)'); $params->{modification_time} = $params->{creation_ts}; - $params->{submitter_id} = Bugzilla->user->id || ThrowUserError('invalid_user'); return $params; } -- cgit v1.2.3-24-g4f1b