Inspection of "Merge pull request #169 from wpdev10/update-authne..." - AyeCode/invoicing - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( def67b...f4bb32 )

by Stiofan

created 2018-06-02 10:00 UTC

Status

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -1,10 +1,10 @@
 block discarded – undo
 <?php
 // don't load directly
-if ( !defined('ABSPATH') )
+if (!defined('ABSPATH'))
     die('-1');
 
-$email_footer = apply_filters( 'wpinv_email_footer_text', wpinv_get_option( 'email_footer_text' ) );
-$email_footer = $email_footer ? wpautop( wp_kses_post( wptexturize( $email_footer ) ) ) : '';
+$email_footer = apply_filters('wpinv_email_footer_text', wpinv_get_option('email_footer_text'));
+$email_footer = $email_footer ? wpautop(wp_kses_post(wptexturize($email_footer))) : '';
 ?>
                                                             </div>
                                                         </td>

Please login to merge, or discard this patch.

templates/emails/wpinv-email-header.php 1 patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,16 +1,16 @@  discard block
 block discarded – undo
 <?php
 // don't load directly
-if ( !defined('ABSPATH') )
+if (!defined('ABSPATH'))
     die('-1');
 
-if ( !isset( $email_heading ) ) {
+if (!isset($email_heading)) {
     global $email_heading;
 }
 ?>
 <!DOCTYPE html>
 <html dir="<?php echo is_rtl() ? 'rtl' : 'ltr'?>">
     <head>
-        <meta http-equiv="Content-Type" content="text/html; charset=<?php bloginfo( 'charset' ); ?>" />
+        <meta http-equiv="Content-Type" content="text/html; charset=<?php bloginfo('charset'); ?>" />
         <meta name="viewport" content="width=device-width, initial-scale=1">
         <meta name="robots" content="noindex,nofollow">
         <title><?php echo wpinv_get_blogname(); ?></title>
@@ -22,13 +22,13 @@  discard block
 block discarded – undo
                     <td align="center" valign="top">
                         <div id="template_header_image">
                         <?php
-                            if ( $img = wpinv_get_option( 'email_header_image' ) ) {
-                                echo '<p style="margin-top:0;"><img style="max-width:100%" src="' . esc_url( $img ) . '" alt="' . esc_attr( wpinv_get_blogname() ) . '" /></p>';
+                            if ($img = wpinv_get_option('email_header_image')) {
+                                echo '<p style="margin-top:0;"><img style="max-width:100%" src="' . esc_url($img) . '" alt="' . esc_attr(wpinv_get_blogname()) . '" /></p>';
                             }
                         ?>
                         </div>
                         <table border="0" cellpadding="0" cellspacing="0" width="100%" id="template_container">
-                            <?php if ( !empty( $email_heading ) ) { ?>
+                            <?php if (!empty($email_heading)) { ?>
                             <tr>
                                 <td align="center" valign="top">
                                     <!-- Header -->

Please login to merge, or discard this patch.

templates/emails/wpinv-email-billing-details.php 1 patch

Spacing +30 added lines, -30 removed lines patch added patch discarded remove patch

@@ -1,67 +1,67 @@
 block discarded – undo
 <?php
 // don't load directly
-if ( !defined('ABSPATH') )
+if (!defined('ABSPATH'))
     die('-1');
 
-do_action( 'wpinv_email_before_billing_details', $invoice ); ?>
+do_action('wpinv_email_before_billing_details', $invoice); ?>
 <div id="wpinv-email-billing">
-    <h3 class="wpinv-address-t"><?php echo apply_filters( 'wpinv_email_billing_title', __( 'Billing Details', 'invoicing' ) ); ?></h3>
+    <h3 class="wpinv-address-t"><?php echo apply_filters('wpinv_email_billing_title', __('Billing Details', 'invoicing')); ?></h3>
     <?php 
     $address_row = '';
-    if ( $address = $invoice->get_address() ) {
-        $address_row .= wpautop( wp_kses_post( $address ) );
+    if ($address = $invoice->get_address()) {
+        $address_row .= wpautop(wp_kses_post($address));
     }
     
     $address_fields = array();
-    if ( !empty( $invoice->city ) ) {
+    if (!empty($invoice->city)) {
         $address_fields[] = $invoice->city;
     }
     
-    $country_code = !empty( $invoice->country ) ? $invoice->country : '';
-    if ( !empty( $invoice->state ) ) {
-        $address_fields[] = wpinv_state_name( $invoice->state, $country_code );
+    $country_code = !empty($invoice->country) ? $invoice->country : '';
+    if (!empty($invoice->state)) {
+        $address_fields[] = wpinv_state_name($invoice->state, $country_code);
     }
     
-    if ( !empty( $address_fields ) ) {
-        $address_fields = implode( ", ", $address_fields );
-        $address_row .= wpautop( wp_kses_post( $address_fields ) );
+    if (!empty($address_fields)) {
+        $address_fields = implode(", ", $address_fields);
+        $address_row .= wpautop(wp_kses_post($address_fields));
     }
     
-    if ( !empty( $country_code ) ) {
-        $country = wpinv_country_name( $country_code );
-        $address_row .= wpautop( wp_kses_post( trim( $country . '(' . $country_code . ')' . ' ' . $invoice->zip ) ) );
+    if (!empty($country_code)) {
+        $country = wpinv_country_name($country_code);
+        $address_row .= wpautop(wp_kses_post(trim($country . '(' . $country_code . ')' . ' ' . $invoice->zip)));
     }
     
     ?>
     <table class="table table-bordered table-sm wpi-billing-details">
         <tbody>
-            <?php do_action( 'wpinv_email_billing_fields_first', $invoice ); ?>
+            <?php do_action('wpinv_email_billing_fields_first', $invoice); ?>
             <tr class="wpi-receipt-name">
-                <th class="text-left"><?php _e( 'Name', 'invoicing' ); ?></th>
-                <td><?php if ( $sent_to_admin && $invoice->user_id ) { ?><a href="<?php echo esc_url( add_query_arg( 'user_id', $invoice->get_user_id(), self_admin_url( 'user-edit.php' ) ) ) ;?>"><?php echo esc_html( $invoice->get_user_full_name() ); ?></a><?php } else { echo esc_html( $invoice->get_user_full_name() ); } ?></td>
+                <th class="text-left"><?php _e('Name', 'invoicing'); ?></th>
+                <td><?php if ($sent_to_admin && $invoice->user_id) { ?><a href="<?php echo esc_url(add_query_arg('user_id', $invoice->get_user_id(), self_admin_url('user-edit.php'))); ?>"><?php echo esc_html($invoice->get_user_full_name()); ?></a><?php } else { echo esc_html($invoice->get_user_full_name()); } ?></td>
             </tr>
             <tr class="wpi-receipt-email">
-                <th class="text-left"><?php _e( 'Email', 'invoicing' ); ?></th>
-                <td><?php echo $invoice->get_email() ;?></td>
+                <th class="text-left"><?php _e('Email', 'invoicing'); ?></th>
+                <td><?php echo $invoice->get_email(); ?></td>
             </tr>
-            <?php if ( $invoice->company ) { ?>
+            <?php if ($invoice->company) { ?>
             <tr class="wpi-receipt-company">
-                <th class="text-left"><?php _e( 'Company', 'invoicing' ); ?></th>
-                <td><?php echo esc_html( $invoice->company ) ;?></td>
+                <th class="text-left"><?php _e('Company', 'invoicing'); ?></th>
+                <td><?php echo esc_html($invoice->company); ?></td>
             </tr>
             <?php } ?>
             <tr class="wpi-receipt-address">
-                <th class="text-left"><?php _e( 'Address', 'invoicing' ); ?></th>
-                <td><?php echo $address_row ;?></td>
+                <th class="text-left"><?php _e('Address', 'invoicing'); ?></th>
+                <td><?php echo $address_row; ?></td>
             </tr>
-            <?php if ( $invoice->phone ) { ?>
+            <?php if ($invoice->phone) { ?>
             <tr class="wpi-receipt-phone">
-                <th class="text-left"><?php _e( 'Phone', 'invoicing' ); ?></th>
-                <td><?php echo esc_html( $invoice->phone ) ;?></td>
+                <th class="text-left"><?php _e('Phone', 'invoicing'); ?></th>
+                <td><?php echo esc_html($invoice->phone); ?></td>
             </tr>
             <?php } ?>
-            <?php do_action( 'wpinv_email_billing_fields_last', $invoice ); ?>
+            <?php do_action('wpinv_email_billing_fields_last', $invoice); ?>
         </tbody>
     </table>
 </div>
-<?php do_action( 'wpinv_email_after_billing_details', $invoice ); ?>
\ No newline at end of file
+<?php do_action('wpinv_email_after_billing_details', $invoice); ?>
\ No newline at end of file

Please login to merge, or discard this patch.

templates/wpinv-payment-processing.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -2,11 +2,11 @@
 block discarded – undo
 global $wpi_invoice;
 
 $success_page_uri = wpinv_get_success_page_uri();
-if ( !empty( $wpi_invoice ) ) {
-    $success_page_uri = add_query_arg( 'invoice_key', $wpi_invoice->get_key(), $success_page_uri );
+if (!empty($wpi_invoice)) {
+    $success_page_uri = add_query_arg('invoice_key', $wpi_invoice->get_key(), $success_page_uri);
 }
 ?>
 <div id="wpinv-payment-processing">
-    <p><?php echo wp_sprintf( __( 'Your payment is processing. This page will reload automatically in 10 seconds. If it does not, click <a href="%s">here</a>.', 'invoicing' ), $success_page_uri ); ?> <i class="fa fa-spin fa-refresh"></i></p>
+    <p><?php echo wp_sprintf(__('Your payment is processing. This page will reload automatically in 10 seconds. If it does not, click <a href="%s">here</a>.', 'invoicing'), $success_page_uri); ?> <i class="fa fa-spin fa-refresh"></i></p>
     <script type="text/javascript">setTimeout(function(){ window.location = '<?php echo $success_page_uri; ?>'; }, 10000);</script>
 </div>
\ No newline at end of file

Please login to merge, or discard this patch.

templates/wpinv-invoice-receipt.php 1 patch

Spacing +14 added lines, -14 removed lines patch added patch discarded remove patch

@@ -4,17 +4,17 @@  discard block
 block discarded – undo
  */
 global $wpinv_receipt_args;
 
-$invoice   = get_post( $wpinv_receipt_args['id'] );
+$invoice = get_post($wpinv_receipt_args['id']);
 
-if( empty( $invoice ) ) {
+if (empty($invoice)) {
     ?>
     <div class="wpinv_errors alert wpi-alert-error">
-        <?php _e( 'The specified receipt ID appears to be invalid', 'invoicing' ); ?>
+        <?php _e('The specified receipt ID appears to be invalid', 'invoicing'); ?>
     </div>
     <?php
     return;
 }
-$invoice = wpinv_get_invoice( $invoice->ID );
+$invoice = wpinv_get_invoice($invoice->ID);
 
 global $ajax_cart_details;
 $ajax_cart_details = $invoice->get_cart_details();
@@ -24,18 +24,18 @@  discard block
 block discarded – undo
 $quantities_enabled = wpinv_item_quantities_enabled();
 $use_taxes          = wpinv_use_taxes();
 $zero_tax           = !(float)$invoice->get_tax() > 0 ? true : false;
-$tax_label          = !$zero_tax && $use_taxes ? ( wpinv_prices_include_tax() ? __( '(Tax Incl.)', 'invoicing' ) : __( '(Tax Excl.)', 'invoicing' ) ) : '';
+$tax_label          = !$zero_tax && $use_taxes ? (wpinv_prices_include_tax() ? __('(Tax Incl.)', 'invoicing') : __('(Tax Excl.)', 'invoicing')) : '';
 ?>
-<?php do_action( 'wpinv_before_receipt', $invoice ); ?>
+<?php do_action('wpinv_before_receipt', $invoice); ?>
 <div class="wpinv-receipt">
-    <?php do_action( 'wpinv_receipt_start', $invoice ); ?>
-    <div class="wpinv-receipt-message"><?php _e( 'Thank you for your payment!', 'invoicing' ); ?></div>
-    <?php do_action( 'wpinv_before_receipt_details', $invoice ); ?>
+    <?php do_action('wpinv_receipt_start', $invoice); ?>
+    <div class="wpinv-receipt-message"><?php _e('Thank you for your payment!', 'invoicing'); ?></div>
+    <?php do_action('wpinv_before_receipt_details', $invoice); ?>
     <div class="wpinv-receipt-details">
-        <h3 class="wpinv-details-t"><?php echo apply_filters( 'wpinv_receipt_details_title', __( 'Invoice Details', 'invoicing' ) ); ?></h3>
-        <?php wpinv_display_invoice_details( $invoice ); ?>
+        <h3 class="wpinv-details-t"><?php echo apply_filters('wpinv_receipt_details_title', __('Invoice Details', 'invoicing')); ?></h3>
+        <?php wpinv_display_invoice_details($invoice); ?>
     </div>
-    <?php do_action( 'wpinv_after_receipt_details', $invoice ); ?>
-    <?php do_action( 'wpinv_receipt_end', $invoice ); ?>
+    <?php do_action('wpinv_after_receipt_details', $invoice); ?>
+    <?php do_action('wpinv_receipt_end', $invoice); ?>
 </div>
-<?php do_action( 'wpinv_after_receipt', $invoice ); ?>
\ No newline at end of file
+<?php do_action('wpinv_after_receipt', $invoice); ?>
\ No newline at end of file

Please login to merge, or discard this patch.

includes/admin/meta-boxes/class-mb-invoice-notes.php 1 patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -1,39 +1,39 @@
 block discarded – undo
 <?php
 // MUST have WordPress.
-if ( !defined( 'WPINC' ) ) {
-    exit( 'Do NOT access this file directly: ' . basename( __FILE__ ) );
+if (!defined('WPINC')) {
+    exit('Do NOT access this file directly: ' . basename(__FILE__));
 }
 
 class WPInv_Meta_Box_Notes {
-    public static function output( $post ) {
+    public static function output($post) {
         global $post;
 
-        $notes = wpinv_get_invoice_notes( $post->ID );
+        $notes = wpinv_get_invoice_notes($post->ID);
 
         echo '<ul class="invoice_notes">';
 
-        if ( $notes ) {
-            foreach( $notes as $note ) {
-                wpinv_get_invoice_note_line_item( $note );
+        if ($notes) {
+            foreach ($notes as $note) {
+                wpinv_get_invoice_note_line_item($note);
             }
 
         } else {
-            echo '<li>' . __( 'There are no notes yet.', 'invoicing' ) . '</li>';
+            echo '<li>' . __('There are no notes yet.', 'invoicing') . '</li>';
         }
 
         echo '</ul>';
         ?>
         <div class="add_note">
-            <h4><?php _e( 'Add note', 'invoicing' ); ?></h4>
+            <h4><?php _e('Add note', 'invoicing'); ?></h4>
             <p>
                 <textarea type="text" name="invoice_note" id="add_invoice_note" class="input-text" cols="20" rows="5"></textarea>
             </p>
             <p>
                 <select name="invoice_note_type" id="invoice_note_type">
-                    <option value=""><?php _e( 'Private note', 'invoicing' ); ?></option>
-                    <option value="customer"><?php _e( 'Note to customer', 'invoicing' ); ?></option>
+                    <option value=""><?php _e('Private note', 'invoicing'); ?></option>
+                    <option value="customer"><?php _e('Note to customer', 'invoicing'); ?></option>
                 </select>
-                <a href="#" class="add_note button"><?php _e( 'Add', 'invoicing' ); ?></a> <span class="description"><?php _e( 'Add a note for your reference, or add a customer note (the user will be notified).', 'invoicing' ); ?></span>
+                <a href="#" class="add_note button"><?php _e('Add', 'invoicing'); ?></a> <span class="description"><?php _e('Add a note for your reference, or add a customer note (the user will be notified).', 'invoicing'); ?></span>
             </p>
         </div>
         <?php

Please login to merge, or discard this patch.

includes/wpinv-error-functions.php 1 patch

Spacing +24 added lines, -24 removed lines patch added patch discarded remove patch

@@ -1,61 +1,61 @@  discard block
 block discarded – undo
 <?php
 // MUST have WordPress.
-if ( !defined( 'WPINC' ) ) {
-    exit( 'Do NOT access this file directly: ' . basename( __FILE__ ) );
+if (!defined('WPINC')) {
+    exit('Do NOT access this file directly: ' . basename(__FILE__));
 }
 
 function wpinv_print_errors() {
     $errors = wpinv_get_errors();
 
-    if ( $errors ) {
-        $classes = apply_filters( 'wpinv_error_class', array(
+    if ($errors) {
+        $classes = apply_filters('wpinv_error_class', array(
             'wpinv_errors', 'wpinv-alert', 'wpinv-alert-error'
-        ) );
-        echo '<div class="' . implode( ' ', $classes ) . '">';
+        ));
+        echo '<div class="' . implode(' ', $classes) . '">';
             // Loop error codes and display errors
-           foreach ( $errors as $error_id => $error ) {
-                echo '<p class="wpinv_error" id="wpinv_error_' . $error_id . '"><strong>' . __( 'Error', 'invoicing' ) . '</strong>: ' . $error . '</p>';
+           foreach ($errors as $error_id => $error) {
+                echo '<p class="wpinv_error" id="wpinv_error_' . $error_id . '"><strong>' . __('Error', 'invoicing') . '</strong>: ' . $error . '</p>';
            }
         echo '</div>';
         wpinv_clear_errors();
     }
 }
-add_action( 'wpinv_purchase_form_before_submit', 'wpinv_print_errors' );
-add_action( 'wpinv_ajax_checkout_errors', 'wpinv_print_errors' );
+add_action('wpinv_purchase_form_before_submit', 'wpinv_print_errors');
+add_action('wpinv_ajax_checkout_errors', 'wpinv_print_errors');
 
 function wpinv_get_errors() {
     global $wpi_session;
     
-    return $wpi_session->get( 'wpinv_errors' );
+    return $wpi_session->get('wpinv_errors');
 }
 
-function wpinv_set_error( $error_id, $error_message ) {
+function wpinv_set_error($error_id, $error_message) {
     global $wpi_session;
     
     $errors = wpinv_get_errors();
 
-    if ( ! $errors ) {
+    if (!$errors) {
         $errors = array();
     }
 
-    $errors[ $error_id ] = $error_message;
-    $wpi_session->set( 'wpinv_errors', $errors );
+    $errors[$error_id] = $error_message;
+    $wpi_session->set('wpinv_errors', $errors);
 }
 
 function wpinv_clear_errors() {
     global $wpi_session;
     
-    $wpi_session->set( 'wpinv_errors', null );
+    $wpi_session->set('wpinv_errors', null);
 }
 
-function wpinv_unset_error( $error_id ) {
+function wpinv_unset_error($error_id) {
     global $wpi_session;
     
     $errors = wpinv_get_errors();
 
-    if ( $errors ) {
-        unset( $errors[ $error_id ] );
-        $wpi_session->set( 'wpinv_errors', $errors );
+    if ($errors) {
+        unset($errors[$error_id]);
+        $wpi_session->set('wpinv_errors', $errors);
     }
 }
 
@@ -63,8 +63,8 @@  discard block
 block discarded – undo
     die();
 }
 
-function wpinv_die( $message = '', $title = '', $status = 400 ) {
-    add_filter( 'wp_die_ajax_handler', 'wpinv_die_handler', 10, 3 );
-    add_filter( 'wp_die_handler', 'wpinv_die_handler', 10, 3 );
-    wp_die( $message, $title, array( 'response' => $status ));
+function wpinv_die($message = '', $title = '', $status = 400) {
+    add_filter('wp_die_ajax_handler', 'wpinv_die_handler', 10, 3);
+    add_filter('wp_die_handler', 'wpinv_die_handler', 10, 3);
+    wp_die($message, $title, array('response' => $status));
 }

Please login to merge, or discard this patch.

includes/libraries/GeoIp2/Model/City.php 1 patch

Spacing +1 added lines, -2 removed lines patch added patch discarded remove patch

@@ -130,7 +130,6 @@
 block discarded – undo
     private function mostSpecificSubdivision()
     {
         return empty($this->subdivisions) ?
-            new \GeoIp2\Record\Subdivision(array(), $this->locales) :
-            end($this->subdivisions);
+            new \GeoIp2\Record\Subdivision(array(), $this->locales) : end($this->subdivisions);
     }
 }

Please login to merge, or discard this patch.

includes/libraries/GeoIp2/WebService/Client.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -76,7 +76,7 @@
 block discarded – undo
         // This is for backwards compatibility. Do not remove except for a
         // major version bump.
         if (is_string($options)) {
-            $options = array( 'host' => $options );
+            $options = array('host' => $options);
         }
 
         if (!isset($options['host'])) {

Please login to merge, or discard this patch.

		@@ -130,7 +130,6 @@
		block discarded – undo
130	130	private function mostSpecificSubdivision()
131	131	{
132	132	return empty($this->subdivisions) ?
133		- new \GeoIp2\Record\Subdivision(array(), $this->locales) :
134		- end($this->subdivisions);
	133	+ new \GeoIp2\Record\Subdivision(array(), $this->locales) : end($this->subdivisions);
135	134	}
136	135	}

		@@ -76,7 +76,7 @@
		block discarded – undo
76	76	// This is for backwards compatibility. Do not remove except for a
77	77	// major version bump.
78	78	if (is_string($options)) {
79		- $options = array( 'host' => $options );
	79	+ $options = array('host' => $options);
80	80	}
81	81
82	82	if (!isset($options['host'])) {

		@@ -1,10 +1,10 @@
		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		-$email_footer = apply_filters( 'wpinv_email_footer_text', wpinv_get_option( 'email_footer_text' ) );
7		-$email_footer = $email_footer ? wpautop( wp_kses_post( wptexturize( $email_footer ) ) ) : '';
	6	+$email_footer = apply_filters('wpinv_email_footer_text', wpinv_get_option('email_footer_text'));
	7	+$email_footer = $email_footer ? wpautop(wp_kses_post(wptexturize($email_footer))) : '';
8	8	?>
9	9	</div>
10	10	</td>

		@@ -1,16 +1,16 @@ discard block
		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		-if ( !isset( $email_heading ) ) {
	6	+if (!isset($email_heading)) {
7	7	global $email_heading;
8	8	}
9	9	?>
10	10	<!DOCTYPE html>
11	11	<html dir="<?php echo is_rtl() ? 'rtl' : 'ltr'?>">
12	12	<head>
13		- <meta http-equiv="Content-Type" content="text/html; charset=<?php bloginfo( 'charset' ); ?>" />
	13	+ <meta http-equiv="Content-Type" content="text/html; charset=<?php bloginfo('charset'); ?>" />
14	14	<meta name="viewport" content="width=device-width, initial-scale=1">
15	15	<meta name="robots" content="noindex,nofollow">
16	16	<title><?php echo wpinv_get_blogname(); ?></title>
		@@ -22,13 +22,13 @@ discard block
		block discarded – undo
22	22	<td align="center" valign="top">
23	23	<div id="template_header_image">
24	24	<?php
25		- if ( $img = wpinv_get_option( 'email_header_image' ) ) {
26		- echo '<p style="margin-top:0;"><img style="max-width:100%" src="' . esc_url( $img ) . '" alt="' . esc_attr( wpinv_get_blogname() ) . '" /></p>';
	25	+ if ($img = wpinv_get_option('email_header_image')) {
	26	+ echo '<p style="margin-top:0;"><img style="max-width:100%" src="' . esc_url($img) . '" alt="' . esc_attr(wpinv_get_blogname()) . '" /></p>';
27	27	}
28	28	?>
29	29	</div>
30	30	<table border="0" cellpadding="0" cellspacing="0" width="100%" id="template_container">
31		- <?php if ( !empty( $email_heading ) ) { ?>
	31	+ <?php if (!empty($email_heading)) { ?>
32	32	<tr>
33	33	<td align="center" valign="top">
34	34	<!-- Header -->

		@@ -1,67 +1,67 @@
		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		-do_action( 'wpinv_email_before_billing_details', $invoice ); ?>
	6	+do_action('wpinv_email_before_billing_details', $invoice); ?>
7	7	<div id="wpinv-email-billing">
8		- <h3 class="wpinv-address-t"><?php echo apply_filters( 'wpinv_email_billing_title', __( 'Billing Details', 'invoicing' ) ); ?></h3>
	8	+ <h3 class="wpinv-address-t"><?php echo apply_filters('wpinv_email_billing_title', __('Billing Details', 'invoicing')); ?></h3>
9	9	<?php
10	10	$address_row = '';
11		- if ( $address = $invoice->get_address() ) {
12		- $address_row .= wpautop( wp_kses_post( $address ) );
	11	+ if ($address = $invoice->get_address()) {
	12	+ $address_row .= wpautop(wp_kses_post($address));
13	13	}
14	14
15	15	$address_fields = array();
16		- if ( !empty( $invoice->city ) ) {
	16	+ if (!empty($invoice->city)) {
17	17	$address_fields[] = $invoice->city;
18	18	}
19	19
20		- $country_code = !empty( $invoice->country ) ? $invoice->country : '';
21		- if ( !empty( $invoice->state ) ) {
22		- $address_fields[] = wpinv_state_name( $invoice->state, $country_code );
	20	+ $country_code = !empty($invoice->country) ? $invoice->country : '';
	21	+ if (!empty($invoice->state)) {
	22	+ $address_fields[] = wpinv_state_name($invoice->state, $country_code);
23	23	}
24	24
25		- if ( !empty( $address_fields ) ) {
26		- $address_fields = implode( ", ", $address_fields );
27		- $address_row .= wpautop( wp_kses_post( $address_fields ) );
	25	+ if (!empty($address_fields)) {
	26	+ $address_fields = implode(", ", $address_fields);
	27	+ $address_row .= wpautop(wp_kses_post($address_fields));
28	28	}
29	29
30		- if ( !empty( $country_code ) ) {
31		- $country = wpinv_country_name( $country_code );
32		- $address_row .= wpautop( wp_kses_post( trim( $country . '(' . $country_code . ')' . ' ' . $invoice->zip ) ) );
	30	+ if (!empty($country_code)) {
	31	+ $country = wpinv_country_name($country_code);
	32	+ $address_row .= wpautop(wp_kses_post(trim($country . '(' . $country_code . ')' . ' ' . $invoice->zip)));
33	33	}
34	34
35	35	?>
36	36	<table class="table table-bordered table-sm wpi-billing-details">
37	37	<tbody>
38		- <?php do_action( 'wpinv_email_billing_fields_first', $invoice ); ?>
	38	+ <?php do_action('wpinv_email_billing_fields_first', $invoice); ?>
39	39	<tr class="wpi-receipt-name">
40		- <th class="text-left"><?php _e( 'Name', 'invoicing' ); ?></th>
41		- <td><?php if ( $sent_to_admin && $invoice->user_id ) { ?><a href="<?php echo esc_url( add_query_arg( 'user_id', $invoice->get_user_id(), self_admin_url( 'user-edit.php' ) ) ) ;?>"><?php echo esc_html( $invoice->get_user_full_name() ); ?></a><?php } else { echo esc_html( $invoice->get_user_full_name() ); } ?></td>
	40	+ <th class="text-left"><?php _e('Name', 'invoicing'); ?></th>
	41	+ <td><?php if ($sent_to_admin && $invoice->user_id) { ?><a href="<?php echo esc_url(add_query_arg('user_id', $invoice->get_user_id(), self_admin_url('user-edit.php'))); ?>"><?php echo esc_html($invoice->get_user_full_name()); ?></a><?php } else { echo esc_html($invoice->get_user_full_name()); } ?></td>
42	42	</tr>
43	43	<tr class="wpi-receipt-email">
44		- <th class="text-left"><?php _e( 'Email', 'invoicing' ); ?></th>
45		- <td><?php echo $invoice->get_email() ;?></td>
	44	+ <th class="text-left"><?php _e('Email', 'invoicing'); ?></th>
	45	+ <td><?php echo $invoice->get_email(); ?></td>
46	46	</tr>
47		- <?php if ( $invoice->company ) { ?>
	47	+ <?php if ($invoice->company) { ?>
48	48	<tr class="wpi-receipt-company">
49		- <th class="text-left"><?php _e( 'Company', 'invoicing' ); ?></th>
50		- <td><?php echo esc_html( $invoice->company ) ;?></td>
	49	+ <th class="text-left"><?php _e('Company', 'invoicing'); ?></th>
	50	+ <td><?php echo esc_html($invoice->company); ?></td>
51	51	</tr>
52	52	<?php } ?>
53	53	<tr class="wpi-receipt-address">
54		- <th class="text-left"><?php _e( 'Address', 'invoicing' ); ?></th>
55		- <td><?php echo $address_row ;?></td>
	54	+ <th class="text-left"><?php _e('Address', 'invoicing'); ?></th>
	55	+ <td><?php echo $address_row; ?></td>
56	56	</tr>
57		- <?php if ( $invoice->phone ) { ?>
	57	+ <?php if ($invoice->phone) { ?>
58	58	<tr class="wpi-receipt-phone">
59		- <th class="text-left"><?php _e( 'Phone', 'invoicing' ); ?></th>
60		- <td><?php echo esc_html( $invoice->phone ) ;?></td>
	59	+ <th class="text-left"><?php _e('Phone', 'invoicing'); ?></th>
	60	+ <td><?php echo esc_html($invoice->phone); ?></td>
61	61	</tr>
62	62	<?php } ?>
63		- <?php do_action( 'wpinv_email_billing_fields_last', $invoice ); ?>
	63	+ <?php do_action('wpinv_email_billing_fields_last', $invoice); ?>
64	64	</tbody>
65	65	</table>
66	66	</div>
67		-<?php do_action( 'wpinv_email_after_billing_details', $invoice ); ?>
68	67	\ No newline at end of file
	68	+<?php do_action('wpinv_email_after_billing_details', $invoice); ?>
69	69	\ No newline at end of file

		@@ -2,11 +2,11 @@
		block discarded – undo
2	2	global $wpi_invoice;
3	3
4	4	$success_page_uri = wpinv_get_success_page_uri();
5		-if ( !empty( $wpi_invoice ) ) {
6		- $success_page_uri = add_query_arg( 'invoice_key', $wpi_invoice->get_key(), $success_page_uri );
	5	+if (!empty($wpi_invoice)) {
	6	+ $success_page_uri = add_query_arg('invoice_key', $wpi_invoice->get_key(), $success_page_uri);
7	7	}
8	8	?>
9	9	<div id="wpinv-payment-processing">
10		- <p><?php echo wp_sprintf( __( 'Your payment is processing. This page will reload automatically in 10 seconds. If it does not, click <a href="%s">here</a>.', 'invoicing' ), $success_page_uri ); ?> <i class="fa fa-spin fa-refresh"></i></p>
	10	+ <p><?php echo wp_sprintf(__('Your payment is processing. This page will reload automatically in 10 seconds. If it does not, click <a href="%s">here</a>.', 'invoicing'), $success_page_uri); ?> <i class="fa fa-spin fa-refresh"></i></p>
11	11	<script type="text/javascript">setTimeout(function(){ window.location = '<?php echo $success_page_uri; ?>'; }, 10000);</script>
12	12	</div>
13	13	\ No newline at end of file

		@@ -4,17 +4,17 @@ discard block
		block discarded – undo
4	4	*/
5	5	global $wpinv_receipt_args;
6	6
7		-$invoice = get_post( $wpinv_receipt_args['id'] );
	7	+$invoice = get_post($wpinv_receipt_args['id']);
8	8
9		-if( empty( $invoice ) ) {
	9	+if (empty($invoice)) {
10	10	?>
11	11	<div class="wpinv_errors alert wpi-alert-error">
12		- <?php _e( 'The specified receipt ID appears to be invalid', 'invoicing' ); ?>
	12	+ <?php _e('The specified receipt ID appears to be invalid', 'invoicing'); ?>
13	13	</div>
14	14	<?php
15	15	return;
16	16	}
17		-$invoice = wpinv_get_invoice( $invoice->ID );
	17	+$invoice = wpinv_get_invoice($invoice->ID);
18	18
19	19	global $ajax_cart_details;
20	20	$ajax_cart_details = $invoice->get_cart_details();
		@@ -24,18 +24,18 @@ discard block
		block discarded – undo
24	24	$quantities_enabled = wpinv_item_quantities_enabled();
25	25	$use_taxes = wpinv_use_taxes();
26	26	$zero_tax = !(float)$invoice->get_tax() > 0 ? true : false;
27		-$tax_label = !$zero_tax && $use_taxes ? ( wpinv_prices_include_tax() ? __( '(Tax Incl.)', 'invoicing' ) : __( '(Tax Excl.)', 'invoicing' ) ) : '';
	27	+$tax_label = !$zero_tax && $use_taxes ? (wpinv_prices_include_tax() ? __('(Tax Incl.)', 'invoicing') : __('(Tax Excl.)', 'invoicing')) : '';
28	28	?>
29		-<?php do_action( 'wpinv_before_receipt', $invoice ); ?>
	29	+<?php do_action('wpinv_before_receipt', $invoice); ?>
30	30	<div class="wpinv-receipt">
31		- <?php do_action( 'wpinv_receipt_start', $invoice ); ?>
32		- <div class="wpinv-receipt-message"><?php _e( 'Thank you for your payment!', 'invoicing' ); ?></div>
33		- <?php do_action( 'wpinv_before_receipt_details', $invoice ); ?>
	31	+ <?php do_action('wpinv_receipt_start', $invoice); ?>
	32	+ <div class="wpinv-receipt-message"><?php _e('Thank you for your payment!', 'invoicing'); ?></div>
	33	+ <?php do_action('wpinv_before_receipt_details', $invoice); ?>
34	34	<div class="wpinv-receipt-details">
35		- <h3 class="wpinv-details-t"><?php echo apply_filters( 'wpinv_receipt_details_title', __( 'Invoice Details', 'invoicing' ) ); ?></h3>
36		- <?php wpinv_display_invoice_details( $invoice ); ?>
	35	+ <h3 class="wpinv-details-t"><?php echo apply_filters('wpinv_receipt_details_title', __('Invoice Details', 'invoicing')); ?></h3>
	36	+ <?php wpinv_display_invoice_details($invoice); ?>
37	37	</div>
38		- <?php do_action( 'wpinv_after_receipt_details', $invoice ); ?>
39		- <?php do_action( 'wpinv_receipt_end', $invoice ); ?>
	38	+ <?php do_action('wpinv_after_receipt_details', $invoice); ?>
	39	+ <?php do_action('wpinv_receipt_end', $invoice); ?>
40	40	</div>
41		-<?php do_action( 'wpinv_after_receipt', $invoice ); ?>
42	41	\ No newline at end of file
	42	+<?php do_action('wpinv_after_receipt', $invoice); ?>
43	43	\ No newline at end of file

		@@ -1,39 +1,39 @@
		block discarded – undo
1	1	<?php
2	2	// MUST have WordPress.
3		-if ( !defined( 'WPINC' ) ) {
4		- exit( 'Do NOT access this file directly: ' . basename( __FILE__ ) );
	3	+if (!defined('WPINC')) {
	4	+ exit('Do NOT access this file directly: ' . basename(__FILE__));
5	5	}
6	6
7	7	class WPInv_Meta_Box_Notes {
8		- public static function output( $post ) {
	8	+ public static function output($post) {
9	9	global $post;
10	10
11		- $notes = wpinv_get_invoice_notes( $post->ID );
	11	+ $notes = wpinv_get_invoice_notes($post->ID);
12	12
13	13	echo '<ul class="invoice_notes">';
14	14
15		- if ( $notes ) {
16		- foreach( $notes as $note ) {
17		- wpinv_get_invoice_note_line_item( $note );
	15	+ if ($notes) {
	16	+ foreach ($notes as $note) {
	17	+ wpinv_get_invoice_note_line_item($note);
18	18	}
19	19
20	20	} else {
21		- echo '<li>' . __( 'There are no notes yet.', 'invoicing' ) . '</li>';
	21	+ echo '<li>' . __('There are no notes yet.', 'invoicing') . '</li>';
22	22	}
23	23
24	24	echo '</ul>';
25	25	?>
26	26	<div class="add_note">
27		- <h4><?php _e( 'Add note', 'invoicing' ); ?></h4>
	27	+ <h4><?php _e('Add note', 'invoicing'); ?></h4>
28	28	<p>
29	29	<textarea type="text" name="invoice_note" id="add_invoice_note" class="input-text" cols="20" rows="5"></textarea>
30	30	</p>
31	31	<p>
32	32	<select name="invoice_note_type" id="invoice_note_type">
33		- <option value=""><?php _e( 'Private note', 'invoicing' ); ?></option>
34		- <option value="customer"><?php _e( 'Note to customer', 'invoicing' ); ?></option>
	33	+ <option value=""><?php _e('Private note', 'invoicing'); ?></option>
	34	+ <option value="customer"><?php _e('Note to customer', 'invoicing'); ?></option>
35	35	</select>
36		- <a href="#" class="add_note button"><?php _e( 'Add', 'invoicing' ); ?></a> <span class="description"><?php _e( 'Add a note for your reference, or add a customer note (the user will be notified).', 'invoicing' ); ?></span>
	36	+ <a href="#" class="add_note button"><?php _e('Add', 'invoicing'); ?></a> <span class="description"><?php _e('Add a note for your reference, or add a customer note (the user will be notified).', 'invoicing'); ?></span>
37	37	</p>
38	38	</div>
39	39	<?php

		@@ -1,61 +1,61 @@ discard block
		block discarded – undo
1	1	<?php
2	2	// MUST have WordPress.
3		-if ( !defined( 'WPINC' ) ) {
4		- exit( 'Do NOT access this file directly: ' . basename( __FILE__ ) );
	3	+if (!defined('WPINC')) {
	4	+ exit('Do NOT access this file directly: ' . basename(__FILE__));
5	5	}
6	6
7	7	function wpinv_print_errors() {
8	8	$errors = wpinv_get_errors();
9	9
10		- if ( $errors ) {
11		- $classes = apply_filters( 'wpinv_error_class', array(
	10	+ if ($errors) {
	11	+ $classes = apply_filters('wpinv_error_class', array(
12	12	'wpinv_errors', 'wpinv-alert', 'wpinv-alert-error'
13		- ) );
14		- echo '<div class="' . implode( ' ', $classes ) . '">';
	13	+ ));
	14	+ echo '<div class="' . implode(' ', $classes) . '">';
15	15	// Loop error codes and display errors
16		- foreach ( $errors as $error_id => $error ) {
17		- echo '<p class="wpinv_error" id="wpinv_error_' . $error_id . '"><strong>' . __( 'Error', 'invoicing' ) . '</strong>: ' . $error . '</p>';
	16	+ foreach ($errors as $error_id => $error) {
	17	+ echo '<p class="wpinv_error" id="wpinv_error_' . $error_id . '"><strong>' . __('Error', 'invoicing') . '</strong>: ' . $error . '</p>';
18	18	}
19	19	echo '</div>';
20	20	wpinv_clear_errors();
21	21	}
22	22	}
23		-add_action( 'wpinv_purchase_form_before_submit', 'wpinv_print_errors' );
24		-add_action( 'wpinv_ajax_checkout_errors', 'wpinv_print_errors' );
	23	+add_action('wpinv_purchase_form_before_submit', 'wpinv_print_errors');
	24	+add_action('wpinv_ajax_checkout_errors', 'wpinv_print_errors');
25	25
26	26	function wpinv_get_errors() {
27	27	global $wpi_session;
28	28
29		- return $wpi_session->get( 'wpinv_errors' );
	29	+ return $wpi_session->get('wpinv_errors');
30	30	}
31	31
32		-function wpinv_set_error( $error_id, $error_message ) {
	32	+function wpinv_set_error($error_id, $error_message) {
33	33	global $wpi_session;
34	34
35	35	$errors = wpinv_get_errors();
36	36
37		- if ( ! $errors ) {
	37	+ if (!$errors) {
38	38	$errors = array();
39	39	}
40	40
41		- $errors[ $error_id ] = $error_message;
42		- $wpi_session->set( 'wpinv_errors', $errors );
	41	+ $errors[$error_id] = $error_message;
	42	+ $wpi_session->set('wpinv_errors', $errors);
43	43	}
44	44
45	45	function wpinv_clear_errors() {
46	46	global $wpi_session;
47	47
48		- $wpi_session->set( 'wpinv_errors', null );
	48	+ $wpi_session->set('wpinv_errors', null);
49	49	}
50	50
51		-function wpinv_unset_error( $error_id ) {
	51	+function wpinv_unset_error($error_id) {
52	52	global $wpi_session;
53	53
54	54	$errors = wpinv_get_errors();
55	55
56		- if ( $errors ) {
57		- unset( $errors[ $error_id ] );
58		- $wpi_session->set( 'wpinv_errors', $errors );
	56	+ if ($errors) {
	57	+ unset($errors[$error_id]);
	58	+ $wpi_session->set('wpinv_errors', $errors);
59	59	}
60	60	}
61	61
		@@ -63,8 +63,8 @@ discard block
		block discarded – undo
63	63	die();
64	64	}
65	65
66		-function wpinv_die( $message = '', $title = '', $status = 400 ) {
67		- add_filter( 'wp_die_ajax_handler', 'wpinv_die_handler', 10, 3 );
68		- add_filter( 'wp_die_handler', 'wpinv_die_handler', 10, 3 );
69		- wp_die( $message, $title, array( 'response' => $status ));
	66	+function wpinv_die($message = '', $title = '', $status = 400) {
	67	+ add_filter('wp_die_ajax_handler', 'wpinv_die_handler', 10, 3);
	68	+ add_filter('wp_die_handler', 'wpinv_die_handler', 10, 3);
	69	+ wp_die($message, $title, array('response' => $status));
70	70	}

AyeCode / invoicing

Push — master ( def67b...f4bb32 )

Status

Category

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +30 added lines, -30 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +14 added lines, -14 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +24 added lines, -24 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch