Inspection of "fix some errors" - AyeCode/invoicing - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 0a2cdb...97ebee )

by Brian

created 2022-03-31 03:39 UTC

Status

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

@@ -7,11 +7,11 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 echo aui()->alert(
     array(
-        'content' => esc_html__( 'The gateway select box will appear here', 'invoicing' ),
+        'content' => esc_html__('The gateway select box will appear here', 'invoicing'),
         'type'    => 'info',
     )
 );

Please login to merge, or discard this patch.

templates/payment-forms-admin/previews/total_payable.php 1 patch

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

@@ -7,11 +7,11 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 echo aui()->alert(
     array(
-        'content' => esc_html__( 'The total payable amount will appear here', 'invoicing' ),
+        'content' => esc_html__('The total payable amount will appear here', 'invoicing'),
         'type'    => 'info',
     )
 );

Please login to merge, or discard this patch.

templates/payment-forms-admin/previews/heading.php 1 patch

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

@@ -8,7 +8,7 @@
 block discarded – undo
  */
 
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 ?>
 <component :is='form_element.level' v-html='form_element.text'></component>

Please login to merge, or discard this patch.

templates/payment-forms-admin/previews/price_input.php 1 patch

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

@@ -7,16 +7,16 @@  discard block
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 // Set the currency position.
 $position = wpinv_currency_position();
 
-if ( $position == 'left_space' ) {
+if ($position == 'left_space') {
     $position = 'left';
 }
 
-if ( $position == 'right_space' ) {
+if ($position == 'right_space') {
     $position = 'right';
 }
 
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
 
 <div class="input-group">
 
-    <?php if ( $position == 'left' ) : ?>
+    <?php if ($position == 'left') : ?>
         <div class="input-group-prepend">
             <span class="input-group-text"><?php echo wpinv_currency_symbol(); ?></span>
         </div>
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
 
     <input :placeholder='form_element.placeholder' class='form-control' type='text'>
 
-    <?php if ( $position == 'right' ) : ?>
+    <?php if ($position == 'right') : ?>
         <div class="input-group-append">
             <span class="input-group-text"><?php echo wpinv_currency_symbol(); ?></span>
         </div>

Please login to merge, or discard this patch.

templates/payment-forms-admin/previews/items.php 1 patch

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

@@ -7,14 +7,14 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 ?>
 
 <div v-if='!is_default'>
-    <div class='alert alert-info' role='alert'><?php _e( 'Item totals will appear here. Click to set items.', 'invoicing' ); ?></div>
+    <div class='alert alert-info' role='alert'><?php _e('Item totals will appear here. Click to set items.', 'invoicing'); ?></div>
 </div>
 
 <div v-if='is_default'>
-    <div class='alert alert-info' role='alert'><?php _e( 'Item totals will appear here.', 'invoicing' ); ?></div>
+    <div class='alert alert-info' role='alert'><?php _e('Item totals will appear here.', 'invoicing'); ?></div>
 </div>

Please login to merge, or discard this patch.

templates/invoice-history.php 2 patches

Indentation +68 added lines, -68 removed lines patch added patch discarded remove patch

@@ -42,92 +42,92 @@  discard block
 block discarded – undo
 					<tr class="wpinv-item wpinv-item-<?php echo sanitize_html_class( $invoice->get_status() ); ?>">
 						<?php
 
-							foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
+                            foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
 
-							$column_id = sanitize_html_class( $column_id );
-							$class     = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
+                            $column_id = sanitize_html_class( $column_id );
+                            $class     = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
 
-							echo "<td class='$column_id $class'>";
-							switch ( $column_id ) {
+                            echo "<td class='$column_id $class'>";
+                            switch ( $column_id ) {
 
-								case 'invoice-number':
-									echo wp_kses_post( wpinv_invoice_link( $invoice ) );
-									break;
+                                case 'invoice-number':
+                                    echo wp_kses_post( wpinv_invoice_link( $invoice ) );
+                                    break;
 
-								case 'created-date':
-									echo getpaid_format_date_value( $invoice->get_date_created() );
-									break;
+                                case 'created-date':
+                                    echo getpaid_format_date_value( $invoice->get_date_created() );
+                                    break;
 
-								case 'payment-date':
-									if ( $invoice->needs_payment() ) {
-										echo '&mdash;';
-										} else {
-										echo getpaid_format_date_value( $invoice->get_date_completed() );
-										}
+                                case 'payment-date':
+                                    if ( $invoice->needs_payment() ) {
+                                        echo '&mdash;';
+                                        } else {
+                                        echo getpaid_format_date_value( $invoice->get_date_completed() );
+                                        }
 
-									break;
+                                    break;
 
-								case 'invoice-status':
-									echo wp_kses_post( $invoice->get_status_label_html() );
+                                case 'invoice-status':
+                                    echo wp_kses_post( $invoice->get_status_label_html() );
 
-									break;
+                                    break;
 
-								case 'invoice-total':
-									echo wpinv_price( $invoice->get_total(), $invoice->get_currency() );
+                                case 'invoice-total':
+                                    echo wpinv_price( $invoice->get_total(), $invoice->get_currency() );
 
-									break;
+                                    break;
 
-								case 'invoice-actions':
-									$actions = array(
+                                case 'invoice-actions':
+                                    $actions = array(
 
-										'pay'   => array(
-											'url'   => $invoice->get_checkout_payment_url(),
-											'name'  => __( 'Pay Now', 'invoicing' ),
-											'class' => 'btn-success',
-										),
+                                        'pay'   => array(
+                                            'url'   => $invoice->get_checkout_payment_url(),
+                                            'name'  => __( 'Pay Now', 'invoicing' ),
+                                            'class' => 'btn-success',
+                                        ),
 
-										'print' => array(
-											'url'   => $invoice->get_view_url(),
-											'name'  => __( 'View', 'invoicing' ),
-											'class' => 'btn-secondary',
-											'attrs' => 'target="_blank"',
-										),
-									);
+                                        'print' => array(
+                                            'url'   => $invoice->get_view_url(),
+                                            'name'  => __( 'View', 'invoicing' ),
+                                            'class' => 'btn-secondary',
+                                            'attrs' => 'target="_blank"',
+                                        ),
+                                    );
 
-									if ( ! $invoice->needs_payment() ) {
-										unset( $actions['pay'] );
-										}
+                                    if ( ! $invoice->needs_payment() ) {
+                                        unset( $actions['pay'] );
+                                        }
 
-									if ( $invoice->needs_payment() ) {
-										$actions['delete'] = array(
-											'url'   => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
-											'name'  => __( 'Delete', 'invoicing' ),
-											'class' => 'btn-danger',
-										);
-										}
+                                    if ( $invoice->needs_payment() ) {
+                                        $actions['delete'] = array(
+                                            'url'   => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
+                                            'name'  => __( 'Delete', 'invoicing' ),
+                                            'class' => 'btn-danger',
+                                        );
+                                        }
 
-									$actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
+                                    $actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
 
-									foreach ( $actions as $key => $action ) {
-										$class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
-										echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class( $key ) . '" ' . ( ! empty( $action['attrs'] ) ? $action['attrs'] : '' ) . '>' . $action['name'] . '</a>';
-										}
+                                    foreach ( $actions as $key => $action ) {
+                                        $class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
+                                        echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class( $key ) . '" ' . ( ! empty( $action['attrs'] ) ? $action['attrs'] : '' ) . '>' . $action['name'] . '</a>';
+                                        }
 
-									break;
+                                    break;
 
-								default:
-									do_action( "wpinv_user_invoices_column_$column_id", $invoice );
-									break;
+                                default:
+                                    do_action( "wpinv_user_invoices_column_$column_id", $invoice );
+                                    break;
 
 
-								}
+                                }
 
-							do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
+                            do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
 
-							echo '</td>';
+                            echo '</td>';
 
-							endforeach;
-						?>
+                            endforeach;
+                        ?>
 					</tr>
 
 				<?php endforeach; ?>
@@ -141,16 +141,16 @@  discard block
 block discarded – undo
 	<?php if ( 1 < $invoices->max_num_pages ) : ?>
 		<div class="invoicing-Pagination">
 			<?php
-			$big = 999999;
+            $big = 999999;
 
-			echo paginate_links(
+            echo paginate_links(
                 array(
-					'base'   => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
-					'format' => '?paged=%#%',
-					'total'  => $invoices->max_num_pages,
+                    'base'   => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
+                    'format' => '?paged=%#%',
+                    'total'  => $invoices->max_num_pages,
                 )
             );
-			?>
+            ?>
 		</div>
 	<?php endif; ?>
 

Please login to merge, or discard this patch.

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

@@ -7,27 +7,27 @@  discard block
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 // Current page.
-$current_page   = empty( $_GET['page'] ) ? 1 : absint( $_GET['page'] );
+$current_page = empty($_GET['page']) ? 1 : absint($_GET['page']);
 
 // Fires before displaying user invoices.
-do_action( 'wpinv_before_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type );
+do_action('wpinv_before_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type);
 
 ?>
 
 
 	<div class="table-responsive">
-		<table class="table table-bordered table-hover getpaid-user-invoices <?php echo sanitize_html_class( $post_type ); ?>">
+		<table class="table table-bordered table-hover getpaid-user-invoices <?php echo sanitize_html_class($post_type); ?>">
 
 
 			<thead>
 				<tr>
 
-					<?php foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) : ?>
-						<th class="<?php echo sanitize_html_class( $column_id ); ?> <?php echo ( ! empty( $column_name['class'] ) ? sanitize_html_class( $column_name['class'] ) : ''); ?> border-bottom-0">
-							<span class="nobr"><?php echo esc_html( $column_name['title'] ); ?></span>
+					<?php foreach (wpinv_get_user_invoices_columns($post_type) as $column_id => $column_name) : ?>
+						<th class="<?php echo sanitize_html_class($column_id); ?> <?php echo (!empty($column_name['class']) ? sanitize_html_class($column_name['class']) : ''); ?> border-bottom-0">
+							<span class="nobr"><?php echo esc_html($column_name['title']); ?></span>
 						</th>
 					<?php endforeach; ?>
 
@@ -37,43 +37,43 @@  discard block
 block discarded – undo
 
 
 			<tbody>
-				<?php foreach ( $invoices->invoices as $invoice ) : ?>
+				<?php foreach ($invoices->invoices as $invoice) : ?>
 
-					<tr class="wpinv-item wpinv-item-<?php echo sanitize_html_class( $invoice->get_status() ); ?>">
+					<tr class="wpinv-item wpinv-item-<?php echo sanitize_html_class($invoice->get_status()); ?>">
 						<?php
 
-							foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
+							foreach (wpinv_get_user_invoices_columns($post_type) as $column_id => $column_name) :
 
-							$column_id = sanitize_html_class( $column_id );
-							$class     = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
+							$column_id = sanitize_html_class($column_id);
+							$class     = empty($column_name['class']) ? '' : sanitize_html_class($column_name['class']);
 
 							echo "<td class='$column_id $class'>";
-							switch ( $column_id ) {
+							switch ($column_id) {
 
 								case 'invoice-number':
-									echo wp_kses_post( wpinv_invoice_link( $invoice ) );
+									echo wp_kses_post(wpinv_invoice_link($invoice));
 									break;
 
 								case 'created-date':
-									echo getpaid_format_date_value( $invoice->get_date_created() );
+									echo getpaid_format_date_value($invoice->get_date_created());
 									break;
 
 								case 'payment-date':
-									if ( $invoice->needs_payment() ) {
+									if ($invoice->needs_payment()) {
 										echo '&mdash;';
 										} else {
-										echo getpaid_format_date_value( $invoice->get_date_completed() );
+										echo getpaid_format_date_value($invoice->get_date_completed());
 										}
 
 									break;
 
 								case 'invoice-status':
-									echo wp_kses_post( $invoice->get_status_label_html() );
+									echo wp_kses_post($invoice->get_status_label_html());
 
 									break;
 
 								case 'invoice-total':
-									echo wpinv_price( $invoice->get_total(), $invoice->get_currency() );
+									echo wpinv_price($invoice->get_total(), $invoice->get_currency());
 
 									break;
 
@@ -82,47 +82,47 @@  discard block
 block discarded – undo
 
 										'pay'   => array(
 											'url'   => $invoice->get_checkout_payment_url(),
-											'name'  => __( 'Pay Now', 'invoicing' ),
+											'name'  => __('Pay Now', 'invoicing'),
 											'class' => 'btn-success',
 										),
 
 										'print' => array(
 											'url'   => $invoice->get_view_url(),
-											'name'  => __( 'View', 'invoicing' ),
+											'name'  => __('View', 'invoicing'),
 											'class' => 'btn-secondary',
 											'attrs' => 'target="_blank"',
 										),
 									);
 
-									if ( ! $invoice->needs_payment() ) {
-										unset( $actions['pay'] );
+									if (!$invoice->needs_payment()) {
+										unset($actions['pay']);
 										}
 
-									if ( $invoice->needs_payment() ) {
+									if ($invoice->needs_payment()) {
 										$actions['delete'] = array(
-											'url'   => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
-											'name'  => __( 'Delete', 'invoicing' ),
+											'url'   => getpaid_get_authenticated_action_url('delete_invoice', add_query_arg('invoice_id', $invoice->get_id())),
+											'name'  => __('Delete', 'invoicing'),
 											'class' => 'btn-danger',
 										);
 										}
 
-									$actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
+									$actions = apply_filters('wpinv_user_invoices_actions', $actions, $invoice, $post_type);
 
-									foreach ( $actions as $key => $action ) {
-										$class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
-										echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class( $key ) . '" ' . ( ! empty( $action['attrs'] ) ? $action['attrs'] : '' ) . '>' . $action['name'] . '</a>';
+									foreach ($actions as $key => $action) {
+										$class = !empty($action['class']) ? sanitize_html_class($action['class']) : '';
+										echo '<a href="' . esc_url($action['url']) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class($key) . '" ' . (!empty($action['attrs']) ? $action['attrs'] : '') . '>' . $action['name'] . '</a>';
 										}
 
 									break;
 
 								default:
-									do_action( "wpinv_user_invoices_column_$column_id", $invoice );
+									do_action("wpinv_user_invoices_column_$column_id", $invoice);
 									break;
 
 
 								}
 
-							do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
+							do_action("wpinv_user_invoices_column_after_$column_id", $invoice);
 
 							echo '</td>';
 
@@ -136,16 +136,16 @@  discard block
 block discarded – undo
 		</table>
 	</div>
 
-	<?php do_action( 'wpinv_before_user_invoices_pagination' ); ?>
+	<?php do_action('wpinv_before_user_invoices_pagination'); ?>
 
-	<?php if ( 1 < $invoices->max_num_pages ) : ?>
+	<?php if (1 < $invoices->max_num_pages) : ?>
 		<div class="invoicing-Pagination">
 			<?php
 			$big = 999999;
 
 			echo paginate_links(
                 array(
-					'base'   => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
+					'base'   => str_replace($big, '%#%', esc_url(get_pagenum_link($big))),
 					'format' => '?paged=%#%',
 					'total'  => $invoices->max_num_pages,
                 )
@@ -154,4 +154,4 @@  discard block
 block discarded – undo
 		</div>
 	<?php endif; ?>
 
-<?php do_action( 'wpinv_after_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type ); ?>
+<?php do_action('wpinv_after_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type); ?>

Please login to merge, or discard this patch.

templates/payment-forms/elements/checkbox.php 1 patch

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

@@ -7,24 +7,24 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
-$label       = empty( $label ) ? '' : wp_kses_post( $label );
-$label_class = sanitize_key( preg_replace( '/[^A-Za-z0-9_-]/', '-', $label ) );
+$label       = empty($label) ? '' : wp_kses_post($label);
+$label_class = sanitize_key(preg_replace('/[^A-Za-z0-9_-]/', '-', $label));
 
-if ( ! empty( $required ) ) {
+if (!empty($required)) {
     $label .= "<span class='text-danger'> *</span>";
 }
 
 echo aui()->input(
     array(
         'type'      => 'checkbox',
-        'name'      => esc_attr( $id ),
-        'id'        => esc_attr( $id ) . uniqid( '_' ),
-        'required'  => ! empty( $required ),
+        'name'      => esc_attr($id),
+        'id'        => esc_attr($id) . uniqid('_'),
+        'required'  => !empty($required),
         'label'     => $label,
-        'value'     => esc_attr__( 'Yes', 'invoicing' ),
-        'help_text' => empty( $description ) ? '' : wp_kses_post( $description ),
+        'value'     => esc_attr__('Yes', 'invoicing'),
+        'help_text' => empty($description) ? '' : wp_kses_post($description),
         'class'     => 'w-auto ' . $label_class,
     )
 );

Please login to merge, or discard this patch.

templates/payment-forms/elements/separator.php 1 patch

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

@@ -7,6 +7,6 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 ?>
 <hr class="featurette-divider" />

Please login to merge, or discard this patch.

templates/payment-forms/elements/email.php 1 patch

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

@@ -7,24 +7,24 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
-$label = empty( $label ) ? '' : wp_kses_post( $label );
-$label_class = sanitize_key( preg_replace( '/[^A-Za-z0-9_-]/', '-', $label ) );
+$label = empty($label) ? '' : wp_kses_post($label);
+$label_class = sanitize_key(preg_replace('/[^A-Za-z0-9_-]/', '-', $label));
 
-if ( ! empty( $required ) ) {
+if (!empty($required)) {
     $label .= "<span class='text-danger'> *</span>";
 }
 
 echo aui()->input(
     array(
-        'name'        => esc_attr( $id ),
-        'id'          => esc_attr( $id ) . uniqid( '_' ),
-        'placeholder' => empty( $placeholder ) ? '' : esc_attr( $placeholder ),
-        'required'    => ! empty( $required ),
+        'name'        => esc_attr($id),
+        'id'          => esc_attr($id) . uniqid('_'),
+        'placeholder' => empty($placeholder) ? '' : esc_attr($placeholder),
+        'required'    => !empty($required),
         'label'       => $label,
         'label_type'  => 'vertical',
-        'help_text'   => empty( $description ) ? '' : wp_kses_post( $description ),
+        'help_text'   => empty($description) ? '' : wp_kses_post($description),
         'type'        => 'email',
         'class'       => $label_class,
     )

Please login to merge, or discard this patch.

		@@ -42,92 +42,92 @@ discard block
		block discarded – undo
42	42	<tr class="wpinv-item wpinv-item-<?php echo sanitize_html_class( $invoice->get_status() ); ?>">
43	43	<?php
44	44
45		- foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
	45	+ foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
46	46
47		- $column_id = sanitize_html_class( $column_id );
48		- $class = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
	47	+ $column_id = sanitize_html_class( $column_id );
	48	+ $class = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
49	49
50		- echo "<td class='$column_id $class'>";
51		- switch ( $column_id ) {
	50	+ echo "<td class='$column_id $class'>";
	51	+ switch ( $column_id ) {
52	52
53		- case 'invoice-number':
54		- echo wp_kses_post( wpinv_invoice_link( $invoice ) );
55		- break;
	53	+ case 'invoice-number':
	54	+ echo wp_kses_post( wpinv_invoice_link( $invoice ) );
	55	+ break;
56	56
57		- case 'created-date':
58		- echo getpaid_format_date_value( $invoice->get_date_created() );
59		- break;
	57	+ case 'created-date':
	58	+ echo getpaid_format_date_value( $invoice->get_date_created() );
	59	+ break;
60	60
61		- case 'payment-date':
62		- if ( $invoice->needs_payment() ) {
63		- echo '—';
64		- } else {
65		- echo getpaid_format_date_value( $invoice->get_date_completed() );
66		- }
	61	+ case 'payment-date':
	62	+ if ( $invoice->needs_payment() ) {
	63	+ echo '—';
	64	+ } else {
	65	+ echo getpaid_format_date_value( $invoice->get_date_completed() );
	66	+ }
67	67
68		- break;
	68	+ break;
69	69
70		- case 'invoice-status':
71		- echo wp_kses_post( $invoice->get_status_label_html() );
	70	+ case 'invoice-status':
	71	+ echo wp_kses_post( $invoice->get_status_label_html() );
72	72
73		- break;
	73	+ break;
74	74
75		- case 'invoice-total':
76		- echo wpinv_price( $invoice->get_total(), $invoice->get_currency() );
	75	+ case 'invoice-total':
	76	+ echo wpinv_price( $invoice->get_total(), $invoice->get_currency() );
77	77
78		- break;
	78	+ break;
79	79
80		- case 'invoice-actions':
81		- $actions = array(
	80	+ case 'invoice-actions':
	81	+ $actions = array(
82	82
83		- 'pay' => array(
84		- 'url' => $invoice->get_checkout_payment_url(),
85		- 'name' => __( 'Pay Now', 'invoicing' ),
86		- 'class' => 'btn-success',
87		- ),
	83	+ 'pay' => array(
	84	+ 'url' => $invoice->get_checkout_payment_url(),
	85	+ 'name' => __( 'Pay Now', 'invoicing' ),
	86	+ 'class' => 'btn-success',
	87	+ ),
88	88
89		- 'print' => array(
90		- 'url' => $invoice->get_view_url(),
91		- 'name' => __( 'View', 'invoicing' ),
92		- 'class' => 'btn-secondary',
93		- 'attrs' => 'target="_blank"',
94		- ),
95		- );
	89	+ 'print' => array(
	90	+ 'url' => $invoice->get_view_url(),
	91	+ 'name' => __( 'View', 'invoicing' ),
	92	+ 'class' => 'btn-secondary',
	93	+ 'attrs' => 'target="_blank"',
	94	+ ),
	95	+ );
96	96
97		- if ( ! $invoice->needs_payment() ) {
98		- unset( $actions['pay'] );
99		- }
	97	+ if ( ! $invoice->needs_payment() ) {
	98	+ unset( $actions['pay'] );
	99	+ }
100	100
101		- if ( $invoice->needs_payment() ) {
102		- $actions['delete'] = array(
103		- 'url' => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
104		- 'name' => __( 'Delete', 'invoicing' ),
105		- 'class' => 'btn-danger',
106		- );
107		- }
	101	+ if ( $invoice->needs_payment() ) {
	102	+ $actions['delete'] = array(
	103	+ 'url' => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
	104	+ 'name' => __( 'Delete', 'invoicing' ),
	105	+ 'class' => 'btn-danger',
	106	+ );
	107	+ }
108	108
109		- $actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
	109	+ $actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
110	110
111		- foreach ( $actions as $key => $action ) {
112		- $class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
113		- echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class( $key ) . '" ' . ( ! empty( $action['attrs'] ) ? $action['attrs'] : '' ) . '>' . $action['name'] . '</a>';
114		- }
	111	+ foreach ( $actions as $key => $action ) {
	112	+ $class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
	113	+ echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class( $key ) . '" ' . ( ! empty( $action['attrs'] ) ? $action['attrs'] : '' ) . '>' . $action['name'] . '</a>';
	114	+ }
115	115
116		- break;
	116	+ break;
117	117
118		- default:
119		- do_action( "wpinv_user_invoices_column_$column_id", $invoice );
120		- break;
	118	+ default:
	119	+ do_action( "wpinv_user_invoices_column_$column_id", $invoice );
	120	+ break;
121	121
122	122
123		- }
	123	+ }
124	124
125		- do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
	125	+ do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
126	126
127		- echo '</td>';
	127	+ echo '</td>';
128	128
129		- endforeach;
130		- ?>
	129	+ endforeach;
	130	+ ?>
131	131	</tr>
132	132
133	133	<?php endforeach; ?>
		@@ -141,16 +141,16 @@ discard block
		block discarded – undo
141	141	<?php if ( 1 < $invoices->max_num_pages ) : ?>
142	142	<div class="invoicing-Pagination">
143	143	<?php
144		- $big = 999999;
	144	+ $big = 999999;
145	145
146		- echo paginate_links(
	146	+ echo paginate_links(
147	147	array(
148		- 'base' => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
149		- 'format' => '?paged=%#%',
150		- 'total' => $invoices->max_num_pages,
	148	+ 'base' => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
	149	+ 'format' => '?paged=%#%',
	150	+ 'total' => $invoices->max_num_pages,
151	151	)
152	152	);
153		- ?>
	153	+ ?>
154	154	</div>
155	155	<?php endif; ?>
156	156

		@@ -7,27 +7,27 @@ discard block
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12	12	// Current page.
13		-$current_page = empty( $_GET['page'] ) ? 1 : absint( $_GET['page'] );
	13	+$current_page = empty($_GET['page']) ? 1 : absint($_GET['page']);
14	14
15	15	// Fires before displaying user invoices.
16		-do_action( 'wpinv_before_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type );
	16	+do_action('wpinv_before_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type);
17	17
18	18	?>
19	19
20	20
21	21	<div class="table-responsive">
22		- <table class="table table-bordered table-hover getpaid-user-invoices <?php echo sanitize_html_class( $post_type ); ?>">
	22	+ <table class="table table-bordered table-hover getpaid-user-invoices <?php echo sanitize_html_class($post_type); ?>">
23	23
24	24
25	25	<thead>
26	26	<tr>
27	27
28		- <?php foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) : ?>
29		- <th class="<?php echo sanitize_html_class( $column_id ); ?> <?php echo ( ! empty( $column_name['class'] ) ? sanitize_html_class( $column_name['class'] ) : ''); ?> border-bottom-0">
30		- <span class="nobr"><?php echo esc_html( $column_name['title'] ); ?></span>
	28	+ <?php foreach (wpinv_get_user_invoices_columns($post_type) as $column_id => $column_name) : ?>
	29	+ <th class="<?php echo sanitize_html_class($column_id); ?> <?php echo (!empty($column_name['class']) ? sanitize_html_class($column_name['class']) : ''); ?> border-bottom-0">
	30	+ <span class="nobr"><?php echo esc_html($column_name['title']); ?></span>
31	31	</th>
32	32	<?php endforeach; ?>
33	33
		@@ -37,43 +37,43 @@ discard block
		block discarded – undo
37	37
38	38
39	39	<tbody>
40		- <?php foreach ( $invoices->invoices as $invoice ) : ?>
	40	+ <?php foreach ($invoices->invoices as $invoice) : ?>
41	41
42		- <tr class="wpinv-item wpinv-item-<?php echo sanitize_html_class( $invoice->get_status() ); ?>">
	42	+ <tr class="wpinv-item wpinv-item-<?php echo sanitize_html_class($invoice->get_status()); ?>">
43	43	<?php
44	44
45		- foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
	45	+ foreach (wpinv_get_user_invoices_columns($post_type) as $column_id => $column_name) :
46	46
47		- $column_id = sanitize_html_class( $column_id );
48		- $class = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
	47	+ $column_id = sanitize_html_class($column_id);
	48	+ $class = empty($column_name['class']) ? '' : sanitize_html_class($column_name['class']);
49	49
50	50	echo "<td class='$column_id $class'>";
51		- switch ( $column_id ) {
	51	+ switch ($column_id) {
52	52
53	53	case 'invoice-number':
54		- echo wp_kses_post( wpinv_invoice_link( $invoice ) );
	54	+ echo wp_kses_post(wpinv_invoice_link($invoice));
55	55	break;
56	56
57	57	case 'created-date':
58		- echo getpaid_format_date_value( $invoice->get_date_created() );
	58	+ echo getpaid_format_date_value($invoice->get_date_created());
59	59	break;
60	60
61	61	case 'payment-date':
62		- if ( $invoice->needs_payment() ) {
	62	+ if ($invoice->needs_payment()) {
63	63	echo '—';
64	64	} else {
65		- echo getpaid_format_date_value( $invoice->get_date_completed() );
	65	+ echo getpaid_format_date_value($invoice->get_date_completed());
66	66	}
67	67
68	68	break;
69	69
70	70	case 'invoice-status':
71		- echo wp_kses_post( $invoice->get_status_label_html() );
	71	+ echo wp_kses_post($invoice->get_status_label_html());
72	72
73	73	break;
74	74
75	75	case 'invoice-total':
76		- echo wpinv_price( $invoice->get_total(), $invoice->get_currency() );
	76	+ echo wpinv_price($invoice->get_total(), $invoice->get_currency());
77	77
78	78	break;
79	79
		@@ -82,47 +82,47 @@ discard block
		block discarded – undo
82	82
83	83	'pay' => array(
84	84	'url' => $invoice->get_checkout_payment_url(),
85		- 'name' => __( 'Pay Now', 'invoicing' ),
	85	+ 'name' => __('Pay Now', 'invoicing'),
86	86	'class' => 'btn-success',
87	87	),
88	88
89	89	'print' => array(
90	90	'url' => $invoice->get_view_url(),
91		- 'name' => __( 'View', 'invoicing' ),
	91	+ 'name' => __('View', 'invoicing'),
92	92	'class' => 'btn-secondary',
93	93	'attrs' => 'target="_blank"',
94	94	),
95	95	);
96	96
97		- if ( ! $invoice->needs_payment() ) {
98		- unset( $actions['pay'] );
	97	+ if (!$invoice->needs_payment()) {
	98	+ unset($actions['pay']);
99	99	}
100	100
101		- if ( $invoice->needs_payment() ) {
	101	+ if ($invoice->needs_payment()) {
102	102	$actions['delete'] = array(
103		- 'url' => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
104		- 'name' => __( 'Delete', 'invoicing' ),
	103	+ 'url' => getpaid_get_authenticated_action_url('delete_invoice', add_query_arg('invoice_id', $invoice->get_id())),
	104	+ 'name' => __('Delete', 'invoicing'),
105	105	'class' => 'btn-danger',
106	106	);
107	107	}
108	108
109		- $actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
	109	+ $actions = apply_filters('wpinv_user_invoices_actions', $actions, $invoice, $post_type);
110	110
111		- foreach ( $actions as $key => $action ) {
112		- $class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
113		- echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class( $key ) . '" ' . ( ! empty( $action['attrs'] ) ? $action['attrs'] : '' ) . '>' . $action['name'] . '</a>';
	111	+ foreach ($actions as $key => $action) {
	112	+ $class = !empty($action['class']) ? sanitize_html_class($action['class']) : '';
	113	+ echo '<a href="' . esc_url($action['url']) . '" class="btn btn-sm btn-block ' . $class . ' ' . sanitize_html_class($key) . '" ' . (!empty($action['attrs']) ? $action['attrs'] : '') . '>' . $action['name'] . '</a>';
114	114	}
115	115
116	116	break;
117	117
118	118	default:
119		- do_action( "wpinv_user_invoices_column_$column_id", $invoice );
	119	+ do_action("wpinv_user_invoices_column_$column_id", $invoice);
120	120	break;
121	121
122	122
123	123	}
124	124
125		- do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
	125	+ do_action("wpinv_user_invoices_column_after_$column_id", $invoice);
126	126
127	127	echo '</td>';
128	128
		@@ -136,16 +136,16 @@ discard block
		block discarded – undo
136	136	</table>
137	137	</div>
138	138
139		- <?php do_action( 'wpinv_before_user_invoices_pagination' ); ?>
	139	+ <?php do_action('wpinv_before_user_invoices_pagination'); ?>
140	140
141		- <?php if ( 1 < $invoices->max_num_pages ) : ?>
	141	+ <?php if (1 < $invoices->max_num_pages) : ?>
142	142	<div class="invoicing-Pagination">
143	143	<?php
144	144	$big = 999999;
145	145
146	146	echo paginate_links(
147	147	array(
148		- 'base' => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
	148	+ 'base' => str_replace($big, '%#%', esc_url(get_pagenum_link($big))),
149	149	'format' => '?paged=%#%',
150	150	'total' => $invoices->max_num_pages,
151	151	)
		@@ -154,4 +154,4 @@ discard block
		block discarded – undo
154	154	</div>
155	155	<?php endif; ?>
156	156
157		-<?php do_action( 'wpinv_after_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type ); ?>
	157	+<?php do_action('wpinv_after_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type); ?>

		@@ -7,11 +7,11 @@
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12	12	echo aui()->alert(
13	13	array(
14		- 'content' => esc_html__( 'The gateway select box will appear here', 'invoicing' ),
	14	+ 'content' => esc_html__('The gateway select box will appear here', 'invoicing'),
15	15	'type' => 'info',
16	16	)
17	17	);

		@@ -8,7 +8,7 @@
		block discarded – undo
8	8	*/
9	9
10	10
11		-defined( 'ABSPATH' ) \|\| exit;
	11	+defined('ABSPATH') \|\| exit;
12	12
13	13	?>
14	14	<component :is='form_element.level' v-html='form_element.text'></component>

		@@ -7,16 +7,16 @@ discard block
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12	12	// Set the currency position.
13	13	$position = wpinv_currency_position();
14	14
15		-if ( $position == 'left_space' ) {
	15	+if ($position == 'left_space') {
16	16	$position = 'left';
17	17	}
18	18
19		-if ( $position == 'right_space' ) {
	19	+if ($position == 'right_space') {
20	20	$position = 'right';
21	21	}
22	22
		@@ -26,7 +26,7 @@ discard block
		block discarded – undo
26	26
27	27	<div class="input-group">
28	28
29		- <?php if ( $position == 'left' ) : ?>
	29	+ <?php if ($position == 'left') : ?>
30	30	<div class="input-group-prepend">
31	31	<span class="input-group-text"><?php echo wpinv_currency_symbol(); ?></span>
32	32	</div>
		@@ -34,7 +34,7 @@ discard block
		block discarded – undo
34	34
35	35	<input :placeholder='form_element.placeholder' class='form-control' type='text'>
36	36
37		- <?php if ( $position == 'right' ) : ?>
	37	+ <?php if ($position == 'right') : ?>
38	38	<div class="input-group-append">
39	39	<span class="input-group-text"><?php echo wpinv_currency_symbol(); ?></span>
40	40	</div>

		@@ -7,14 +7,14 @@
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12	12	?>
13	13
14	14	<div v-if='!is_default'>
15		- <div class='alert alert-info' role='alert'><?php _e( 'Item totals will appear here. Click to set items.', 'invoicing' ); ?></div>
	15	+ <div class='alert alert-info' role='alert'><?php _e('Item totals will appear here. Click to set items.', 'invoicing'); ?></div>
16	16	</div>
17	17
18	18	<div v-if='is_default'>
19		- <div class='alert alert-info' role='alert'><?php _e( 'Item totals will appear here.', 'invoicing' ); ?></div>
	19	+ <div class='alert alert-info' role='alert'><?php _e('Item totals will appear here.', 'invoicing'); ?></div>
20	20	</div>

		@@ -7,24 +7,24 @@
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12		-$label = empty( $label ) ? '' : wp_kses_post( $label );
13		-$label_class = sanitize_key( preg_replace( '/[^A-Za-z0-9_-]/', '-', $label ) );
	12	+$label = empty($label) ? '' : wp_kses_post($label);
	13	+$label_class = sanitize_key(preg_replace('/[^A-Za-z0-9_-]/', '-', $label));
14	14
15		-if ( ! empty( $required ) ) {
	15	+if (!empty($required)) {
16	16	$label .= "<span class='text-danger'> *</span>";
17	17	}
18	18
19	19	echo aui()->input(
20	20	array(
21	21	'type' => 'checkbox',
22		- 'name' => esc_attr( $id ),
23		- 'id' => esc_attr( $id ) . uniqid( '_' ),
24		- 'required' => ! empty( $required ),
	22	+ 'name' => esc_attr($id),
	23	+ 'id' => esc_attr($id) . uniqid('_'),
	24	+ 'required' => !empty($required),
25	25	'label' => $label,
26		- 'value' => esc_attr__( 'Yes', 'invoicing' ),
27		- 'help_text' => empty( $description ) ? '' : wp_kses_post( $description ),
	26	+ 'value' => esc_attr__('Yes', 'invoicing'),
	27	+ 'help_text' => empty($description) ? '' : wp_kses_post($description),
28	28	'class' => 'w-auto ' . $label_class,
29	29	)
30	30	);

		@@ -7,6 +7,6 @@
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11	?>
12	12	<hr class="featurette-divider" />

AyeCode / invoicing

Push — master ( 0a2cdb...97ebee )

Status

Category

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

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

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

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

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

Indentation +68 added lines, -68 removed lines patch added patch discarded remove patch

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

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

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

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