Skip to content

Commit

Permalink
Switch to new logger implementation (#79)
Browse files Browse the repository at this point in the history
  • Loading branch information
stjosh authored Sep 21, 2024
1 parent f284385 commit d8248cf
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 6 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Changed
- Use PSR Logger Interface (#79)
- Compatibility up to NC31

## 1.6.0 - 2024-09-21
### Changed
- Support SAML Logins (#77) - shout out to @elainabialkowski
Expand Down
4 changes: 2 additions & 2 deletions appinfo/info.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ Note that this app prevents group deletions for groups referenced as Auto Groups

In addition, I plan to add some more features over time, e.g., "Union Groups" - see the [Milestone Plans](https://github.com/stjosh/auto_groups/milestones) for more details.
</description>
<version>1.6.0</version>
<version>1.6.1</version>
<licence>agpl</licence>
<author mail="der@digitalwerker.ch" >Josua Hunziker</author>
<namespace>AutoGroups</namespace>
Expand All @@ -34,7 +34,7 @@ In addition, I plan to add some more features over time, e.g., "Union Groups" -
<repository type="git">https://github.com/stjosh/auto_groups.git</repository>
<screenshot>https://raw.githubusercontent.com/stjosh/auto_groups/master/screenshots/settings.png</screenshot>
<dependencies>
<nextcloud min-version="26" max-version="30"/>
<nextcloud min-version="26" max-version="31"/>
</dependencies>
<settings>
<admin>OCA\AutoGroups\Settings\Admin</admin>
Expand Down
5 changes: 3 additions & 2 deletions lib/AutoGroupsManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,12 @@
use OCP\IGroupManager;
use OCP\EventDispatcher\IEventDispatcher;
use OCP\IConfig;
use OCP\ILogger;
use OCP\IL10N;

use OCP\AppFramework\OCS\OCSBadRequestException;

use Psr\Log\LoggerInterface;

class AutoGroupsManager
{
private $groupManager;
Expand All @@ -51,7 +52,7 @@ class AutoGroupsManager
/**
* Listener manager constructor.
*/
public function __construct(IGroupManager $groupManager, IEventDispatcher $eventDispatcher, IConfig $config, ILogger $logger, IL10N $l)
public function __construct(IGroupManager $groupManager, IEventDispatcher $eventDispatcher, IConfig $config, LoggerInterface $logger, IL10N $l)
{
$this->groupManager = $groupManager;
$this->logger = $logger;
Expand Down
5 changes: 3 additions & 2 deletions tests/Unit/AutoGroupsManagerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@
use OCP\IGroupManager;
use OCP\EventDispatcher\IEventDispatcher;
use OCP\IConfig;
use OCP\ILogger;
use OCP\IL10N;

use OCP\AppFramework\OCS\OCSBadRequestException;
Expand All @@ -45,6 +44,8 @@

use OCA\AutoGroups\AutoGroupsManager;

use Psr\Log\LoggerInterface;

use Test\TestCase;


Expand All @@ -63,7 +64,7 @@ protected function setUp(): void
$this->groupManager = $this->createMock(IGroupManager::class);
$this->eventDispatcher = $this->createMock(IEventDispatcher::class);
$this->config = $this->createMock(IConfig::class);
$this->logger = $this->createMock(ILogger::class);
$this->logger = $this->createMock(LoggerInterface::class);
$this->il10n = $this->createMock(IL10N::class);

$this->testUser = $this->createMock(IUser::class);
Expand Down

0 comments on commit d8248cf

Please sign in to comment.