fix latest cs issues

gocodebox · Oct 26, 2023 · 8ab1f0e · 8ab1f0e
1 parent 640c512
commit 8ab1f0e
Show file tree

Hide file tree

Showing 9 changed files with 80 additions and 54 deletions.
diff --git a/composer.json b/composer.json
@@ -22,7 +22,6 @@
 
       "CHANGELOG.md",
       "composer.json",
-      "docker-compose.yml",
       "package.json",
       "package-lock.json",
       "README.md",
@@ -31,7 +30,9 @@
       "node_modules",
       "tests",
       "tmp",
-      "wordpress"
+      "wordpress",
+
+      "vendor"
     ]
   },
   "scripts": {

diff --git a/inc/labs/inc/beaver-builder/modules/course-author/includes/frontend.php b/inc/labs/inc/beaver-builder/modules/course-author/includes/frontend.php
@@ -1,18 +1,21 @@
 <?php
 /**
- * LifterLMS Course Author Module HTML
+ * LifterLMS Course Author Module HTML.
  *
- * @since    1.3.0
- * @version  1.3.0
+ * @package LifterLMS_Labs/Labs/BeaverBuilder/Modules/CourseAuthor/Templates
+ *
+ * @since 1.3.0
+ * @since [version] Escaped attributes.
+ * @version [version]
+ *
+ * @param $settings obj Beaver Builder module settings instance.
  */
 
-// Restrict direct access
-if ( ! defined( 'ABSPATH' ) ) {
-	exit; }
+defined( 'ABSPATH' ) || exit;
 
 $course_id = ! empty( $settings->llms_course_id ) ? $settings->llms_course_id : get_the_ID();
 ?>
 
 <div class="llms-lab-course-author">
-	[lifterlms_course_author avatar_size="<?php echo $settings->llms_avatar_size; ?>" bio="<?php echo $settings->llms_show_bio; ?>" course_id="<?php echo $course_id; ?>"]
+	[lifterlms_course_author avatar_size="<?php echo esc_attr( $settings->llms_avatar_size ); ?>" bio="<?php echo esc_attr( $settings->llms_show_bio ); ?>" course_id="<?php echo esc_attr( $course_id ); ?>"]
 </div>
diff --git a/inc/labs/inc/beaver-builder/modules/course-continue-button/includes/frontend.php b/inc/labs/inc/beaver-builder/modules/course-continue-button/includes/frontend.php
@@ -1,18 +1,21 @@
 <?php
 /**
- * LifterLMS Course Continue Button Module HTML
+ * LifterLMS Course Continue Button Module HTML.
  *
- * @since    1.3.0
- * @version  1.3.0
+ * @package LifterLMS_Labs/Labs/BeaverBuilder/Modules/CourseContinueButton/Templates
+ *
+ * @since 1.3.0
+ * @since [version] Escaped attributes.
+ * @version [version]
+ *
+ * @param $settings obj Beaver Builder module settings instance.
  */
 
-// Restrict direct access
-if ( ! defined( 'ABSPATH' ) ) {
-	exit; }
+defined( 'ABSPATH' ) || exit;
 
 $course_id = ! empty( $settings->llms_course_id ) ? $settings->llms_course_id : get_the_ID();
 ?>
 
 <div class="llms-lab-course-continue-button">
-	[lifterlms_course_continue_button course_id="<?php echo $course_id; ?>"]
+	[lifterlms_course_continue_button course_id="<?php echo esc_attr( $course_id ); ?>"]
 </div>
diff --git a/inc/labs/inc/beaver-builder/modules/course-meta-info/includes/frontend.php b/inc/labs/inc/beaver-builder/modules/course-meta-info/includes/frontend.php
@@ -1,18 +1,21 @@
 <?php
 /**
- * LifterLMS Course Meta Info Module HTML
+ * LifterLMS Course Meta Info Module HTML.
  *
- * @since    1.3.0
- * @version  1.3.0
+ * @package LifterLMS_Labs/Labs/BeaverBuilder/Modules/CourseMetaInfo/Templates
+ *
+ * @since 1.3.0
+ * @since [version] Escaped attributes.
+ * @version [version]
+ *
+ * @param $settings obj Beaver Builder module settings instance.
  */
 
-// Restrict direct access
-if ( ! defined( 'ABSPATH' ) ) {
-	exit; }
+defined( 'ABSPATH' ) || exit;
 
 $course_id = ! empty( $settings->llms_course_id ) ? $settings->llms_course_id : get_the_ID();
 ?>
 
 <div class="llms-lab-course-meta-info">
-	[lifterlms_course_meta_info course_id="<?php echo $course_id; ?>"]
+	[lifterlms_course_meta_info course_id="<?php echo esc_attr( $course_id ); ?>"]
 </div>
diff --git a/inc/labs/inc/beaver-builder/modules/course-progress-bar/includes/frontend.php b/inc/labs/inc/beaver-builder/modules/course-progress-bar/includes/frontend.php
@@ -1,15 +1,17 @@
 <?php
 /**
- * LifterLMS Course Progress Bar Module HTML
+ * LifterLMS Course Progress Bar Module HTML.
  *
- * @since    1.3.0
- * @version  1.3.0
+ * @package LifterLMS_Labs/Labs/BeaverBuilder/Modules/CourseProgressBar/Templates
+ *
+ * @since 1.3.0
+ * @since [version] Escaped attributes.
+ * @version [version]
+ *
+ * @param $settings obj Beaver Builder module settings instance.
  */
 
-// Restrict direct access
-if ( ! defined( 'ABSPATH' ) ) {
-	exit; }
-
+defined( 'ABSPATH' ) || exit;
 ?>
 
 <div class="llms-lab-course-progress-bar">

diff --git a/inc/labs/inc/beaver-builder/modules/course-syllabus/includes/frontend.php b/inc/labs/inc/beaver-builder/modules/course-syllabus/includes/frontend.php
@@ -1,15 +1,17 @@
 <?php
 /**
- * LifterLMS Course Syllabus Module HTML
+ * LifterLMS Course Syllabus Module HTML.
  *
- * @since    1.3.0
- * @version  1.3.0
+ * @package LifterLMS_Labs/Labs/BeaverBuilder/Modules/CourseSyllabus/Templates
+ *
+ * @since 1.3.0
+ * @since [version] Escaped attributes.
+ * @version [version]
+ *
+ * @param $settings obj Beaver Builder module settings instance.
  */
 
-// Restrict direct access
-if ( ! defined( 'ABSPATH' ) ) {
-	exit; }
-
+defined( 'ABSPATH' ) || exit;
 ?>
 
 <div class="llms-lab-course-syllabus">

diff --git a/inc/labs/inc/beaver-builder/modules/lesson-mark-complete/includes/frontend.php b/inc/labs/inc/beaver-builder/modules/lesson-mark-complete/includes/frontend.php
@@ -1,14 +1,17 @@
 <?php
 /**
- * LifterLMS Lesson Mark Complete Module HTML
+ * LifterLMS Lesson Mark Complete Module HTML.
  *
- * @since    1.3.0
- * @version  1.3.0
+ * @package LifterLMS_Labs/Labs/BeaverBuilder/Modules/LessonMarkComplete/Templates
+ *
+ * @since 1.3.0
+ * @since [version] Escaped strings.
+ * @version [version]
+ *
+ * @param $settings obj Beaver Builder module settings instance.
  */
 
-// Restrict direct access
-if ( ! defined( 'ABSPATH' ) ) {
-	exit; }
+defined( 'ABSPATH' ) || exit;
 ?>
 
 <div class="llms-lab-lesson-mark-complete">
@@ -21,7 +24,7 @@
 				'columns'     => 12,
 				'classes'     => 'llms-button-primary auto button',
 				'id'          => 'llms_mark_complete',
-				'value'       => apply_filters( 'lifterlms_mark_lesson_complete_button_text', __( 'Mark Complete', 'lifterlms-labs' ), llms_get_post( get_the_ID() ) ),
+				'value'       => apply_filters( 'lifterlms_mark_lesson_complete_button_text', esc_html__( 'Mark Complete', 'lifterlms-labs' ), llms_get_post( get_the_ID() ) ),
 				'last_column' => true,
 				'name'        => 'mark_complete',
 				'required'    => false,

diff --git a/inc/labs/inc/beaver-builder/modules/pricing-table/includes/frontend.php b/inc/labs/inc/beaver-builder/modules/pricing-table/includes/frontend.php
@@ -1,14 +1,17 @@
 <?php
 /**
- * LifterLMS Course/Membership Pricing Table Module HTML
+ * LifterLMS Course/Membership Pricing Table Module HTML.
  *
- * @since    1.3.0
- * @version  1.3.0
+ * @package LifterLMS_Labs/Labs/BeaverBuilder/Modules/PricingTable/Templates
+ *
+ * @since 1.3.0
+ * @since [version] Escaped attributes.
+ * @version [version]
+ *
+ * @param $settings obj Beaver Builder module settings instance.
  */
 
-// Restrict direct access
-if ( ! defined( 'ABSPATH' ) ) {
-	exit; }
+defined( 'ABSPATH' ) || exit;
 
 $product_id = $module->get_product_id( $settings );
 if ( ! $product_id ) {
@@ -18,6 +21,6 @@
 
 <div class="llms-lab-pricing-table">
 	<?php do_action( 'llms_lab_bb_before_pricing_table', $product_id ); ?>
-	[lifterlms_pricing_table product="<?php echo $product_id; ?>"]
+	[lifterlms_pricing_table product="<?php echo esc_attr( $product_id ); ?>"]
 	<?php do_action( 'llms_lab_bb_after_pricing_table', $product_id ); ?>
 </div>
diff --git a/phpcs.xml b/phpcs.xml
@@ -9,11 +9,17 @@
 		<exclude name="WordPress.Files.FileName.InvalidClassFileName" />
 	</rule>
 
+	<rule ref="LifterLMS.Commenting.FileComment.DuplicateSinceTag">
+		<exclude-pattern>inc/labs/inc/beaver-builder/modules/**/includes/frontend.php</exclude-pattern>
+	</rule>
+
+	<rule ref="Squiz.Commenting.FileComment.Missing">
+		<exclude-pattern>inc/labs/inc/beaver-builder/modules/**/js/frontend.js</exclude-pattern>
+	</rule>
+
 	<rule ref="WordPress.WP.I18n">
 		<properties>
-			<property name="text_domain" type="array">
-				<element value="lifterlms-labs" />
-			</property>
+			<property name="text_domain" value="lifterlms-labs" />
 		</properties>
 	</rule>