LIMS-1025: Fix lab contact search

api/src/Database/DatabaseQueryBuilder.php

@@ -164,4 +164,16 @@ private function addBoundVariable($value): int
         array_push($this->query_bound_values, $value);
         return sizeof($this->query_bound_values);
     }
+
+    public static function getWhereSearch($searchValue, $fields, &$query_bound_values)
+    {
+        $where = " AND (0=1";
+
+        foreach ($fields as $field) {
+            array_push($query_bound_values, $searchValue);
+            $where .= " OR lower(" . $field . ") LIKE lower(CONCAT('%', :" . sizeof($query_bound_values) . ", '%'))";
+        }
+        $where .= ")";
+        return $where;
+    }
 }

api/src/Page/Contact.php

@@ -3,6 +3,7 @@
 namespace SynchWeb\Page;
 
 use SynchWeb\Page;
+use SynchWeb\Database\DatabaseQueryBuilder;
 
 class Contact extends Page
 {
@@ -46,9 +47,17 @@ function _get_contacts() {
                 $where .= ' AND c.labcontactid=:'.(sizeof($args)+1);
                 array_push($args, $this->arg('cid'));
             }
-
 
-            $tot = $this->db->pq("SELECT count(c.labcontactid) as tot FROM labcontact c  $where", $args);
+            if ($this->has_arg('s')) {
+                $fields = array('pe.givenname', 'pe.familyname', 'c.cardname',
+                    'l.name', 'l.address', 'l.city', 'l.country', 'pe.phonenumber', 'l.postcode');
+                $where = $where . DatabaseQueryBuilder::getWhereSearch($this->arg('s'), $fields, $args);
+            }
+
+            $tot = $this->db->pq("SELECT count(c.labcontactid) as tot FROM labcontact c
+                                    INNER JOIN person pe ON c.personid = pe.personid
+                                    INNER JOIN laboratory l ON l.laboratoryid = pe.laboratoryid
+                                    $where", $args);
             $tot = intval($tot[0]['TOT']);
 
             $pp = $this->has_arg('per_page') ? $this->arg('per_page') : 15;

api/tests/Database/DatabaseQueryBuilderTest.php

@@ -6,11 +6,13 @@
 use SynchWeb\Database\DatabaseParent;
 use SynchWeb\Database\DatabaseQueryBuilder;
 
-final class DatabaseHelperTest extends TestCase
+use function PHPUnit\Framework\assertEquals;
+
+final class DatabaseQueryBuilderTest extends TestCase
 {
     use \phpmock\phpunit\PHPMock;
 
-    private function create_database_helper($fields, $expected_sql, $expected_id_value = null)
+    private function create_database_helper($fields, $expected_sql=null, $expected_id_value = null)
     {
         $expected_values = [];
         foreach ($fields as $field)
@@ -168,4 +170,18 @@ public function testWithTwoValuesWhenInsertingDBInsertIsCorrect()
         }
         $dbh->insert($expected_table);
     }
+
+    public function testWithSingleIdWhereClauseGetWhereClauseReturnIt() {
+        $fields = ["a.field1", "b.field2"];
+        $expected_sql = " AND (0=1 OR lower({$fields[0]}) LIKE lower(CONCAT('%', :2, '%')) OR lower({$fields[1]}) LIKE lower(CONCAT('%', :3, '%')))";
+        $expectedSearchValue = "searchVal";
+
+        $element1 = "first";
+        $args=[$element1];
+        $sql = DatabaseQueryBuilder::getWhereSearch($expectedSearchValue, $fields, $args);
+
+        assertEquals($expected_sql, $sql);
+        assertEquals([$element1, $expectedSearchValue, $expectedSearchValue], $args);
+    }
+
 }