Changeset 2527305

Timestamp:

05/06/2021 02:04:32 PM (5 years ago)

Author:

subodhghulaxe

Message:

Updated tested up to 5.7.1
Improved security by sanitizing inputs

Location:

wp-change-default-from-email

Files:

• tags/1.1.4 (added)
• tags/1.1.4/WCDFE.php (added)
• tags/1.1.4/admin (added)
• tags/1.1.4/admin/WCDFE_Admin.php (added)
• tags/1.1.4/admin/index.php (added)
• tags/1.1.4/admin/pages (added)
• tags/1.1.4/admin/pages/index.php (added)
• tags/1.1.4/admin/pages/settings.php (added)
• tags/1.1.4/assets (added)
• tags/1.1.4/assets/css (added)
• tags/1.1.4/assets/css/admin.css (added)
• tags/1.1.4/assets/index.php (added)
• tags/1.1.4/index.php (added)
• tags/1.1.4/languages (added)
• tags/1.1.4/languages/WCDFE_translation-ar_SA.mo (added)
• tags/1.1.4/languages/WCDFE_translation-ar_SA.po (added)
• tags/1.1.4/languages/WCDFE_translation-de_DE.mo (added)
• tags/1.1.4/languages/WCDFE_translation-de_DE.po (added)
• tags/1.1.4/languages/WCDFE_translation-es_ES.mo (added)
• tags/1.1.4/languages/WCDFE_translation-es_ES.po (added)
• tags/1.1.4/languages/WCDFE_translation-fr_FR.mo (added)
• tags/1.1.4/languages/WCDFE_translation-fr_FR.po (added)
• tags/1.1.4/languages/WCDFE_translation-ru_RU.mo (added)
• tags/1.1.4/languages/WCDFE_translation-ru_RU.po (added)
• tags/1.1.4/readme.txt (added)
• tags/1.1.4/screenshot-1.jpg (added)
• tags/1.1.4/screenshot-2.jpg (added)
• tags/1.1.4/uninstall.php (added)
• trunk/WCDFE.php (modified) (1 diff)
• trunk/admin/WCDFE_Admin.php (modified) (3 diffs)
• trunk/admin/pages/settings.php (modified) (3 diffs)
• trunk/assets/css/admin.css (modified) (3 diffs)
• trunk/readme.txt (modified) (4 diffs)

wp-change-default-from-email/trunk/WCDFE.php

@@ -2 +2 @@
 /**
  * Plugin Name: WP Change Default From Email
- * Plugin URI: 
+ * Plugin URI: https://wordpress.org/plugins/wp-change-default-from-email/
  * Text Domain: WCDFE_translation
  * Domain Path: /languages
  * Description: A simple and easy way to change the from email address and from email name that appear on emails sent from WordPress.
- * Version: 1.1.3
+ * Version: 1.1.4
  * Author: Subodh Ghulaxe
  * Author URI: http://www.subodhghulaxe.com

wp-change-default-from-email/trunk/admin/WCDFE_Admin.php

@@ -1 +1 @@
 <?php
 
-//avoid direct calls to this file where wp core files not present
+// Avoid direct calls to this file where wp core files not present
 if (!function_exists ('add_action')) {
     header('Status: 403 Forbidden');
@@ -81 +81 @@
             check_ajax_referer( 'wcdfe-save-settings', 'security' );
 
-            $wcdfe_settings = isset($_POST['wcdfe_settings']) ? $_POST['wcdfe_settings'] : array();
+            // Sanitize inputs
+            $wcdfe_settings = isset($_POST['wcdfe_settings']) ? (array) array_map('sanitize_text_field', $_POST['wcdfe_settings']) : array();
 
             if( isset($wcdfe_settings['from_name']) && empty($wcdfe_settings['from_name']) ) {
@@ -94 +95 @@
             if(empty($error)) {
                 // Save setting in WordPress options
-                $result = update_option('wcdfe_settings',$wcdfe_settings);
-                if($result){
-                    $response['status'] = 'success';
-                    $response['message'] = __( 'Settings updated successfully.', WCDFE_TEXTDOMAIN );
-                } else {
-                    $response['status'] = 'error';
-                    $response['message'] = __( 'No settings were updated.', WCDFE_TEXTDOMAIN );
-                }
+                update_option('wcdfe_settings', $wcdfe_settings);
+                $response['status'] = 'success';
+                $response['message'] = __( 'Settings updated successfully.', WCDFE_TEXTDOMAIN );
             } else {
                 $response['status'] = 'error';

wp-change-default-from-email/trunk/admin/pages/settings.php

@@ -50 +50 @@
         
         <div class="wcdfe-save-settings-container">
-            <input type="submit" value="<?php echo __( 'Save Settings', WCDFE_TEXTDOMAIN ); ?>" class="button button-large button-primary" id="wcdfe-save-settings" name="save_settings">
+            <input type="submit" value="<?php echo __( 'Save Settings', WCDFE_TEXTDOMAIN ); ?>" class="button button-large button-primary wcdfe-button" id="wcdfe-save-settings" name="save_settings">
             <div id="wcdfe-error-message"></div>
         </div>
@@ -104 +104 @@
     jQuery("#wcdfe-save-settings").on('click', function(e) {
         e.preventDefault();
+        jQuery("#wcdfe-error-message").html('');
+        jQuery("#wcdfe-save-settings").val("Saving Settings ...");
+        jQuery("#wcdfe-save-settings").addClass("is-busy");
+
         var data = jQuery("#wcdfe-settings-form").serialize();
         jQuery.ajax({
@@ -118 +122 @@
                     jQuery("#wcdfe-error-message").html('<div class="error"><p><?php echo __( "No settings were saved.", WCDFE_TEXTDOMAIN ); ?></p></div>');
                 }
+                jQuery("#wcdfe-save-settings").val("Save Settings");
+        jQuery("#wcdfe-save-settings").removeClass("is-busy");
+            },
+            error: function (jqXHR, exception) {
+                var msg = '';
+                if (jqXHR.status === 0) {
+                        msg = 'Not connect.\n Verify Network.';
+                } else if (jqXHR.status == 404) {
+                        msg = 'Requested page not found. [404]';
+                } else if (jqXHR.status == 500) {
+                        msg = 'Internal Server Error [500].';
+                } else if (exception === 'parsererror') {
+                        msg = 'Requested JSON parse failed.';
+                } else if (exception === 'timeout') {
+                        msg = 'Time out error.';
+                } else if (exception === 'abort') {
+                        msg = 'Request aborted.';
+                } else {
+                        msg = 'Uncaught Error.\n' + jqXHR.responseText;
+                }
+                jQuery("#wcdfe-error-message").html('<div class="error"><p>' + msg + '</p></div>');
+                jQuery("#wcdfe-save-settings").val("Saving Settings");
+                jQuery("#wcdfe-save-settings").removeClass("is-busy");
             }
         });

wp-change-default-from-email/trunk/assets/css/admin.css

@@ +1 @@
+@keyframes wcdfe-button__busy-animation {
+    0% {
+      background-position: 200px 0;
+    }
+  }
+  .wcdfe-button.is-busy,
+  .wcdfe-button.is-busy:hover,
+  .wcdfe-button.is-busy:active,
+  .wcdfe-button.is-busy:focus {
+    animation: wcdfe-button__busy-animation 2.5s linear infinite;
+    background-image: linear-gradient(
+      -45deg,
+      #007cba 33%,
+      #005a87 0,
+      #005a87 70%,
+      #007cba 0
+    );
+}
 .wcdfe-row{
     clear: both;
@@ -26 +44 @@
 .wcdfe-onoffswitch {
     position: relative; width: 90px;
-    -webkit-user-select:none; -moz-user-select:none; -ms-user-select: none;
+    user-select: none;
+    -webkit-user-select: none; 
+    -moz-user-select: none; 
+    -ms-user-select: none;
 }
 .wcdfe-onoffswitch-checkbox {
@@ -81 +102 @@
     position: relative;
     display: inline-block;
+    margin-left: 10px;
 }
 #wcdfe-error-message .updated, #wcdfe-error-message .error {

wp-change-default-from-email/trunk/readme.txt

@@ -4 +4 @@
 Donate link: https://www.patreon.com/subodhghulaxe
 Requires at least: 2.7
-Tested up to: 5.7
-Stable tag: trunk
+Tested up to: 5.7.1
+Requires PHP: 5.2.4
+Stable tag: 1.1.4
 License: GPLv2 or later
 License URI: http://www.gnu.org/licenses/gpl-2.0.html
@@ -24 +25 @@
 * **Easy Settings.** Easy to enable/disable and change settings
 * **Clean & Well Commented Code.** Easy to enable/disable and change settings
+
+**Please support my other plugins:**
+<ol>
+  <li>
+    <a href="https://wordpress.org/plugins/change-login-page-logo/">Change Login Page Logo</a>
+  </li>
+  <li>
+    <a href="https://wordpress.org/plugins/hide-admin-bar-for-user-roles/">Hide Admin Bar For User Roles</a>
+  </li>
+</ol>
 
 **Translations/Languages:**
@@ -80 +91 @@
 Updated contact information
 
+= 1.1.4 =
+Updated tested up to 5.7.1
+Improved security by sanitizing inputs
+
 == Upgrade Notice ==
 = 1.0.0 =
@@ -97 +112 @@
 Updated screenshots
 Updated contact information
+
+= 1.1.4 =
+Updated tested up to 5.7.1
+Improved security by sanitizing inputs