Feeling Lucky
?
PR-4.3-dev
[#41468] - [4.3] fix finder common words
- Fixed in Code Base
- 19 Sep 2023
- Medium
- Build: 4.3-dev
- # 41468
- Diff
- heelc29:4.3/patch-40215
User tests: Successful: Unsuccessful:
Pull Request for Issue #40215 .
Summary of Changes
remove duplicate entries which will cause database error
Testing Instructions
install dutch language pack
Actual result BEFORE applying this Pull Request
no entries in #__finder_terms_common for dutch language
Expected result AFTER applying this Pull Request
88 entries in #__finder_terms_common for dutch language (v4.3.4.1) in database
Link to documentations
Please select:
- No documentation changes for docs.joomla.org needed
- No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Status | New | ⇒ | Pending |
brianteeman
- comment
- 27 Aug 2023
While this does resolve the issue surely the correct way is to remove the duplicates at source in the NL language pack
heelc29
- comment
- 28 Aug 2023
Yes, that's what I've done here joomlagerman/joomla#2661 for the german language pack... so I'm happy with how it is now.
Quy
- comment
- 16 Sep 2023
I have tested this item ✅ successfully on 357fa73
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41468.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
PR-4.3-dev
|
||
Quy
- comment
- 16 Sep 2023
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41468.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-09-19 01:00:15 |
| Closed_By | ⇒ | obuisard |
I have tested this item ✅ successfully on 357fa73
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41468.