[#29164] - [4.0] [com_finder] Disable search statistics by default
- Fixed in Code Base
- 24 May 2020
- Medium
- Build: 4.0-dev
- # 29164
- Diff
- SharkyKZ:j4/bug/finder-statistics
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Summary of Changes
In the administrator part of Smart Search, search statistics option appears to be disabled but in frontend statistics are still gathered. This changes default value of statistics option in frontend to disabled.
Also renames logging_enabled parameter to gather_search_statistics to avoid confusion with enable_logging.
Testing Instructions
In backend go to Components -> Smart Search -> Statistics.
Notice the warning:
Gathering of statistics is disabled. Enable it in the Options.
Go to frontend. Perform a search using Smart Search.
Go back to statistics page.
Expected result
No entries logged.
Actual result
Search entry logged.
Documentation Changes Required
No.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_finder Front End |
| Labels |
Added:
?
|
||
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29164.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29164.
@SharkyKZ Could you update the branch of this PR to latest 4.0-dev of the CMS? It needs to do that in order to make drone pass, and this is needed for merging (except someone merges who has permission to force merge, which I haven't). Wait, I first try it with restarting drone.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-05-24 09:31:38 |
| Closed_By | ⇒ | zero-24 |
I have tested this item✅ successfully on 3b27551
Tested successfully in Nightly Build of May 20.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29164.