Passed
Pull Request — master (#39)
by Kiran
03:58
created
includes/wpinv-invoice-functions.php 1 patch
Braces   +21 added lines, -16 removed lines patch added patch discarded remove patch
@@ -83,8 +83,9 @@  discard block
 block discarded – undo
83 83
 
84 84
     $invoice_id = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_wpinv_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
85 85
 
86
-    if ( $invoice_id != NULL )
87
-        return $invoice_id;
86
+    if ( $invoice_id != NULL ) {
87
+            return $invoice_id;
88
+    }
88 89
 
89 90
     return 0;
90 91
 }
@@ -750,8 +751,9 @@  discard block
 block discarded – undo
750 751
 function wpinv_checkout_validate_cc_zip( $zip = 0, $country_code = '' ) {
751 752
     $ret = false;
752 753
 
753
-    if ( empty( $zip ) || empty( $country_code ) )
754
-        return $ret;
754
+    if ( empty( $zip ) || empty( $country_code ) ) {
755
+            return $ret;
756
+    }
755 757
 
756 758
     $country_code = strtoupper( $country_code );
757 759
 
@@ -913,8 +915,9 @@  discard block
 block discarded – undo
913 915
         "ZM" => "\d{5}"
914 916
     );
915 917
 
916
-    if ( ! isset ( $zip_regex[ $country_code ] ) || preg_match( "/" . $zip_regex[ $country_code ] . "/i", $zip ) )
917
-        $ret = true;
918
+    if ( ! isset ( $zip_regex[ $country_code ] ) || preg_match( "/" . $zip_regex[ $country_code ] . "/i", $zip ) ) {
919
+            $ret = true;
920
+    }
918 921
 
919 922
     return apply_filters( 'wpinv_is_zip_valid', $ret, $zip, $country_code );
920 923
 }
@@ -1238,14 +1241,15 @@  discard block
 block discarded – undo
1238 1241
         }
1239 1242
     }
1240 1243
     
1241
-    if ( get_query_var( 'paged' ) )
1242
-        $args['page'] = get_query_var('paged');
1243
-    else if ( get_query_var( 'page' ) )
1244
-        $args['page'] = get_query_var( 'page' );
1245
-    else if ( !empty( $args[ 'page' ] ) )
1246
-        $args['page'] = $args[ 'page' ];
1247
-    else
1248
-        $args['page'] = 1;
1244
+    if ( get_query_var( 'paged' ) ) {
1245
+            $args['page'] = get_query_var('paged');
1246
+    } else if ( get_query_var( 'page' ) ) {
1247
+            $args['page'] = get_query_var( 'page' );
1248
+    } else if ( !empty( $args[ 'page' ] ) ) {
1249
+            $args['page'] = $args[ 'page' ];
1250
+    } else {
1251
+            $args['page'] = 1;
1252
+    }
1249 1253
 
1250 1254
     /**
1251 1255
      * Generate WP_Query args. This logic will change if orders are moved to
@@ -1380,8 +1384,9 @@  discard block
 block discarded – undo
1380 1384
 
1381 1385
 	$invoice_id = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_wpinv_key' AND meta_value = %s LIMIT 1", $key ) );
1382 1386
 
1383
-	if ( $invoice_id != NULL )
1384
-		return $invoice_id;
1387
+	if ( $invoice_id != NULL ) {
1388
+			return $invoice_id;
1389
+	}
1385 1390
 
1386 1391
 	return 0;
1387 1392
 }
Please login to merge, or discard this patch.
templates/emails/wpinv-email-completed_invoice.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 block discarded – undo
1 1
 <?php
2 2
 // don't load directly
3
-if ( !defined('ABSPATH') )
3
+if ( !defined('ABSPATH') ) {
4 4
     die('-1');
5
+}
5 6
 
6 7
 do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
7 8
 ?>
Please login to merge, or discard this patch.
templates/emails/wpinv-email-failed_invoice.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 block discarded – undo
1 1
 <?php
2 2
 // don't load directly
3
-if ( !defined('ABSPATH') )
3
+if ( !defined('ABSPATH') ) {
4 4
     die('-1');
5
+}
5 6
 
6 7
 do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
7 8
 ?>
Please login to merge, or discard this patch.
templates/emails/wpinv-email-onhold_invoice.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 block discarded – undo
1 1
 <?php
2 2
 // don't load directly
3
-if ( !defined('ABSPATH') )
3
+if ( !defined('ABSPATH') ) {
4 4
     die('-1');
5
+}
5 6
 
6 7
 do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
7 8
 ?>
Please login to merge, or discard this patch.
templates/emails/wpinv-email-user_note.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 block discarded – undo
1 1
 <?php
2 2
 // don't load directly
3
-if ( !defined('ABSPATH') )
3
+if ( !defined('ABSPATH') ) {
4 4
     die('-1');
5
+}
5 6
 
6 7
 do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
7 8
 
Please login to merge, or discard this patch.
templates/emails/wpinv-email-cancelled_invoice.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 block discarded – undo
1 1
 <?php
2 2
 // don't load directly
3
-if ( !defined('ABSPATH') )
3
+if ( !defined('ABSPATH') ) {
4 4
     die('-1');
5
+}
5 6
 
6 7
 do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
7 8
 ?>
Please login to merge, or discard this patch.
templates/emails/wpinv-email-overdue.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 block discarded – undo
1 1
 <?php
2 2
 // don't load directly
3
-if ( !defined('ABSPATH') )
3
+if ( !defined('ABSPATH') ) {
4 4
     die('-1');
5
+}
5 6
 
6 7
 do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
7 8
 
Please login to merge, or discard this patch.
templates/emails/wpinv-email-new_invoice.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 block discarded – undo
1 1
 <?php
2 2
 // don't load directly
3
-if ( !defined('ABSPATH') )
3
+if ( !defined('ABSPATH') ) {
4 4
     die('-1');
5
+}
5 6
 
6 7
 do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
7 8
 ?>
Please login to merge, or discard this patch.
includes/wpinv-user-functions.php 1 patch
Braces   +13 added lines, -9 removed lines patch added patch discarded remove patch
@@ -1,6 +1,8 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 // Exit if accessed directly
3
-if ( ! defined( 'ABSPATH' ) ) exit;
3
+if ( ! defined( 'ABSPATH' ) ) {
4
+    exit;
5
+}
4 6
 
5 7
 function wpinv_user_invoices( $current_page = 1 ) {    
6 8
     global $current_page;
@@ -17,12 +19,13 @@  discard block
 block discarded – undo
17 19
     }
18 20
 
19 21
     if ( $pagination ) {
20
-        if ( get_query_var( 'paged' ) )
21
-            $paged = get_query_var('paged');
22
-        else if ( get_query_var( 'page' ) )
23
-            $paged = get_query_var( 'page' );
24
-        else
25
-            $paged = 1;
22
+        if ( get_query_var( 'paged' ) ) {
23
+                    $paged = get_query_var('paged');
24
+        } else if ( get_query_var( 'page' ) ) {
25
+                    $paged = get_query_var( 'page' );
26
+        } else {
27
+                    $paged = 1;
28
+        }
26 29
     }
27 30
 
28 31
     $args = array(
@@ -37,8 +40,9 @@  discard block
 block discarded – undo
37 40
     $invoices = get_posts( $args );
38 41
 
39 42
     // No invoices
40
-    if ( ! $invoices )
41
-        return false;
43
+    if ( ! $invoices ) {
44
+            return false;
45
+    }
42 46
 
43 47
     return $invoices;
44 48
 }
Please login to merge, or discard this patch.