This PR fixes 2 issues in pretty much all search views in the admin side:
leading/trailing spaces are not ignored
foo bar will not match foo xxx bar
So to test:
Search for the beginning part of something in the list and add a space before it.
Search for the ending part of something in the list and add a space after it.
Search - for example - for something that has three parts/words. But only enter the first and last part/word in the search field (separated by a space).
Unpatched the above should not return the item you are looking for. With this fix it should.
Check the Files changes to see which views are affected (and should be tested).
<p>This PR fixes 2 issues in pretty much all search views in the admin side:</p>
<ul class="task-list">
<li>leading/trailing spaces are not ignored</li>
<li>
<code>foo bar</code> will not match <code>foo xxx bar</code>
</li>
</ul><p>Affected list views in this PR:</p>
<ul class="task-list">
<li>com_modules</li>
<li>com_plugins</li>
</ul>
⇒
<p>This PR fixes 2 issues in pretty much all search views in the admin side:</p>
<ul class="task-list">
<li>leading/trailing spaces are not ignored</li>
<li>
<code>foo bar</code> will not match <code>foo xxx bar</code>
</li>
</ul><p>Check the Files changes to see which views are affected (and should be tested).</p>
For testers: This refers to the search field in the administrator list views.
My testing result: The patch works fine and makes sense. I think the same should be done in frontend as well, but it can be done in a separate PR if there is interest.
jissues-bot
- 
brianteeman
- 
can you explain how i have to test this PR?