Passed
Push — master ( f14fa1...c29029 )
by Greg
02:55
created
settings/display/index.php 1 patch
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -51,10 +51,18 @@  discard block
 block discarded – undo
51 51
 
52 52
 if ($_SERVER['REQUEST_METHOD'] == 'POST') {
53 53
 
54
-    if (is_null($domain_column_options)) $domain_column_options = array('');
55
-    if (is_null($custom_domain_fields)) $custom_domain_fields = array('');
56
-    if (is_null($custom_ssl_fields)) $custom_ssl_fields = array('');
57
-    if (is_null($ssl_column_options)) $ssl_column_options = array('');
54
+    if (is_null($domain_column_options)) {
55
+        $domain_column_options = array('');
56
+    }
57
+    if (is_null($custom_domain_fields)) {
58
+        $custom_domain_fields = array('');
59
+    }
60
+    if (is_null($custom_ssl_fields)) {
61
+        $custom_ssl_fields = array('');
62
+    }
63
+    if (is_null($ssl_column_options)) {
64
+        $ssl_column_options = array('');
65
+    }
58 66
 
59 67
     foreach ($_SESSION['s_cdf_data'] as $field) {
60 68
 
@@ -214,8 +222,12 @@  discard block
 block discarded – undo
214 222
 
215 223
     if ($_SERVER['REQUEST_METHOD'] == 'POST') {
216 224
 
217
-        if ($new_number_of_domains === 0) $_SESSION['s_message_danger'] .= "Enter the default number of domains to display<BR>";
218
-        if ($new_number_of_ssl_certs === 0) $_SESSION['s_message_danger'] .= "Enter the default number of SSL certficates to display<BR>";
225
+        if ($new_number_of_domains === 0) {
226
+            $_SESSION['s_message_danger'] .= "Enter the default number of domains to display<BR>";
227
+        }
228
+        if ($new_number_of_ssl_certs === 0) {
229
+            $_SESSION['s_message_danger'] .= "Enter the default number of SSL certficates to display<BR>";
230
+        }
219 231
 
220 232
     } else {
221 233
 
Please login to merge, or discard this patch.
classes/DomainMOD/Sanitize.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 class Sanitize
25 25
 {
26 26
 
27
-   public function text($text)
27
+    public function text($text)
28 28
     {
29 29
         $sanitize = new \GJClasses\Sanitize();
30 30
         return $sanitize->text($text);
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
 {
26 26
 
27 27
    public function text($text)
28
-    {
28
+   {
29 29
         $sanitize = new \GJClasses\Sanitize();
30 30
         return $sanitize->text($text);
31 31
     }
Please login to merge, or discard this patch.
classes/DomainMOD/Unsanitize.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 class Unsanitize
25 25
 {
26 26
 
27
-   public function text($text)
27
+    public function text($text)
28 28
     {
29 29
         $unsanitize = new \GJClasses\Unsanitize();
30 30
         return $unsanitize->text($text);
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
 {
26 26
 
27 27
    public function text($text)
28
-    {
28
+   {
29 29
         $unsanitize = new \GJClasses\Unsanitize();
30 30
         return $unsanitize->text($text);
31 31
     }
Please login to merge, or discard this patch.
classes/DomainMOD/Email.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -188,7 +188,9 @@
 block discarded – undo
188 188
             return array($domains_expiring, $ssl_expiring);
189 189
         } else {
190 190
             $_SESSION['s_message_success'] .= 'No Upcoming Expirations<BR>';
191
-            if ($from_cron === true) exit;
191
+            if ($from_cron === true) {
192
+                exit;
193
+            }
192 194
             return false;
193 195
         }
194 196
     }
Please login to merge, or discard this patch.
bulk/index.php 1 patch
Braces   +24 added lines, -8 removed lines patch added patch discarded remove patch
@@ -201,13 +201,27 @@  discard block
 block discarded – undo
201 201
 
202 202
                 if (!$date->checkDateFormat($new_expiry_date) || $new_pcid === 0 || $new_dnsid === 0 || $new_ipid === 0 || $new_whid === 0 || $new_raid === 0 || $has_existing_domains == '1') {
203 203
 
204
-                    if ($has_existing_domains == '1') $_SESSION['s_message_danger'] .= "At least one of the domains you entered already exists in " . SOFTWARE_TITLE . ".<BR><BR>You should run the domain list through a Segment filter to determine which one(s).<BR>";
205
-                    if (!$date->checkDateFormat($new_expiry_date)) $_SESSION['s_message_danger'] .= "You have entered an invalid expiry date<BR>";
206
-                    if ($new_pcid === 0) $_SESSION['s_message_danger'] .= "Please choose the new Category<BR>";
207
-                    if ($new_dnsid === 0) $_SESSION['s_message_danger'] .= "Please choose the new DNS Profile<BR>";
208
-                    if ($new_ipid === 0) $_SESSION['s_message_danger'] .= "Please choose the new IP Address<BR>";
209
-                    if ($new_whid === 0) $_SESSION['s_message_danger'] .= "Please choose the new Web Hosting Provider<BR>";
210
-                    if ($new_raid === 0) $_SESSION['s_message_danger'] .= "Please choose the new Registrar Account<BR>";
204
+                    if ($has_existing_domains == '1') {
205
+                        $_SESSION['s_message_danger'] .= "At least one of the domains you entered already exists in " . SOFTWARE_TITLE . ".<BR><BR>You should run the domain list through a Segment filter to determine which one(s).<BR>";
206
+                    }
207
+                    if (!$date->checkDateFormat($new_expiry_date)) {
208
+                        $_SESSION['s_message_danger'] .= "You have entered an invalid expiry date<BR>";
209
+                    }
210
+                    if ($new_pcid === 0) {
211
+                        $_SESSION['s_message_danger'] .= "Please choose the new Category<BR>";
212
+                    }
213
+                    if ($new_dnsid === 0) {
214
+                        $_SESSION['s_message_danger'] .= "Please choose the new DNS Profile<BR>";
215
+                    }
216
+                    if ($new_ipid === 0) {
217
+                        $_SESSION['s_message_danger'] .= "Please choose the new IP Address<BR>";
218
+                    }
219
+                    if ($new_whid === 0) {
220
+                        $_SESSION['s_message_danger'] .= "Please choose the new Web Hosting Provider<BR>";
221
+                    }
222
+                    if ($new_raid === 0) {
223
+                        $_SESSION['s_message_danger'] .= "Please choose the new Registrar Account<BR>";
224
+                    }
211 225
 
212 226
                     $submission_failed = 1;
213 227
 
@@ -1937,7 +1951,9 @@  discard block
 block discarded – undo
1937 1951
 
1938 1952
     // Casting $new_active as int sets it to 0 on first load, which sets the default status to 'Expired'. The below
1939 1953
     // line sets the default to 'Active' instead.
1940
-    if ($action == "AD") $new_active = 1;
1954
+    if ($action == "AD") {
1955
+        $new_active = 1;
1956
+    }
1941 1957
 
1942 1958
 }
1943 1959
 ?>
Please login to merge, or discard this patch.