From 71c0a77a318da09046bdba9fccd51d10708a04dc Mon Sep 17 00:00:00 2001 From: marcusquinn <6428977+marcusquinn@users.noreply.github.com> Date: Fri, 18 Apr 2025 18:16:31 +0100 Subject: [PATCH] Fix: Update PHPUnit bootstrap and fix comment punctuation --- admin/js/admin-scripts.js | 22 ++++++++-------- admin/js/update-source-selector.js | 40 +++++++++++++++--------------- admin/templates/modal.php | 2 +- includes/class-plugin.php | 2 +- tests/bootstrap.php | 4 +-- 5 files changed, 35 insertions(+), 35 deletions(-) diff --git a/admin/js/admin-scripts.js b/admin/js/admin-scripts.js index d0f4337..5afe43d 100644 --- a/admin/js/admin-scripts.js +++ b/admin/js/admin-scripts.js @@ -15,10 +15,10 @@ * Initialize */ init: function () { - // Initialize components + // Initialize components. this.initComponents(); - // Bind events + // Bind events. this.bindEvents(); }, @@ -26,17 +26,17 @@ * Initialize components */ initComponents: function () { - // Initialize any components here + // Initialize any components here. }, /** * Bind events */ bindEvents: function () { - // Example: Toggle sections + // Example: Toggle sections. $( '.wpst-toggle-section' ).on( 'click', this.toggleSection ); - // Example: Form submission + // Example: Form submission. $( '#wpst-settings-form' ).on( 'submit', this.handleFormSubmit ); }, @@ -67,10 +67,10 @@ const $submitButton = $form.find( 'input[type="submit"]' ); const formData = $form.serialize(); - // Disable submit button and show loading state + // Disable submit button and show loading state. $submitButton.prop( 'disabled', true ).addClass( 'loading' ); - // Send AJAX request + // Send AJAX request. $.ajax( { url: wpstData.ajaxUrl, @@ -91,7 +91,7 @@ WPSTAdmin.showNotice( 'error', 'An error occurred. Please try again.' ); }, complete: function () { - // Re-enable submit button and remove loading state + // Re-enable submit button and remove loading state. $submitButton.prop( 'disabled', false ).removeClass( 'loading' ); } } @@ -107,10 +107,10 @@ showNotice: function (type, message) { const $notice = $( '

' + message + '

' ); - // Add notice to the page + // Add notice to the page. $( '.wpst-notices' ).html( $notice ); - // Automatically remove notice after 5 seconds + // Automatically remove notice after 5 seconds. setTimeout( function () { $notice.fadeOut( @@ -125,7 +125,7 @@ } }; - // Initialize when document is ready + // Initialize when document is ready. $( document ).ready( function () { WPSTAdmin.init(); diff --git a/admin/js/update-source-selector.js b/admin/js/update-source-selector.js index ce55df7..d782527 100644 --- a/admin/js/update-source-selector.js +++ b/admin/js/update-source-selector.js @@ -25,10 +25,10 @@ * Initialize */ init: function () { - // Cache DOM elements + // Cache DOM elements. this.$modal = $( '#wpst-update-source-modal' ); - // Bind events + // Bind events. this.bindEvents(); }, @@ -36,10 +36,10 @@ * Bind events */ bindEvents: function () { - // Open modal when clicking on the update source link + // Open modal when clicking on the update source link. $( document ).on( 'click', '.wpst-update-source-selector', this.openModal.bind( this ) ); - // Close modal when clicking on the close button or outside the modal + // Close modal when clicking on the close button or outside the modal. this.$modal.on( 'click', '.wpst-modal-close', this.closeModal.bind( this ) ); $( document ).on( 'click', @@ -51,10 +51,10 @@ } ); - // Select source option + // Select source option. this.$modal.on( 'click', '.wpst-source-option', this.selectSource.bind( this ) ); - // Save source selection + // Save source selection. this.$modal.on( 'click', '#wpst-save-source', this.saveSource.bind( this ) ); }, @@ -83,14 +83,14 @@ selectSource: function (e) { const $option = $( e.currentTarget ); - // Update selected state + // Update selected state. this.$modal.find( '.wpst-source-option' ).removeClass( 'selected' ); $option.addClass( 'selected' ); - // Update radio button + // Update radio button. $option.find( 'input[type="radio"]' ).prop( 'checked', true ); - // Store selected source + // Store selected source. this.selectedSource = $option.find( 'input[type="radio"]' ).val(); }, @@ -98,31 +98,31 @@ * Save the selected source */ saveSource: function () { - // Validate selection + // Validate selection. if ( ! this.selectedSource) { this.showMessage( 'error', 'Please select an update source.' ); return; } - // Show loading state + // Show loading state. const $saveButton = $( '#wpst-save-source' ); $saveButton.prop( 'disabled', true ).html( ' Saving...' ); - // Send AJAX request + // Send AJAX request. $.ajax( { - url: wpstModalData.ajaxUrl, + url: wpstModalData.ajaxUrl, // WordPress AJAX URL. type: 'POST', data: { - action: 'wpst_set_update_source', - nonce: wpstModalData.nonce, + action: 'wpst_set_update_source', // AJAX action hook. + nonce: wpstModalData.nonce, // Security nonce. source: this.selectedSource }, success: function (response) { if (response.success) { WPSTUpdateSourceSelector.showMessage( 'success', response.data.message ); - // Close modal after a short delay + // Close modal after a short delay. setTimeout( function () { WPSTUpdateSourceSelector.closeModal(); @@ -137,7 +137,7 @@ WPSTUpdateSourceSelector.showMessage( 'error', 'An error occurred. Please try again.' ); }, complete: function () { - // Reset button state + // Reset button state. $saveButton.prop( 'disabled', false ).text( wpstModalData.i18n.confirm ); } } @@ -153,10 +153,10 @@ showMessage: function (type, message) { const $message = this.$modal.find( '.wpst-modal-message' ); - // Set message content and type + // Set message content and type. $message.html( message ).removeClass( 'success error' ).addClass( type ).show(); - // Hide message after a delay for success messages + // Hide message after a delay for success messages. if (type === 'success') { setTimeout( function () { @@ -168,7 +168,7 @@ } }; - // Initialize when document is ready + // Initialize when document is ready. $( document ).ready( function () { WPSTUpdateSourceSelector.init(); diff --git a/admin/templates/modal.php b/admin/templates/modal.php index 6f68b8e..9a7df91 100644 --- a/admin/templates/modal.php +++ b/admin/templates/modal.php @@ -26,7 +26,7 @@ if ( ! defined( 'ABSPATH' ) ) {
diff --git a/includes/class-plugin.php b/includes/class-plugin.php index daba398..fcc6a07 100644 --- a/includes/class-plugin.php +++ b/includes/class-plugin.php @@ -49,6 +49,6 @@ class Plugin { * Initialize the plugin */ public function init() { - // Initialization logic goes here + // Initialization logic goes here. } } diff --git a/tests/bootstrap.php b/tests/bootstrap.php index d150eea..814757b 100644 --- a/tests/bootstrap.php +++ b/tests/bootstrap.php @@ -24,6 +24,6 @@ define('WPST_VERSION', '0.1.0'); * Now we include any plugin files that we need to be able to run the tests. * This should be files that define the functions and classes you're going to test. */ -require_once WPST_PLUGIN_DIR . 'includes/core.php'; -require_once WPST_PLUGIN_DIR . 'includes/plugin.php'; +require_once WPST_PLUGIN_DIR . 'includes/class-core.php'; +require_once WPST_PLUGIN_DIR . 'includes/class-plugin.php'; require_once WPST_PLUGIN_DIR . 'admin/lib/admin.php';