From 09f40a179db6309b77b6a8b2014eb212208e890f Mon Sep 17 00:00:00 2001
From: Joey Kudish <joachim.kudish@gmail.com>
Date: Mon, 27 Nov 2023 19:37:26 -0800
Subject: [PATCH] implement product_id and product_price_id filters

implement filtering by product ID or product price ID

while implementing this, I discovered a bug in the EDD query system. Specifically, when using `date_query` in combination with `product_id`, the query produces an invalid query error "WordPress database error Column 'date_created' in where clause is ambiguous for query" because the `date_query` is not properly scoped to the `edd_orders` table.

This bug has been reported here: https://github.com/awesomemotive/easy-digital-downloads/issues/9699
and I will work on a PR to fix the issue.

A workaround in the form of applying the more specific `date_created_query` has been implemented as a workaround for now
---
 includes/edd-export-command.php | 47 +++++++++++++++++++++++----------
 1 file changed, 33 insertions(+), 14 deletions(-)

diff --git a/includes/edd-export-command.php b/includes/edd-export-command.php
index 74de40f..29c68c1 100644
--- a/includes/edd-export-command.php
+++ b/includes/edd-export-command.php
@@ -80,6 +80,12 @@ class EDD_Export_Command extends WP_CLI_Command {
 		 * [--emails=<emails>]
 		 * : Filter by an array of emails
 		 *
+		 * [--product_id=<product_id>]
+		 * : Filter by a specific product ID
+		 *
+		 * [--product_price_id=<product_price_id>]
+		 * : Filter by a specific product price ID
+		 *
 		 *  ## EXAMPLES
 		 *
 		 *      # Export default fields to a CLI table
@@ -105,23 +111,25 @@ public function payments( $args, $assoc_args ) {
 
 			$assoc_args = WP_CLI\Utils\parse_shell_arrays( $assoc_args, array( 'fields', 'customer_ids', 'emails' ) );
 			$args       = wp_parse_args( $assoc_args, array(
-				'output-format' => 'table',
-				'destination'   => wp_upload_dir()['basedir'] . '/edd-exports',
-				'days'          => 30,
-				'start'         => null,
-				'end'           => null,
-				'per_page'      => 1000,
-				'minamount'     => null,
-				'maxamount'     => null,
-				'status'        => null,
-				'customer_ids'  => [],
-				'emails'        => [],
-				'max'           => in_array( $assoc_args['output-format'], array(
+				'output-format'    => 'table',
+				'destination'      => wp_upload_dir()['basedir'] . '/edd-exports',
+				'days'             => 30,
+				'start'            => null,
+				'end'              => null,
+				'per_page'         => 1000,
+				'minamount'        => null,
+				'maxamount'        => null,
+				'status'           => null,
+				'customer_ids'     => [],
+				'emails'           => [],
+				'product_id'       => null,
+				'product_price_id' => null,
+				'max'              => in_array( $assoc_args['output-format'], array(
 					'table',
 					'csv',
 					'json'
 				) ) ? 100 : null,
-				'fields'        => array(
+				'fields'           => array(
 					'customer_id',
 					'customer_email',
 					'order_id',
@@ -260,7 +268,18 @@ private function determine_payment_query( array $args ) {
 				$query['email__in'] = $args['emails'];
 			}
 
-			$query['date_query'] = $this->get_date_query( $args['days'], $args['start'], $args['end'] );
+			if ( ! empty( $args['product_id'] ) ) {
+				$query['product_id'] = $args['product_id'];
+			}
+
+			if ( ! empty( $args['product_price_id'] ) ) {
+				$query['product_price_id'] = $args['product_price_id'];
+			}
+
+			// when using `date_query` in combination with `product_id`, it produces an invalid query error "WordPress database error Column 'date_created' in where clause is ambiguous for query" because the `date_query` is not properly scoped to the `edd_orders` table. This bug has been reported here: https://github.com/awesomemotive/easy-digital-downloads/issues/9699
+			// For now, we'll use date_created_query but this is a bug that should be fixed in EDD core - and then we should use `date_query` here instead.
+//			$query['date_query'] = $this->get_date_query( $args['days'], $args['start'], $args['end'] );
+			$query['date_created_query'] = $this->get_date_query( $args['days'], $args['start'], $args['end'] );
 
 			// there's no "total" query arg in the \EDD\Database\Queries\Order class, so we have to use a filter to apply a custom where clause
 			if ( is_numeric( $args['minamount'] ) ) {