Sales reporting fixes and improvements

.changelogs/fix_date-range-widget-1.yml

@@ -0,0 +1,7 @@
+significance: patch
+type: fixed
+links:
+  - "#2860"
+  - "#2861"
+entry: Net Sales reporting includes partially refunded transactions, and
+  transactions from orders regardless of status.

.changelogs/fix_date-range-widget-2.yml

@@ -0,0 +1,6 @@
+significance: patch
+type: fixed
+links:
+  - "#2860"
+entry: On-hold, pending cancellation, cancelled and expired orders now included
+  in "# of Sales" widget.

.changelogs/fix_date-range-widget-3.yml

@@ -0,0 +1,3 @@
+significance: patch
+type: added
+entry: New "# of transactions" sales reporting widget.

.changelogs/fix_date-range-widget.yml

@@ -0,0 +1,6 @@
+significance: minor
+type: fixed
+links:
+  - "#2858"
+entry: Fixes sales reporting for transactions or orders that happened between
+  23:23:59 and midnight.

includes/abstracts/abstract.llms.analytics.widget.php

@@ -116,21 +116,18 @@ protected function get_posted_dates() {
 
 		$dates = llms_filter_input_sanitize_string( INPUT_POST, 'dates', array( FILTER_REQUIRE_ARRAY ) );
 		return $dates ? $dates : '';
-
 	}
 
 	protected function get_posted_courses() {
 
 		$courses = llms_filter_input( INPUT_POST, 'courses', FILTER_SANITIZE_NUMBER_INT, FILTER_REQUIRE_ARRAY );
 		return $courses ? $courses : array();
-
 	}
 
 	protected function get_posted_memberships() {
 
 		$memberships = llms_filter_input( INPUT_POST, 'memberships', FILTER_SANITIZE_NUMBER_INT, FILTER_REQUIRE_ARRAY );
 		return $memberships ? $memberships : array();
-
 	}
 
 	protected function get_posted_posts() {
@@ -140,7 +137,6 @@ protected function get_posted_posts() {
 	protected function get_posted_students() {
 		$students = llms_filter_input( INPUT_POST, 'students', FILTER_SANITIZE_NUMBER_INT, FILTER_REQUIRE_ARRAY );
 		return $students ? $students : array();
-
 	}
 
 	protected function get_prepared_query() {
@@ -169,20 +165,26 @@ protected function format_date( $date, $type ) {
 				break;
 
 			case 'end':
-				$date .= ' 23:23:59';
+				/**
+				 * Return 00:00:00 on the next day after this date, using PHP datetime functions to avoid issues with daylight savings time or leap years.
+				 *
+				 * 23:59:59 is not a safe way to capture the end of day in newer versions of MySQL, if the transaction happened at (say) 23:59:59.999.
+				 */
+				$end_date = new DateTime( $date );
+				$end_date->modify( '+1 day' );
+
+				$date = $end_date->format( 'Y-m-d' ) . ' 00:00:00';
 
 				break;
 
 		}
 
 		return $date;
-
 	}
 
 	protected function is_error() {
 
 		return ( $this->success ) ? false : true;
-
 	}
 
 	protected function set_order_data_query( $args = array() ) {
@@ -235,7 +237,7 @@ protected function set_order_data_query( $args = array() ) {
 		$order_dates = '';
 		if ( $date_range ) {
 			$dates              = $this->get_posted_dates();
-			$order_dates        = "AND orders.{$date_field} BETWEEN CAST( %s AS DATETIME ) AND CAST( %s AS DATETIME )";
+			$order_dates        = "AND orders.{$date_field} >= %s AND orders.{$date_field} < %s";
 			$this->query_vars[] = $this->format_date( $dates['start'], 'start' );
 			$this->query_vars[] = $this->format_date( $dates['end'], 'end' );
 		}
@@ -291,7 +293,6 @@ protected function set_order_data_query( $args = array() ) {
 							{$wheres_clause}
 						{$order_clause}
 						;";
-
 	}
 
 	/**
@@ -333,7 +334,6 @@ protected function query() {
 			$this->message = $wpdb->last_error;
 
 		}
-
 	}
 
 	/**
@@ -384,7 +384,6 @@ public function can_be_processed() {
 			$widget_name,
 			$this
 		);
-
 	}
 
 	/**
@@ -409,8 +408,5 @@ public function output() {
 		header( 'Content-Type: application/json' );
 		echo wp_json_encode( $this );
 		wp_die();
-
 	}
-
-
 }

includes/admin/reporting/tabs/class.llms.admin.reporting.tab.sales.php

@@ -27,7 +27,6 @@ public function __construct() {
 
 		add_action( 'llms_reporting_after_nav', array( $this, 'output_filters' ), 10, 1 );
 		add_action( 'llms_reporting_content_sales', array( $this, 'output' ) );
-
 	}
 
 	public static function get_filter_data() {
@@ -49,7 +48,6 @@ public static function get_filter_data() {
 		$data['date_end']   = $data['dates']['end'];
 
 		return $data;
-
 	}
 
 	/**
@@ -68,7 +66,7 @@ public function get_widget_data() {
 						'title'   => __( '# of Sales', 'lifterlms' ),
 						'cols'    => '1-4',
 						'content' => __( 'loading...', 'lifterlms' ),
-						'info'    => __( 'Number of new active or completed orders placed within this period', 'lifterlms' ),
+						'info'    => __( 'Number of new non-refunded orders placed within this period', 'lifterlms' ),
 					),
 					'sold'     => array(
 						'title'   => __( 'Net Sales', 'lifterlms' ),
@@ -90,24 +88,24 @@ public function get_widget_data() {
 					),
 				),
 				array(
-					// 'revenue' => array(
-					// 'title' => __( 'Grosse Revenue', 'lifterlms' ),
-					// 'cols' => '1-4',
-					// 'content' => __( 'loading...', 'lifterlms' ),
-					// 'info' => __( 'Total of all transactions minus all refunds processed during this period', 'lifterlms' ),
-					// ),
-					'coupons'   => array(
+					'coupons'      => array(
 						'title'   => __( '# of Coupons Used', 'lifterlms' ),
 						'cols'    => '1-4',
 						'content' => __( 'loading...', 'lifterlms' ),
 						'info'    => __( 'Number of orders completed using coupons during this period', 'lifterlms' ),
 					),
-					'discounts' => array(
+					'discounts'    => array(
 						'title'   => __( 'Amount of Coupons', 'lifterlms' ),
 						'cols'    => '1-4',
 						'content' => __( 'loading...', 'lifterlms' ),
 						'info'    => __( 'Total amount of coupons used during this period', 'lifterlms' ),
 					),
+					'transactions' => array(
+						'title'   => __( '# of Transactions', 'lifterlms' ),
+						'cols'    => '1-4',
+						'content' => __( 'loading...', 'lifterlms' ),
+						'info'    => __( 'Number of transactions within this period', 'lifterlms' ),
+					),
 				),
 			)
 		);
@@ -129,7 +127,6 @@ public function output() {
 				'widget_data' => $this->get_widget_data(),
 			)
 		);
-
 	}
 
 	/**
@@ -146,8 +143,6 @@ public function output_filters( $tab ) {
 			llms_get_template( 'admin/reporting/nav-filters.php', self::get_filter_data() );
 
 		}
-
 	}
-
 }
 return new LLMS_Admin_Reporting_Tab_Sales();

includes/admin/reporting/widgets/class.llms.analytics.widget.coursecompletions.php

@@ -60,7 +60,8 @@ public function set_query() {
 							    upm.meta_key = '_is_complete'
 							AND p.post_type = 'course'
 							AND upm.meta_value = 'yes'
-							AND upm.updated_date BETWEEN CAST( %s AS DATETIME ) AND CAST( %s AS  DATETIME )
+							AND upm.updated_date >= %s
+						    AND upm.updated_date < %s
 							{$student_ids}
 							{$lesson_ids}
 						;";
@@ -69,7 +70,6 @@ public function set_query() {
 			$this->format_date( $dates['start'], 'start' ),
 			$this->format_date( $dates['end'], 'end' ),
 		);
-
 	}
 
 	protected function format_response() {
@@ -79,7 +79,5 @@ protected function format_response() {
 			return count( $this->get_results() );
 
 		}
-
 	}
-
 }

includes/admin/reporting/widgets/class.llms.analytics.widget.enrollments.php

@@ -59,7 +59,8 @@ public function set_query() {
 						WHERE
 							    meta_key = '_status'
 							AND ( meta_value = 'Enrolled' OR meta_value = 'enrolled' )
-							AND updated_date BETWEEN CAST( %s AS DATETIME ) AND CAST( %s AS  DATETIME )
+							AND updated_date >= %s
+						    AND updated_date < %s
 							{$student_ids}
 							{$product_ids}
 						;";
@@ -68,7 +69,6 @@ public function set_query() {
 			$this->format_date( $dates['start'], 'start' ),
 			$this->format_date( $dates['end'], 'end' ),
 		);
-
 	}
 
 	protected function format_response() {
@@ -78,7 +78,5 @@ protected function format_response() {
 			return count( $this->get_results() );
 
 		}
-
 	}
-
 }

includes/admin/reporting/widgets/class.llms.analytics.widget.lessoncompletions.php

@@ -58,7 +58,6 @@ private function get_lesson_ids( $products ) {
 		}
 
 		return $lessons;
-
 	}
 
 	public function set_query() {
@@ -90,7 +89,8 @@ public function set_query() {
 							    upm.meta_key = '_is_complete'
 							AND p.post_type = 'lesson'
 							AND upm.meta_value = 'yes'
-							AND upm.updated_date BETWEEN CAST( %s AS DATETIME ) AND CAST( %s AS  DATETIME )
+							AND upm.updated_date >= %s
+						    AND upm.updated_date < %s
 							{$student_ids}
 							{$lesson_ids}
 						;";
@@ -99,7 +99,6 @@ public function set_query() {
 			$this->format_date( $dates['start'], 'start' ),
 			$this->format_date( $dates['end'], 'end' ),
 		);
-
 	}
 
 	protected function format_response() {
@@ -109,7 +108,5 @@ protected function format_response() {
 			return count( $this->get_results() );
 
 		}
-
 	}
-
 }

includes/admin/reporting/widgets/class.llms.analytics.widget.refunded.php

@@ -95,12 +95,12 @@ public function set_query() {
 						WHERE
 						        ( txns.post_status = 'llms-txn-succeeded' OR txns.post_status = 'llms-txn-refunded' )
 						    AND txns.post_type = 'llms_transaction'
-							AND txns.post_date BETWEEN CAST( %s AS DATETIME ) AND CAST( %s AS DATETIME )
+							AND txns.post_date >= %s
+							AND txns.post_date < %s
 							AND refund.meta_key = '_llms_refund_amount'
 							{$txn_meta_where}
 							ORDER BY txns.post_modified ASC
 						;";
-
 	}
 
 	/**
@@ -117,7 +117,5 @@ protected function format_response() {
 			return llms_price_raw( floatval( is_array( $results ) ? array_sum( wp_list_pluck( $results, 'amount' ) ) : $results ) );
 
 		}
-
 	}
-
 }

includes/admin/reporting/widgets/class.llms.analytics.widget.registrations.php

@@ -48,15 +48,15 @@ public function set_query() {
 		$this->query = "SELECT user_registered AS date
 						FROM {$wpdb->users}
 						WHERE
-							user_registered BETWEEN CAST( %s AS DATETIME ) AND CAST( %s AS  DATETIME )
+							user_registered >= %s
+						    AND user_registered < %s
 							{$student_ids}
 						;";
 
 		$this->query_vars = array(
 			$this->format_date( $dates['start'], 'start' ),
 			$this->format_date( $dates['end'], 'end' ),
 		);
-
 	}
 
 	protected function format_response() {
@@ -66,7 +66,5 @@ protected function format_response() {
 			return count( $this->get_results() );
 
 		}
-
 	}
-
 }

includes/admin/reporting/widgets/class.llms.analytics.widget.revenue.php

@@ -91,10 +91,10 @@ public function set_query() {
 						WHERE
 						        ( txns.post_status = 'llms-txn-succeeded' OR txns.post_status = 'llms-txn-refunded' )
 						    AND txns.post_type = 'llms_transaction'
-							AND txns.post_date BETWEEN CAST( %s AS DATETIME ) AND CAST( %s AS DATETIME )
+							AND txns.post_date >= %s
+							AND txns.post_date < %s
 							{$txn_meta_where}
 						;";
-
 	}
 
 	protected function format_response() {
@@ -104,7 +104,5 @@ protected function format_response() {
 			return llms_price_raw( floatval( $this->get_results() ) );
 
 		}
-
 	}
-
 }

includes/admin/reporting/widgets/class.llms.analytics.widget.sales.php

@@ -46,10 +46,13 @@ public function set_query() {
 				'statuses'       => array(
 					'llms-active',
 					'llms-completed',
+					'llms-on-hold',
+					'llms-pending-cancel',
+					'llms-cancelled',
+					'llms-expired',
 				),
 			)
 		);
-
 	}
 
 	protected function format_response() {
@@ -59,7 +62,5 @@ protected function format_response() {
 			return count( $this->get_results() );
 
 		}
-
 	}
-
 }