Wrong information on a comment of PostRepository: method sanitizeSearchQuery()

[o-alquimista]

The DocBlock of sanitizeSearchQuery() says it is removing non-alphanumeric characters, while it seems to be only replacing duplicated spaces with a single space.

    /**
     * Removes all non-alphanumeric characters except whitespaces.
     */
    private function sanitizeSearchQuery(string $query): string
    {
        return trim(preg_replace('/[[:space:]]+/', ' ', $query));
    }

Someone forgot to implement the functionality? Or maybe forgot to remove the comment after the code it is about was removed.

[voronkovich]

@o-alquimista, You're right. See this commit: 0e1c9d3. I suggest to delete the sanitizeSearchQuery method and move it's code to the extractSearchTerms method. Could you please create a PR?

[o-alquimista]

@voronkovich It is done: #1025, but the CI check failed. Why?

[javiereguiluz]

Fixed by #1025. Thanks!