Convert strict_search to wildcard property and add psalm docs

Signed-off-by: Joas Schilling <coding@schilljs.com>

diff --git a/apps/dav/lib/CardDAV/AddressBookImpl.php b/apps/dav/lib/CardDAV/AddressBookImpl.php
index 6d95b0bc420edad733fa532708644acbc55512cc..8902156afe186293b0720eff94d029bca5041405 100644 (file)
--- a/apps/dav/lib/CardDAV/AddressBookImpl.php
+++ b/apps/dav/lib/CardDAV/AddressBookImpl.php
@@ -108,6 +108,7 @@ class AddressBookImpl implements IAddressBook {
         *      - 'limit' - Set a numeric limit for the search results
         *      - 'offset' - Set the offset for the limited search results
         *      - 'wildcard' - Whether the search should use wildcards
+        * @psalm-param array{types?: bool, escape_like_param?: bool, limit?: int, offset?: int, wildcard?: bool} $options
         * @return array an array of contacts which are arrays of key-value-pairs
         *  example result:
         *  [

diff --git a/apps/dav/lib/CardDAV/CardDavBackend.php b/apps/dav/lib/CardDAV/CardDavBackend.php
index 634870dbaa1e1aefd47a5afa3ee7997fb6654afa..0696fb2f5fb98766b857b1321126856ed5dee099 100644 (file)
--- a/apps/dav/lib/CardDAV/CardDavBackend.php
+++ b/apps/dav/lib/CardDAV/CardDavBackend.php
@@ -1005,6 +1005,7 @@ class CardDavBackend implements BackendInterface, SyncSupport {
         *    - 'limit' - Set a numeric limit for the search results
         *    - 'offset' - Set the offset for the limited search results
         *    - 'wildcard' - Whether the search should use wildcards
+        * @psalm-param array{escape_like_param?: bool, limit?: int, offset?: int, wildcard?: bool} $options
         * @return array an array of contacts which are arrays of key-value-pairs
         */
        public function search($addressBookId, $pattern, $searchProperties, $options = []): array {
@@ -1036,6 +1037,7 @@ class CardDavBackend implements BackendInterface, SyncSupport {
         * @param string $pattern
         * @param array $searchProperties
         * @param array $options
+        * @psalm-param array{types?: bool, escape_like_param?: bool, limit?: int, offset?: int, wildcard?: bool} $options
         * @return array
         */
        private function searchByAddressBookIds(array $addressBookIds,

diff --git a/lib/private/ContactsManager.php b/lib/private/ContactsManager.php
index 2751f22148570719f3c7cd0ad19f6ce32738fca6..f83315ade2444ff132f687482905589f64be43d1 100644 (file)
--- a/lib/private/ContactsManager.php
+++ b/lib/private/ContactsManager.php
@@ -43,8 +43,10 @@ class ContactsManager implements IManager {
         *      - 'escape_like_param' - If set to false wildcards _ and % are not escaped
         *      - 'limit' - Set a numeric limit for the search results
         *      - 'offset' - Set the offset for the limited search results
-        *      - 'enumeration' - Whether user enumeration on system address book is allowed
-        *      - 'fullmatch' - Whether matching on full detail in system address book is allowed
+        *      - 'enumeration' - (since 23.0.0) Whether user enumeration on system address book is allowed
+        *      - 'fullmatch' - (since 23.0.0) Whether matching on full detail in system address book is allowed
+        *      - 'strict_search' - (since 23.0.0) Whether the search pattern is full string or partial search
+        * @psalm-param array{escape_like_param?: bool, limit?: int, offset?: int, enumeration?: bool, fullmatch?: bool, strict_search?: bool} $options
         * @return array an array of contacts which are arrays of key-value-pairs
         */
        public function search($pattern, $searchProperties = [], $options = []) {
@@ -52,6 +54,7 @@ class ContactsManager implements IManager {
                $result = [];
                foreach ($this->addressBooks as $addressBook) {
                        $searchOptions = $options;
+                       $strictSearch = array_key_exists('strict_search', $options) && $options['strict_search'] === true;
 
                        if ($addressBook->isSystemAddressBook()) {
                                $fullMatch = !\array_key_exists('fullmatch', $options) || $options['fullmatch'] !== false;
@@ -59,7 +62,13 @@ class ContactsManager implements IManager {
                                        // Neither full match is allowed, so skip the system address book
                                        continue;
                                }
-                               $searchOptions['wildcard'] = !\array_key_exists('enumeration', $options) || $options['enumeration'] !== false;
+                               if ($strictSearch) {
+                                       $searchOptions['wildcard'] = false;
+                               } else {
+                                       $searchOptions['wildcard'] = !\array_key_exists('enumeration', $options) || $options['enumeration'] !== false;
+                               }
+                       } else {
+                               $searchOptions['wildcard'] = !$strictSearch;
                        }
 
                        $r = $addressBook->search($pattern, $searchProperties, $searchOptions);

diff --git a/lib/public/Contacts/IManager.php b/lib/public/Contacts/IManager.php
index 827c4f5724b6b5c0e817a2853d52b2fd96308e73..544eaaafee31ecc7f6acc0e93844f252ea4e6568 100644 (file)
--- a/lib/public/Contacts/IManager.php
+++ b/lib/public/Contacts/IManager.php
@@ -99,8 +99,10 @@ interface IManager {
         *      - 'escape_like_param' - If set to false wildcards _ and % are not escaped
         *      - 'limit' - Set a numeric limit for the search results
         *      - 'offset' - Set the offset for the limited search results
-        *      - 'enumeration' - Whether user enumeration on system address book is allowed
-        *      - 'fullmatch' - Whether matching on full detail in system addresss book is allowed
+        *      - 'enumeration' - (since 23.0.0) Whether user enumeration on system address book is allowed
+        *      - 'fullmatch' - (since 23.0.0) Whether matching on full detail in system addresss book is allowed
+        *      - 'strict_search' - (since 23.0.0) Whether the search pattern is full string or partial search
+        * @psalm-param array{escape_like_param?: bool, limit?: int, offset?: int, enumeration?: bool, fullmatch?: bool, strict_search?: bool} $options
         * @return array an array of contacts which are arrays of key-value-pairs
         * @since 6.0.0
         */

diff --git a/lib/public/IAddressBook.php b/lib/public/IAddressBook.php
index c2f06a49de4038c619f5f979ba379b534dd78df8..6c1e10ef1aa6e9950bf4eb06955a802e185db1b2 100644 (file)
--- a/lib/public/IAddressBook.php
+++ b/lib/public/IAddressBook.php
@@ -73,7 +73,8 @@ namespace OCP {
                 *      - 'escape_like_param' - If set to false wildcards _ and % are not escaped
                 *      - 'limit' - Set a numeric limit for the search results
                 *      - 'offset' - Set the offset for the limited search results
-                *      - 'wildcard' - Whether the search should use wildcards
+                *      - 'wildcard' - (since 23.0.0) Whether the search should use wildcards
+                * @psalm-param array{types?: bool, escape_like_param?: bool, limit?: int, offset?: int, wildcard?: bool} $options
                 * @return array an array of contacts which are arrays of key-value-pairs
                 *  example result:
                 *  [