= 4.2.6.5 =

~ Tweak: method get_total_user_enrolled, get_total_user_enrolled_or_purchased
LearnPress · Apr 20, 2024 · 99236bb · 99236bb
1 parent a479fbb
commit 99236bb
Showing 1 changed file with 32 additions and 28 deletions.
diff --git a/inc/Databases/class-lp-course-db.php b/inc/Databases/class-lp-course-db.php
@@ -142,8 +142,10 @@ public function get_sections( int $course_id = 0 ) {
 	 * @param int $user_id
 	 *
 	 * @return int
+	 * @deprecated 4.2.6.6 not use anywhere
 	 */
 	public function get_user_item_id( $order_id = 0, $course_id = 0, $user_id = 0 ): int {
+		_deprecated_function( __METHOD__, '4.2.6.6' );
 		$query = $this->wpdb->prepare(
 			"
 			SELECT user_item_id
@@ -313,23 +315,21 @@ public function get_user_ids_enrolled( int $course_id ) {
 	 * @version 1.0.0
 	 */
 	public function get_total_user_enrolled( int $course_id ): int {
-		$query = $this->wpdb->prepare(
-			"
-				SELECT COUNT(DISTINCT user_id) AS total FROM {$this->tb_lp_user_items}
-				WHERE item_id = %d
-				AND item_type = %s
-				AND user_id > 0
-				AND (status = %s OR status = %s )
-			",
-			$course_id,
-			LP_COURSE_CPT,
-			LP_COURSE_ENROLLED,
-			LP_COURSE_FINISHED
-		);
+		$filter              = new LP_User_Items_Filter();
+		$filter->only_fields = [ 'DISTINCT(user_id)' ];
+		$filter->item_id     = $course_id;
+		$filter->item_type   = LP_COURSE_CPT;
+		$filter->field_count = 'ui.user_id';
+		$filter->join[]      = "INNER JOIN {$this->tb_users} AS u ON ui.user_id = u.ID";
+		$filter->where[]     = 'AND ui.user_id > 0';
+		$filter->where[]     = $this->wpdb->prepare( 'AND ( ui.status = %s OR ui.status = %s )', LP_COURSE_ENROLLED, LP_COURSE_FINISHED );
+		$filter->query_count = true;
 
-		$this->check_execute_has_error();
+		$total            = 0;
+		$lp_user_items_db = LP_User_Items_DB::getInstance();
+		$lp_user_items_db->get_user_items( $filter, $total );
 
-		return (int) $this->wpdb->get_var( $query );
+		return $total;
 	}
 
 	/**
@@ -344,24 +344,26 @@ public function get_total_user_enrolled( int $course_id ): int {
 	 * @version 1.0.0
 	 */
 	public function get_total_user_enrolled_or_purchased( int $course_id ): int {
-		$query = $this->wpdb->prepare(
-			"
-				SELECT COUNT(DISTINCT user_id) AS total FROM {$this->tb_lp_user_items}
-				WHERE item_id = %d
-				AND item_type = %s
-				AND user_id > 0
-				AND (status = %s OR status = %s OR status = %s )
-			",
-			$course_id,
-			LP_COURSE_CPT,
+		$filter              = new LP_User_Items_Filter();
+		$filter->only_fields = [ 'DISTINCT(user_id)' ];
+		$filter->item_id     = $course_id;
+		$filter->item_type   = LP_COURSE_CPT;
+		$filter->field_count = 'ui.user_id';
+		$filter->join[]      = "INNER JOIN {$this->tb_users} AS u ON ui.user_id = u.ID";
+		$filter->where[]     = 'AND ui.user_id > 0';
+		$filter->where[]     = $this->wpdb->prepare(
+			'AND ( ui.status = %s OR ui.status = %s OR ui.status = %s )',
 			LP_COURSE_ENROLLED,
 			LP_COURSE_FINISHED,
 			LP_COURSE_PURCHASED
 		);
+		$filter->query_count = true;
 
-		$this->check_execute_has_error();
+		$total            = 0;
+		$lp_user_items_db = LP_User_Items_DB::getInstance();
+		$lp_user_items_db->get_user_items( $filter, $total );
 
-		return (int) $this->wpdb->get_var( $query );
+		return $total;
 	}
 
 	/**
@@ -673,7 +675,7 @@ public function count_course_free( LP_Course_Filter $filter ): int {
 			$filter->only_fields = [ 'COUNT( DISTINCT(ID) )' ];
 			$this->get_courses_sort_by_free( $filter );
 			$filter->return_string_query = true;
-			$query_count                 = LP_Course::get_courses( $filter, $count );
+			$query_count                 = $this->get_courses( $filter, $count );
 			$count                       = $this->wpdb->get_var( $query_count );
 		} catch ( Throwable $e ) {
 			error_log( __METHOD__ . ': ' . $e->getMessage() );
@@ -746,8 +748,10 @@ public function get_courses_order_by_popular( LP_Course_Filter &$filter ): LP_Co
 	 * @throws Exception
 	 * @version 1.0.0
 	 * @since 4.1.6
+	 * @deprecated 4.2.6.6 not use anywhere
 	 */
 	public function count_courses_publish_of_author( int $author_id ): LP_Course_Filter {
+		_deprecated_function( __METHOD__, '4.2.6.6' );
 		$filter_course              = new LP_Course_Filter();
 		$filter_course->only_fields = array( 'ID' );
 		$filter_course->post_author = $author_id;