Disable tests that are now superfluous

* Simple binds: matchSimpleBinds() also handles the bind marker, and more elaborately: handles enclosing quotes, checks the number of markers…
* Named binds: there just won't be replacements for matchNamedBinds() to make.

As a difference, if there are no markers/placeholders, compileBinds() will return the original SQL string instead of null. So it's possible to use a query string without markers/placeholders, and a $binds array with superflous binds; not ideal, but not problematic either.

Author: vlakoff

system/Database/Query.php

@@ -288,18 +288,14 @@ protected function compileBinds()
         }
 
         if (is_numeric(key(array_slice($binds, 0, 1)))) {
-            if (empty($this->bindMarker) || strpos($sql, $this->bindMarker) === false) {
+            if (empty($this->bindMarker)) {
                 return;
             }
 
             $ml  = strlen($this->bindMarker);
 
             $this->finalQueryString = $this->matchSimpleBinds($sql, $binds, $bindCount, $ml);
         } else {
-            if (! preg_match('/:(?!=).+:/', $sql)) {
-                return;
-            }
-
             // Reverse the binds so that duplicate named binds
             // will be processed prior to the original binds.
             $binds = array_reverse($binds);