Changeset 2646428

Timestamp:

12/19/2021 03:47:02 PM (3 years ago)

Author:

aigenseer

Message:

Fix save settings by duplicate entries

Location:

qr-scanner-redirect/trunk

Files:

• index.php (modified) (1 diff)
• readme.txt (modified) (2 diffs)
• sql/pluginsettings.class.php (modified) (3 diffs)

qr-scanner-redirect/trunk/index.php

@@ -4 +4 @@
  * Plugin URI:        https://github.com/aigenseer/qr-scanner-redirect
  * Description:       Wordpress web qr-scanner with redirect function
- * Version:           1.2.2
+ * Version:           1.2.3
  * Requires at least: 5.2
  * Requires PHP:      7.2

qr-scanner-redirect/trunk/readme.txt

@@ -5 +5 @@
 Tested up to: 5.8.1
 Requires PHP: 5.2.4
-Stable tag: 1.2.1
+Stable tag: 1.2.3
 License: GPLv3 or later
 License URI: http://www.gnu.org/licenses/gpl-3.0.html
@@ -50 +50 @@
 == Changelog ==
 
+= 1.2.3 =
+* cd
+
 = 1.2.2 =
 * Fix display metabox

qr-scanner-redirect/trunk/sql/pluginsettings.class.php

@@ -49 +49 @@
      * @return boolean
      */
-    public function is($name='')
-    {
-      $str = "SELECT COUNT(`id`) as count FROM `%s` WHERE `name` like '%s'";
-      $sql = sprintf($str, $this->tablename, $name);
+
+    /**
+     * [return the boolean if the entry value exist]
+     * @param  string  $name [name of value]
+     * @param $group
+     * @return bool
+     */
+    public function is($name='', $group)
+    {
+      $str = "SELECT COUNT(`id`) as count FROM `%s` WHERE `name` like '%s' AND `group` like '%s'";
+      $sql = sprintf($str, $this->tablename, $name, $group);
       return intval($this->wpdb->get_var($sql))>0;
     }
@@ -67 +74 @@
         $group = $this->DEFAULT_GROUP;
       }
-      if(!$this->is($name)){
+      if(!$this->is($name, $group)){
         $str = "INSERT INTO `%s` (`name`, `data`, `group`) values('%s', '%s', '%s') ";
         $sql = sprintf($str, $this->tablename, $name, $data, $group);
@@ -225 +232 @@
       $_values = $this->mergeWithDefaultValues($post, $group, false);
       foreach ($_values as $key => $value) {
-        if(!$value->change && $this->is($key)){
+        if(!$value->change && $this->is($key, $group)){
           switch ($value->type) {
             case 'boolean':