[Reviews] Extraneous metadata rows added for every post type

[actual-saurabh]

The review metadata saving is hooked into saved posts and no post type is checked in

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

Line 32 in cdbee2b

add_action( 'save_post', array( $this, 'save_review_meta_boxes' ) );

and

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

Lines 159 to 187 in cdbee2b

	/**
	* This function handles the logic to save the information that is contained
	* in the custom metabox. It looks at each of the values, then makes sure that
	* there are proper default values so that the program doesn't go crashy crashy
	* (nobody likes crashy crashy)
	*
	* @return void
	*/
	public function save_review_meta_boxes() {
	$enabled = (isset( $_POST['_llms_reviews_enabled'] )) ? $_POST['_llms_reviews_enabled'] : '';
	$display = (isset( $_POST['_llms_display_reviews'] )) ? $_POST['_llms_display_reviews'] : '';
	$num = (isset( $_POST['_llms_num_reviews'] )) ? $_POST['_llms_num_reviews'] : 0;
	$multiple = (isset( $_POST['_llms_multiple_reviews_disabled'] )) ? $_POST['_llms_multiple_reviews_disabled'] : '';
	if ( isset( $_POST['post_ID'] ) ) {
	update_post_meta( $_POST['post_ID'], '_llms_reviews_enabled', $enabled );
	}
	if ( isset( $_POST['post_ID'] ) ) {
	update_post_meta( $_POST['post_ID'], '_llms_display_reviews', $display );
	}
	if ( isset( $_POST['post_ID'] ) ) {
	update_post_meta( $_POST['post_ID'], '_llms_num_reviews', $num );
	}
	if ( isset( $_POST['post_ID'] ) ) {
	update_post_meta( $_POST['post_ID'], '_llms_multiple_reviews_disabled', $multiple );
	}
	}
	}

Additionally, even for posts where there are no reviews (metadata value is empty), the metadata is saved anyway leading to useless rows in the postmeta table. This example is of a certificate: