save post meta only for course post type (#2570)

* save post meta only for course post type

* fix coding standards

* Adding changelog.

* Implementing changes in new template location.

* Revert "Implementing changes in new template location."

This reverts commit 2af59ba.

---------

Co-authored-by: Brian Hogg <brian@lifterlms.com>
Co-authored-by: Brian Hogg <brian@bhconsulting.ca>
Co-authored-by: Jason Coleman <33220397+ideadude@users.noreply.github.com>

Author: 3 people

.changelogs/issue-887.yml

@@ -0,0 +1,7 @@
+significance: patch
+type: fixed
+links:
+  - "#887"
+attributions:
+  - "@bsetiawan88"
+entry: Avoids saving review meta information for non-courses.

includes/admin/class.llms.admin.reviews.php

@@ -179,6 +179,15 @@ public function add_review_meta_boxes( $content ) {
 	 * @return void
 	 */
 	public function save_review_meta_boxes() {
+		$post_id = llms_filter_input( INPUT_POST, 'post_ID', FILTER_SANITIZE_NUMBER_INT );
+		if ( ! $post_id ) {
+			return;
+		}
+
+		$post_type = get_post_type( $post_id );
+		if ( 'course' !== $post_type ) {
+			return;
+		}
 
 		// phpcs:disable WordPress.Security.NonceVerification.Missing -- Nonce verified by core before triggering hook.
 
@@ -187,14 +196,10 @@ public function save_review_meta_boxes() {
 		$num      = ( isset( $_POST['_llms_num_reviews'] ) ) ? llms_filter_input_sanitize_string( INPUT_POST, '_llms_num_reviews' ) : 0;
 		$multiple = ( isset( $_POST['_llms_multiple_reviews_disabled'] ) ) ? llms_filter_input_sanitize_string( INPUT_POST, '_llms_multiple_reviews_disabled' ) : '';
 
-		$post_id = llms_filter_input( INPUT_POST, 'post_ID', FILTER_SANITIZE_NUMBER_INT );
-
-		if ( $post_id ) {
-			update_post_meta( $post_id, '_llms_reviews_enabled', $enabled );
-			update_post_meta( $post_id, '_llms_display_reviews', $display );
-			update_post_meta( $post_id, '_llms_num_reviews', $num );
-			update_post_meta( $post_id, '_llms_multiple_reviews_disabled', $multiple );
-		}
+		update_post_meta( $post_id, '_llms_reviews_enabled', $enabled );
+		update_post_meta( $post_id, '_llms_display_reviews', $display );
+		update_post_meta( $post_id, '_llms_num_reviews', $num );
+		update_post_meta( $post_id, '_llms_multiple_reviews_disabled', $multiple );
 		// phpcs:enable WordPress.Security.NonceVerification.Missing
 	}
 }