Changeset 718 for trunk/lib

Timestamp:

Feb 17, 2020 11:01:55 PM (4 years ago)

Author:

anonymous

Message:

Minor fixes

Location:

trunk/lib

Files:

• Lock.inc.php (modified) (1 diff)
• PDO.inc.php (modified) (3 diffs)
• Utilities.inc.php (modified) (3 diffs)

trunk/lib/Lock.inc.php

@@ -217 +217 @@
             return true;
         } else {
-            $app->logMsg(sprintf('No locked record: %s %s %s', $record_table_or_lock_id, $record_key, $record_val), LOG_DEBUG, __FILE__, __LINE__);
             return false;
         }

trunk/lib/PDO.inc.php

@@ -401 +401 @@
             $stmt = $this->dbh->query($query);
             if (!$stmt) {
-                throw new Exception('PDO::query returned false');
+                throw new \Exception('PDO::query returned false');
             }
         } catch (\Exception $e) {
@@ -454 +454 @@
             $stmt = $this->dbh->prepare($query, ...$params);
             if (!$stmt) {
-                throw new Exception('PDO::query returned false');
+                throw new \Exception('PDO::prepare returned false');
             }
         } catch (\PDOException $e) {
@@ -499 +499 @@
 
     /*
-    *
-    *
-    * @access   public
-    * @param
-    * @return
+    * Remove unsafe characters from SQL identifiers (tables, views, indexes, columns, and constraints).
+    *
+    * @access   public
+    * @param    string  $idname     Identifier name.
+    * @return   string              Clean string.
     * @author   Quinn Comendant <quinn@strangecode.com>
     * @since    09 Jul 2019 18:32:55
     */
-    static function sanitizeIdentifier($str)
-    {
-        return preg_replace('/\W/u', '', $str);
+    static function sanitizeIdentifier($idname)
+    {
+        return preg_replace('/\W/u', '', $idname);
     }
 

trunk/lib/Utilities.inc.php

@@ -677 +677 @@
     $val = trim(ini_get($val));
     if ($val != '') {
-        $last = strtolower($val{strlen($val) - 1});
-    } else {
-        $last = '';
-    }
-    switch ($last) {
-        // The 'G' modifier is available since PHP 5.1.0
-        case 'g':
-            $val *= 1024;
-        case 'm':
-            $val *= 1024;
-        case 'k':
-            $val *= 1024;
+        $unit = strtolower($val{strlen($val) - 1});
+        $val = preg_replace('/\D/', '', $val);
+
+        switch ($unit) {
+            // No `break`, so these multiplications are cumulative.
+            case 'g':
+                $val *= 1024;
+            case 'm':
+                $val *= 1024;
+            case 'k':
+                $val *= 1024;
+        }
     }
 
@@ -1063 +1063 @@
     $app =& App::getInstance();
 
-    if ('POST' == getenv('REQUEST_METHOD') && null === $key) {
-        return dispelMagicQuotes($_POST, $app->getParam('always_dispel_magicquotes'));
-    } else if ('GET' == getenv('REQUEST_METHOD') && null === $key) {
-        return dispelMagicQuotes($_GET, $app->getParam('always_dispel_magicquotes'));
+    if (null === $key) {
+        // Return entire array.
+        switch (strtoupper(getenv('REQUEST_METHOD'))) {
+        case 'POST':
+            return dispelMagicQuotes($_POST, $app->getParam('always_dispel_magicquotes'));
+
+        case 'GET':
+            return dispelMagicQuotes($_GET, $app->getParam('always_dispel_magicquotes'));
+
+        default:
+            return dispelMagicQuotes($_REQUEST, $app->getParam('always_dispel_magicquotes'));
+        }
     }
 
@@ -1139 +1147 @@
         break;
     }
+
+    $_REQUEST[$key] = $val;
 }