AyeCode / invoicing

includes/reports/class-getpaid-graph-downloader.php

Spacing +55 added lines, -55 removed lines

@@ -5,7 +5,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid_Graph_Downloader Class.
@@ -30,13 +30,13 @@ 
 	 *
 	 * @return GetPaid_Reports_Report_Items|GetPaid_Reports_Report_Gateways|GetPaid_Reports_Report_Discounts
 	 */
-	public function prepare_handler( $graph ) {
+	public function prepare_handler($graph) {
 
-		if ( empty( $this->handler->views[ $graph ] ) ) {
-			wp_die( __( 'Invalid Graph', 'invoicing' ), 400 );
+		if (empty($this->handler->views[$graph])) {
+			wp_die(__('Invalid Graph', 'invoicing'), 400);
 		}
 
-		return new $this->handler->views[ $graph ]['class']();
+		return new $this->handler->views[$graph]['class']();
 
 	}
 
@@ -47,10 +47,10 @@ 
 	 */
 	public function prepare_output() {
 
-		$output  = fopen( 'php://output', 'w' );
+		$output = fopen('php://output', 'w');
 
-		if ( false === $output ) {
-			wp_die( __( 'Unsupported server', 'invoicing' ), 500 );
+		if (false === $output) {
+			wp_die(__('Unsupported server', 'invoicing'), 500);
 		}
 
 		return $output;
@@ -61,13 +61,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function prepare_file_type( $graph ) {
+	public function prepare_file_type($graph) {
 
-		$file_type = empty( $_GET['file_type'] ) ? 'csv' : sanitize_text_field( $_GET['file_type'] );
-		$file_name = "getpaid-$graph-" . current_time( 'Y-m-d' );
+		$file_type = empty($_GET['file_type']) ? 'csv' : sanitize_text_field($_GET['file_type']);
+		$file_name = "getpaid-$graph-" . current_time('Y-m-d');
 
-		header( "Content-Type:application/$file_type" );
-		header( "Content-Disposition:attachment;filename=$file_name.$file_type" );
+		header("Content-Type:application/$file_type");
+		header("Content-Disposition:attachment;filename=$file_name.$file_type");
 
 		return $file_type;
 	}
@@ -76,24 +76,24 @@ 
 	 * Handles the actual download.
 	 *
 	 */
-	public function download( $graph ) {
+	public function download($graph) {
 		global $wpdb;
 
-		$handler   = $this->prepare_handler( $graph );
+		$handler   = $this->prepare_handler($graph);
 		$stream    = $this->prepare_output();
-		$stats     = $wpdb->get_results( $handler->get_sql( $handler->get_range() ) );
-		$headers   = array( $handler->field, 'total', 'total_raw' );
-		$file_type = $this->prepare_file_type( $graph );
-
-		if ( 'csv' == $file_type ) {
-			$this->download_csv( $stats, $stream, $headers );
-		} else if( 'xml' == $file_type ) {
-			$this->download_xml( $stats, $stream, $headers );
+		$stats     = $wpdb->get_results($handler->get_sql($handler->get_range()));
+		$headers   = array($handler->field, 'total', 'total_raw');
+		$file_type = $this->prepare_file_type($graph);
+
+		if ('csv' == $file_type) {
+			$this->download_csv($stats, $stream, $headers);
+		} else if ('xml' == $file_type) {
+			$this->download_xml($stats, $stream, $headers);
 		} else {
-			$this->download_json( $stats, $stream, $headers );
+			$this->download_json($stats, $stream, $headers);
 		}
 
-		fclose( $stream );
+		fclose($stream);
 		exit;
 	}
 
@@ -105,15 +105,15 @@ 
 	 * @param array $headers The fields to stream.
 	 * @since       1.0.19
 	 */
-	public function download_csv( $stats, $stream, $headers ) {
+	public function download_csv($stats, $stream, $headers) {
 
 		// Output the csv column headers.
-		fputcsv( $stream, $headers );
+		fputcsv($stream, $headers);
 
 		// Loop through 
-		foreach ( $stats as $stat ) {
-			$row  = array_values( $this->prepare_row( $stat, $headers ) );
-			fputcsv( $stream, $row );
+		foreach ($stats as $stat) {
+			$row = array_values($this->prepare_row($stat, $headers));
+			fputcsv($stream, $row);
 		}
 
 	}
@@ -126,16 +126,16 @@ 
 	 * @param array $headers The fields to stream.
 	 * @since       1.0.19
 	 */
-	public function download_json( $stats, $stream, $headers ) {
+	public function download_json($stats, $stream, $headers) {
 
 		$prepared = array();
 
 		// Loop through 
-		foreach ( $stats as $stat ) {
-			$prepared[] = $this->prepare_row( $stat, $headers );
+		foreach ($stats as $stat) {
+			$prepared[] = $this->prepare_row($stat, $headers);
 		}
 
-		fwrite( $stream, wp_json_encode( $prepared ) );
+		fwrite($stream, wp_json_encode($prepared));
 
 	}
 
@@ -147,19 +147,19 @@ 
 	 * @param array $headers The fields to stream.
 	 * @since       1.0.19
 	 */
-	public function download_xml( $stats, $stream, $headers ) {
+	public function download_xml($stats, $stream, $headers) {
 
 		$prepared = array();
 
 		// Loop through 
-		foreach ( $stats as $stat ) {
-			$prepared[] = $this->prepare_row( $stat, $headers );
+		foreach ($stats as $stat) {
+			$prepared[] = $this->prepare_row($stat, $headers);
 		}
 
 		$xml = new SimpleXMLElement('<?xml version="1.0"?><data></data>');
-		$this->convert_array_xml( $prepared, $xml );
+		$this->convert_array_xml($prepared, $xml);
 
-		fwrite( $stream, $xml->asXML() );
+		fwrite($stream, $xml->asXML());
 
 	}
 
@@ -169,24 +169,24 @@ 
 	 * @access      public
 	 * @since      1.0.19
 	 */
-	public function convert_array_xml( $data, $xml ) {
+	public function convert_array_xml($data, $xml) {
 
 		// Loop through 
-		foreach ( $data as $key => $value ) {
+		foreach ($data as $key => $value) {
 
-			$key = preg_replace( "/[^A-Za-z0-9_\-]/", '', $key );
+			$key = preg_replace("/[^A-Za-z0-9_\-]/", '', $key);
 
-			if ( is_array( $value ) ) {
+			if (is_array($value)) {
 
-				if ( is_numeric( $key ) ){
-					$key = 'item'.$key; //dealing with <0/>..<n/> issues
+				if (is_numeric($key)) {
+					$key = 'item' . $key; //dealing with <0/>..<n/> issues
 				}
 
-				$subnode = $xml->addChild( $key );
-				$this->convert_array_xml( $value, $subnode );
+				$subnode = $xml->addChild($key);
+				$this->convert_array_xml($value, $subnode);
 
 			} else {
-				$xml->addChild( $key, htmlspecialchars( $value ) );
+				$xml->addChild($key, htmlspecialchars($value));
 			}
 
 		}
@@ -201,24 +201,24 @@ 
 	 * @since       1.0.19
 	 * @return array
 	 */
-	public function prepare_row( $row, $fields ) {
+	public function prepare_row($row, $fields) {
 
 		$prepared = array();
 		$row      = (array) $row;
 
-		foreach ( $fields as $field ) {
+		foreach ($fields as $field) {
 
-			if ( $field === 'total' ) {
-				$prepared[ $field ] = html_entity_decode( strip_tags( wpinv_price( $row['total'] ) ), ENT_QUOTES );
+			if ($field === 'total') {
+				$prepared[$field] = html_entity_decode(strip_tags(wpinv_price($row['total'])), ENT_QUOTES);
 				continue;
 			}
 
-			if ( $field === 'total_raw' ) {
-				$prepared[ $field ] = wpinv_round_amount( wpinv_sanitize_amount( $row['total'] ) );
+			if ($field === 'total_raw') {
+				$prepared[$field] = wpinv_round_amount(wpinv_sanitize_amount($row['total']));
 				continue;
 			}
 
-			$prepared[ $field ] = strip_tags( $row[ $field ] );
+			$prepared[$field] = strip_tags($row[$field]);
 
 		}
 

includes/reports/class-getpaid-reports.php

Spacing +25 added lines, -25 removed lines

@@ -5,7 +5,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid_Reports Class.
@@ -17,10 +17,10 @@ 
 	 *
 	 */
 	public function __construct() {
-		add_action( 'admin_menu', array( $this, 'register_reports_page' ), 20 );
-		add_action( 'wpinv_reports_tab_reports', array( $this, 'display_reports_tab' ) );
-		add_action( 'wpinv_reports_tab_export', array( $this, 'display_exports_tab' ) );
-		add_action( 'getpaid_authenticated_admin_action_download_graph', array( $this, 'download_graph' ) );
+		add_action('admin_menu', array($this, 'register_reports_page'), 20);
+		add_action('wpinv_reports_tab_reports', array($this, 'display_reports_tab'));
+		add_action('wpinv_reports_tab_export', array($this, 'display_exports_tab'));
+		add_action('getpaid_authenticated_admin_action_download_graph', array($this, 'download_graph'));
 	}
 
 	/**
@@ -31,11 +31,11 @@ 
 
 		add_submenu_page(
             'wpinv',
-            __( 'Reports', 'invoicing' ),
-            __( 'Reports', 'invoicing' ),
+            __('Reports', 'invoicing'),
+            __('Reports', 'invoicing'),
             wpinv_get_capability(),
             'wpinv-reports',
-            array( $this, 'display_reports_page' )
+            array($this, 'display_reports_page')
 		);
 
 	}
@@ -48,26 +48,26 @@ 
 
 		// Prepare variables.
 		$tabs        = $this->get_tabs();
-		$current_tab = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'reports';
-		$current_tab = array_key_exists( $current_tab, $tabs ) ? $current_tab : 'reports';
+		$current_tab = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'reports';
+		$current_tab = array_key_exists($current_tab, $tabs) ? $current_tab : 'reports';
 
 		// Display the current tab.
 		?>
 
         <div class="wrap">
 
-			<h1><?php echo sanitize_text_field( $tabs[ $current_tab ] ); ?></h1>
+			<h1><?php echo sanitize_text_field($tabs[$current_tab]); ?></h1>
 
 			<nav class="nav-tab-wrapper">
 
 				<?php
-					foreach( $tabs as $key => $label ) {
+					foreach ($tabs as $key => $label) {
 
-						$key   = sanitize_text_field( $key );
-						$label = sanitize_text_field( $label );
+						$key   = sanitize_text_field($key);
+						$label = sanitize_text_field($label);
 						$class = $key == $current_tab ? 'nav-tab nav-tab-active' : 'nav-tab';
 						$url   = esc_url(
-							add_query_arg( 'tab', $key, admin_url( 'admin.php?page=wpinv-reports' ) )
+							add_query_arg('tab', $key, admin_url('admin.php?page=wpinv-reports'))
 						);
 
 						echo "\n\t\t\t<a href='$url' class='$class'>$label</a>";
@@ -77,15 +77,15 @@ 
 
 			</nav>
 
-			<div class="bsui <?php echo esc_attr( $current_tab ); ?>">
-				<?php do_action( "wpinv_reports_tab_{$current_tab}" ); ?>
+			<div class="bsui <?php echo esc_attr($current_tab); ?>">
+				<?php do_action("wpinv_reports_tab_{$current_tab}"); ?>
 			</div>
 
         </div>
 		<?php
 
-			wp_enqueue_script( 'chart-js', WPINV_PLUGIN_URL . 'assets/js/chart.bundle.min.js', array( 'jquery' ), '2.9.4', true );
-			wp_enqueue_style( 'chart-js', WPINV_PLUGIN_URL . 'assets/css/chart.min.css', array(), '2.9.4' );
+			wp_enqueue_script('chart-js', WPINV_PLUGIN_URL . 'assets/js/chart.bundle.min.js', array('jquery'), '2.9.4', true);
+			wp_enqueue_style('chart-js', WPINV_PLUGIN_URL . 'assets/css/chart.min.css', array(), '2.9.4');
 
 	}
 
@@ -97,11 +97,11 @@ 
 	public function get_tabs() {
 
 		$tabs = array(
-			'reports' => __( 'Reports', 'invoicing' ),
-			'export'  => __( 'Export', 'invoicing' ),
+			'reports' => __('Reports', 'invoicing'),
+			'export'  => __('Export', 'invoicing'),
 		);
 
-		return apply_filters( 'getpaid_report_tabs', $tabs );
+		return apply_filters('getpaid_report_tabs', $tabs);
 	}
 
 	/**
@@ -131,11 +131,11 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function download_graph( $args ) {
+	public function download_graph($args) {
 
-		if ( ! empty( $args['graph'] ) ) {
+		if (!empty($args['graph'])) {
 			$downloader = new GetPaid_Graph_Downloader();
-			$downloader->download( $args['graph'] );
+			$downloader->download($args['graph']);
 		}
 
 	}

includes/reports/class-getpaid-reports-report.php

Spacing +16 added lines, -16 removed lines

@@ -5,7 +5,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid_Reports_Report Class.
@@ -23,26 +23,26 @@ 
 	 */
 	public function __construct() {
 
-		$this->views        = array(
+		$this->views = array(
 
             'items'     => array(
-				'label' => __( 'Items', 'invoicing' ),
+				'label' => __('Items', 'invoicing'),
 				'class' => 'GetPaid_Reports_Report_Items',
 			),
 
 			'gateways'  => array(
-				'label' => __( 'Payment Methods', 'invoicing' ),
+				'label' => __('Payment Methods', 'invoicing'),
 				'class' => 'GetPaid_Reports_Report_Gateways',
 			),
 
 			'discounts'  => array(
-				'label' => __( 'Discount Codes', 'invoicing' ),
+				'label' => __('Discount Codes', 'invoicing'),
 				'class' => 'GetPaid_Reports_Report_Discounts',
 			),
 
         );
 
-		$this->views        = apply_filters( 'wpinv_report_views', $this->views );
+		$this->views = apply_filters('wpinv_report_views', $this->views);
 
 	}
 
@@ -84,14 +84,14 @@ 
 	 * Retrieves the download url.
 	 *
 	 */
-	public function get_download_url( $graph, $file_type) {
+	public function get_download_url($graph, $file_type) {
 
 		return wp_nonce_url(
 			add_query_arg(
 				array(
 					'getpaid-admin-action' => 'download_graph',
-					'file_type'            => urlencode( $file_type ),
-					'graph'                => urlencode( $graph ),
+					'file_type'            => urlencode($file_type),
+					'graph'                => urlencode($graph),
 				)
 			),
 			'getpaid-nonce',
@@ -108,7 +108,7 @@ 
 
 		?>
 
-			<?php foreach ( $this->views as $key => $view ) : ?>
+			<?php foreach ($this->views as $key => $view) : ?>
 				<div class="row mb-4">
 					<div class="col-12">
 						<div class="card m-0 p-0" style="max-width:100%">
@@ -118,17 +118,17 @@ 
 										<strong><?php echo $view['label']; ?></strong>
 									</div>
 									<div class="col-3">
-										<a title="<?php esc_attr_e( 'Download JSON', 'invoicing' ); ?>" href="<?php echo esc_url( $this->get_download_url( $key, 'json' ) ); ?>">
+										<a title="<?php esc_attr_e('Download JSON', 'invoicing'); ?>" href="<?php echo esc_url($this->get_download_url($key, 'json')); ?>">
 											<i class="fa fa-download text-dark" style="font-size: 16px" aria-hidden="true"></i>
-											<span class="screen-reader-text"><?php _e( 'Download JSON', 'invoicing' ); ?></span>
+											<span class="screen-reader-text"><?php _e('Download JSON', 'invoicing'); ?></span>
 										</a>
-										<a title="<?php esc_attr_e( 'Download CSV', 'invoicing' ); ?>" href="<?php echo esc_url( $this->get_download_url( $key, 'csv' ) ); ?>">
+										<a title="<?php esc_attr_e('Download CSV', 'invoicing'); ?>" href="<?php echo esc_url($this->get_download_url($key, 'csv')); ?>">
 											<i class="fa fa-file-csv text-dark" style="font-size: 16px" aria-hidden="true"></i>
-											<span class="screen-reader-text"><?php _e( 'Download CSV', 'invoicing' ); ?></span>
+											<span class="screen-reader-text"><?php _e('Download CSV', 'invoicing'); ?></span>
 										</a>
-										<a title="<?php esc_attr_e( 'Download XML', 'invoicing' ); ?>" href="<?php echo esc_url( $this->get_download_url( $key, 'xml' ) ); ?>">
+										<a title="<?php esc_attr_e('Download XML', 'invoicing'); ?>" href="<?php echo esc_url($this->get_download_url($key, 'xml')); ?>">
 											<i class="fa fa-file-code text-dark" style="font-size: 16px" aria-hidden="true"></i>
-											<span class="screen-reader-text"><?php _e( 'Download XML', 'invoicing' ); ?></span>
+											<span class="screen-reader-text"><?php _e('Download XML', 'invoicing'); ?></span>
 										</a>
 									</div>
 								</div>