[#41353] - Update Language.php in com_finder - Allow period before character in search (example search for: .net)
- Fixed in Code Base
- 1 Sep 2023
- Medium
- Build: 4.3-dev
- # 41353
- Diff
- BrainforgeUK:patch-17
User tests: Successful: Unsuccessful:
Allow period before letter characters (for example when searching for .net).
Pull Request for Issue #41338 .
Summary of Changes
In tokenise() allow a period before letter characters.
Testing Instructions
Create an article with the text
I require a .net developer
Then search for .net
Actual result BEFORE applying this Pull Request
No match found.
Expected result AFTER applying this Pull Request
Article added is found.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Administration com_finder |
| Status | New | ⇒ | Pending |
| Title |
|
||||||
| Title |
|
||||||
Expanded title
Should this be moved to 4.4 or 5?
I have tested this item ✅ successfully on 9ea78a0
With patch searches with a period before the word give the right result
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41353.
I have tested this item ✅ successfully on 9ea78a0
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41353.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41353.
| Labels |
Added:
?
PBF
bug
PR-4.3-dev
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-09-01 22:07:48 |
| Closed_By | ⇒ | obuisard |
Thank you Jonathan @BrainforgeUK for the PR.
The release team decided to include it in the release.
Please update the title to something that has some meaning