moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-05 17:06:53 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge branch 'master-MDL-72187' of https://github.com/hdagheda/moodle

Browse source
This commit is contained in:
Andrew Nicols 2021-08-26 12:28:54 +08:00
commit 1c167ac64c
2 changed files with 21 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
admin/tool/log/stores.php
View file

@ -50,6 +50,7 @@ switch ($action) {
case 'disable': case 'disable':
unset($enabled[$enrol]); unset($enabled[$enrol]);
set_config('enabled_stores', implode(',', array_keys($enabled)), 'tool_log'); set_config('enabled_stores', implode(',', array_keys($enabled)), 'tool_log');
add_to_config_log('tool_logstore_visibility', '1', '0', $enrol);
break; break;
case 'enable': case 'enable':
@ -59,6 +60,7 @@ switch ($action) {
$enabled = array_keys($enabled); $enabled = array_keys($enabled);
$enabled[] = $enrol; $enabled[] = $enrol;
set_config('enabled_stores', implode(',', $enabled), 'tool_log'); set_config('enabled_stores', implode(',', $enabled), 'tool_log');
add_to_config_log('tool_logstore_visibility', '0', '1', $enrol);
break; break;
case 'up': case 'up':

19
admin/tool/log/tests/behat/logstore_store_visibility_change.feature Normal file
View file

@ -0,0 +1,19 @@
@tool_log @report @report_configlog
Feature: In a report, admin can see logstore visibility changes
# Change log stores visibility so the report contains known data.
Background:
Given I log in as "admin"
And the following config values are set as admin:
| enabled_stores | logstore_standard | tool_log |
@javascript
Scenario: Display configuration changes report
When I navigate to "Plugins > Logging > Manage log stores" in site administration
And I click on "Disable" "icon" in the "Standard log" "table_row"
And I click on "Enable" "icon" in the "External database log" "table_row"
And I navigate to "Reports > Config changes" in site administration
Then the following should exist in the "reportbuilder-table" table:
| User | Plugin | Setting | New value | Original value |
| Admin User | logstore_standard | tool_logstore_visibility | 0 | 1 |
| Admin User | logstore_database | tool_logstore_visibility | 1 | 0 |