From: Mahdi Mokhtari <mokhi64@gmail.com>
Date: Tue, 5 Jan 2016 14:04:39 +0000 (+0100)
Subject: Bug 1235270: Set submitter_id before calling _check_data()
X-Git-Tag: release-5.1.1~113
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=66e373d115707a31506abb62b95425555e94435b;p=thirdparty%2Fbugzilla.git

Bug 1235270: Set submitter_id before calling _check_data()
r=LpSolit
---

diff --git a/Bugzilla/Attachment.pm b/Bugzilla/Attachment.pm
index ed43cbe94d..1f2bddda51 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;
 }