[#15086] - [com_fields] Fix search query due to removed context and renamed field property.
- Fixed in Code Base
- 4 Apr 2017
- Medium
- Build: staging
- # 15086
- Diff
- Bakual:FixFieldSearch
User tests: Successful: Unsuccessful:
Pull Request for Issue #15080 .
Summary of Changes
Since w removed the context from the #__fields_values table, we need to check the context from the #__fields table now.
Also the property got renamed from fields to jcfields.
Testing Instructions
Search for a field value in an article.
Expected result
Article shows up
Actual result
Error message
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Title |
|
||||||
Without PR: Search for an Field-Value (en-GB) on a multi.-lang. Site using Menu Smart Search (en-GB) Article is found.
Looks good on mono-lingual website.
However on multi-lingual index.php/nl/component/search/?searchword=dutch&searchphrase=all returns 6 rows while the index.php/fr/component/search/?searchword=dutch&searchphrase=all returns 0 rows.
I need to check what was the 3.6.5 behaviour to be able to confirm the fix.
i tested on latest staging 3.7.
Site using Menu Smart Search (en-GB) Article is found.
This would be about the standard search, not the Smart Search
However on multi-lingual index.php/nl/component/search/?searchword=dutch&searchphrase=all returns 6 rows with my data while the index.php/fr/component/search/?searchword=dutch&searchphrase=all returns 0 rows.
I need to check what was the 3.6.5 behaviour to be able to confirm the fix.
Search is performed on active language only. So if your articles are assigned to dutch, then this would be expected behavior.
I have tested this item
Works fine on mysql, same behaviour as 3.6.5 mono and multi-lingual
Thank you all.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15086.
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful testes.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-04-04 19:09:08 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
?
|
||
I have tested this item✅ successfully on c4d0c5c
Patch ok for me
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15086.