diff options
author | Ben Burgess <88810029+bx80@users.noreply.github.com> | 2021-12-22 23:26:51 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-12-22 23:26:51 +0300 |
commit | d68dc6dd06606e8a7611c53d8ee63da8afbf99f7 (patch) | |
tree | b0363d254677ec77d42ff75352992405b41aef81 /plugins/UsersManager | |
parent | 10afd46db0e033e67127fb4e50eea986ed5296e6 (diff) |
Show a summary of new features (#18065)
* Added "What is new" notification display, populated by a new event
* Removed test example event hook
* Added support for applying a link attribute to menu items, fixes layout issue for mobile with html menu items
* Updated UI test screenshots
* Revert accidental edit
* Hide the "What's new" icon if there are no new features to show
* Changed to use changes.json, track user last viewed, added ui test
* Fix UserManager unit tests broken by new ts_changes_viewed user field
* Moved getChanges to separate helper class, added unit test, added user view access check
* Updated to add new changes table and populate only on plugin update/install
* Added missing fixture class, updated UI screenshots
* Updated matomo font to add ringing bell and new releases icons
* Fix for integration test
* Reworked class structure, removed unnecessary angular directive, merged templates, other tidy ups
* built vue files
* built vue files
* Added null user check, missing table exception handling, show plugin name in change title, better handling of missing change fields
* Added sample changes file, moved UserChanges db code to changes model, added return type hints, better db error code handling, various other improvements
* Revert accidental UI screenshot commit
* Fix for incorrect link name parameter in sample changes, switched back to using $db->query for INSERT IGNORE
* Integration test fix, UI screenshot updates
* Test fix
* Added link styling, show CoreHome changes without plugin prefix in title
* Update UI test screenshot
* Added styles to the popover, added event for filtering changes
* Test fix
* UI test screenshot updates
Co-authored-by: sgiehl <stefan@matomo.org>
Co-authored-by: bx80 <bx80@users.noreply.github.com>
Diffstat (limited to 'plugins/UsersManager')
-rw-r--r-- | plugins/UsersManager/API.php | 1 | ||||
-rw-r--r-- | plugins/UsersManager/Model.php | 1 | ||||
-rw-r--r-- | plugins/UsersManager/tests/Integration/UsersManagerTest.php | 1 | ||||
-rw-r--r-- | plugins/UsersManager/tests/System/ApiTest.php | 2 |
4 files changed, 4 insertions, 1 deletions
diff --git a/plugins/UsersManager/API.php b/plugins/UsersManager/API.php index 8aa5a37ca5..7bc1ae4068 100644 --- a/plugins/UsersManager/API.php +++ b/plugins/UsersManager/API.php @@ -877,6 +877,7 @@ class API extends \Piwik\Plugin\API unset($user['token_auth']); unset($user['password']); unset($user['ts_password_modified']); + unset($user['idchange_last_viewed']); if ($lastSeen = LastSeenTimeLogger::getLastSeenTimeForUser($user['login'])) { $user['last_seen'] = Date::getDatetimeFromTimestamp($lastSeen); diff --git a/plugins/UsersManager/Model.php b/plugins/UsersManager/Model.php index 618d169866..6d38068274 100644 --- a/plugins/UsersManager/Model.php +++ b/plugins/UsersManager/Model.php @@ -456,6 +456,7 @@ class Model 'date_registered' => $dateRegistered, 'superuser_access' => 0, 'ts_password_modified' => Date::now()->getDatetime(), + 'idchange_last_viewed' => null ); $db = $this->getDb(); diff --git a/plugins/UsersManager/tests/Integration/UsersManagerTest.php b/plugins/UsersManager/tests/Integration/UsersManagerTest.php index 686668531e..0599554b34 100644 --- a/plugins/UsersManager/tests/Integration/UsersManagerTest.php +++ b/plugins/UsersManager/tests/Integration/UsersManagerTest.php @@ -104,6 +104,7 @@ class UsersManagerTest extends IntegrationTestCase unset($userAfter['date_registered']); unset($userAfter['ts_password_modified']); + unset($userAfter['idchange_last_viewed']); unset($userAfter['password']); // implicitly checks password! diff --git a/plugins/UsersManager/tests/System/ApiTest.php b/plugins/UsersManager/tests/System/ApiTest.php index 2c93dc2eb5..17ea5104e6 100644 --- a/plugins/UsersManager/tests/System/ApiTest.php +++ b/plugins/UsersManager/tests/System/ApiTest.php @@ -61,7 +61,7 @@ class ApiTest extends SystemTestCase // login1 = super user, login2 = some admin access, login4 = only view access foreach ($logins as $login => $appendix) { $params['token_auth'] = self::$fixture->users[$login]['token']; - $xmlFieldsToRemove = array('date_registered','last_seen', 'password', 'token_auth', 'ts_password_modified'); + $xmlFieldsToRemove = array('date_registered', 'last_seen', 'password', 'token_auth', 'ts_password_modified', 'idchange_last_viewed'); $this->runAnyApiTest($api, $apiId . '_' . $appendix, $params, array('xmlFieldsToRemove' => $xmlFieldsToRemove)); } |