Merge pull request #33658 from nextcloud/dashboard-api-widgets

Extend dashboard api to allow listing of widgets

2 files changed, 92 insertions, 35 deletions

diff --git a/apps/user_status/lib/Dashboard/UserStatusWidget.php b/apps/user_status/lib/Dashboard/UserStatusWidget.php
index 10411dc7f9d..5a89040dfa5 100644
--- a/apps/user_status/lib/Dashboard/UserStatusWidget.php
+++ b/apps/user_status/lib/Dashboard/UserStatusWidget.php
@@ -28,50 +28,56 @@ namespace OCA\UserStatus\Dashboard;
 use OCA\UserStatus\AppInfo\Application;
 use OCA\UserStatus\Db\UserStatus;
 use OCA\UserStatus\Service\StatusService;
-use OCP\Dashboard\IWidget;
-use OCP\IInitialStateService;
+use OCP\AppFramework\Services\IInitialState;
+use OCP\Dashboard\IAPIWidget;
+use OCP\Dashboard\IButtonWidget;
+use OCP\Dashboard\IIconWidget;
+use OCP\Dashboard\IOptionWidget;
+use OCP\Dashboard\Model\WidgetItem;
+use OCP\Dashboard\Model\WidgetOptions;
+use OCP\IDateTimeFormatter;
 use OCP\IL10N;
+use OCP\IURLGenerator;
 use OCP\IUserManager;
 use OCP\IUserSession;
 use OCP\UserStatus\IUserStatus;
+use OCP\Util;
 
 /**
  * Class UserStatusWidget
  *
  * @package OCA\UserStatus
  */
-class UserStatusWidget implements IWidget {
-
-	/** @var IL10N */
-	private $l10n;
-
-	/** @var IInitialStateService */
-	private $initialStateService;
-
-	/** @var IUserManager */
-	private $userManager;
-
-	/** @var IUserSession */
-	private $userSession;
-
-	/** @var StatusService */
-	private $service;
+class UserStatusWidget implements IAPIWidget, IIconWidget, IOptionWidget {
+	private IL10N $l10n;
+	private IDateTimeFormatter $dateTimeFormatter;
+	private IURLGenerator $urlGenerator;
+	private IInitialState $initialStateService;
+	private IUserManager $userManager;
+	private IUserSession $userSession;
+	private StatusService $service;
 
 	/**
 	 * UserStatusWidget constructor
 	 *
 	 * @param IL10N $l10n
-	 * @param IInitialStateService $initialStateService
+	 * @param IDateTimeFormatter $dateTimeFormatter
+	 * @param IURLGenerator $urlGenerator
+	 * @param IInitialState $initialStateService
 	 * @param IUserManager $userManager
 	 * @param IUserSession $userSession
 	 * @param StatusService $service
 	 */
 	public function __construct(IL10N $l10n,
-								IInitialStateService $initialStateService,
+								IDateTimeFormatter $dateTimeFormatter,
+								IURLGenerator $urlGenerator,
+								IInitialState $initialStateService,
 								IUserManager $userManager,
 								IUserSession $userSession,
 								StatusService $service) {
 		$this->l10n = $l10n;
+		$this->dateTimeFormatter = $dateTimeFormatter;
+		$this->urlGenerator = $urlGenerator;
 		$this->initialStateService = $initialStateService;
 		$this->userManager = $userManager;
 		$this->userSession = $userSession;
@@ -109,6 +115,15 @@ class UserStatusWidget implements IWidget {
 	/**
 	 * @inheritDoc
 	 */
+	public function getIconUrl(): string {
+		return $this->urlGenerator->getAbsoluteURL(
+			$this->urlGenerator->imagePath(Application::APP_ID, 'app.svg')
+		);
+	}
+
+	/**
+	 * @inheritDoc
+	 */
 	public function getUrl(): ?string {
 		return null;
 	}
@@ -117,28 +132,33 @@ class UserStatusWidget implements IWidget {
 	 * @inheritDoc
 	 */
 	public function load(): void {
-		\OCP\Util::addScript(Application::APP_ID, 'dashboard');
+		Util::addScript(Application::APP_ID, 'dashboard');
 
 		$currentUser = $this->userSession->getUser();
 		if ($currentUser === null) {
-			$this->initialStateService->provideInitialState(Application::APP_ID, 'dashboard_data', []);
+			$this->initialStateService->provideInitialState('dashboard_data', []);
 			return;
 		}
 		$currentUserId = $currentUser->getUID();
 
+		$widgetItemsData = $this->getWidgetData($currentUserId);
+		$this->initialStateService->provideInitialState('dashboard_data', $widgetItemsData);
+	}
+
+	private function getWidgetData(string $userId, ?string $since = null, int $limit = 7): array {
 		// Fetch status updates and filter current user
 		$recentStatusUpdates = array_slice(
 			array_filter(
-				$this->service->findAllRecentStatusChanges(8, 0),
-				static function (UserStatus $status) use ($currentUserId): bool {
-					return $status->getUserId() !== $currentUserId;
+				$this->service->findAllRecentStatusChanges($limit + 1, 0),
+				static function (UserStatus $status) use ($userId, $since): bool {
+					return $status->getUserId() !== $userId
+						&& ($since === null || $status->getStatusTimestamp() > (int) $since);
 				}
 			),
 			0,
-			7
+			$limit
 		);
-
-		$this->initialStateService->provideInitialState(Application::APP_ID, 'dashboard_data', array_map(function (UserStatus $status): array {
+		return array_map(function (UserStatus $status): array {
 			$user = $this->userManager->get($status->getUserId());
 			$displayName = $status->getUserId();
 			if ($user !== null) {
@@ -155,6 +175,33 @@ class UserStatusWidget implements IWidget {
 				'message' => $status->getCustomMessage(),
 				'timestamp' => $status->getStatusTimestamp(),
 			];
-		}, $recentStatusUpdates));
+		}, $recentStatusUpdates);
+	}
+
+	/**
+	 * @inheritDoc
+	 */
+	public function getItems(string $userId, ?string $since = null, int $limit = 7): array {
+		$widgetItemsData = $this->getWidgetData($userId, $since, $limit);
+
+		return array_map(function(array $widgetData) {
+			$formattedDate = $this->dateTimeFormatter->formatTimeSpan($widgetData['timestamp']);
+			return new WidgetItem(
+				$widgetData['displayName'],
+				$widgetData['icon'] . ($widgetData['icon'] ? ' ' : '') . $widgetData['message'] . ', ' . $formattedDate,
+				// https://nextcloud.local/index.php/u/julien
+				$this->urlGenerator->getAbsoluteURL(
+					$this->urlGenerator->linkToRoute('core.ProfilePage.index', ['targetUserId' => $widgetData['userId']])
+				),
+				$this->urlGenerator->getAbsoluteURL(
+					$this->urlGenerator->linkToRoute('core.avatar.getAvatar', ['userId' => $widgetData['userId'], 'size' => 44])
+				),
+				(string) $widgetData['timestamp']
+			);
+		}, $widgetItemsData);
+	}
+
+	public function getWidgetOptions(): WidgetOptions {
+		return new WidgetOptions(true);
 	}
 }
diff --git a/apps/user_status/tests/Unit/Dashboard/UserStatusWidgetTest.php b/apps/user_status/tests/Unit/Dashboard/UserStatusWidgetTest.php
index d23b2bc02ff..ab7252370ce 100644
--- a/apps/user_status/tests/Unit/Dashboard/UserStatusWidgetTest.php
+++ b/apps/user_status/tests/Unit/Dashboard/UserStatusWidgetTest.php
@@ -29,8 +29,10 @@ namespace OCA\UserStatus\Tests\Dashboard;
 use OCA\UserStatus\Dashboard\UserStatusWidget;
 use OCA\UserStatus\Db\UserStatus;
 use OCA\UserStatus\Service\StatusService;
-use OCP\IInitialStateService;
+use OCP\AppFramework\Services\IInitialState;
+use OCP\IDateTimeFormatter;
 use OCP\IL10N;
+use OCP\IURLGenerator;
 use OCP\IUser;
 use OCP\IUserManager;
 use OCP\IUserSession;
@@ -41,7 +43,13 @@ class UserStatusWidgetTest extends TestCase {
 	/** @var IL10N|\PHPUnit\Framework\MockObject\MockObject */
 	private $l10n;
 
-	/** @var IInitialStateService|\PHPUnit\Framework\MockObject\MockObject */
+	/** @var IDateTimeFormatter|\PHPUnit\Framework\MockObject\MockObject */
+	private $dateTimeFormatter;
+
+	/** @var IURLGenerator|\PHPUnit\Framework\MockObject\MockObject */
+	private $urlGenerator;
+
+	/** @var IInitialState|\PHPUnit\Framework\MockObject\MockObject */
 	private $initialState;
 
 	/** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
@@ -60,12 +68,14 @@ class UserStatusWidgetTest extends TestCase {
 		parent::setUp();
 
 		$this->l10n = $this->createMock(IL10N::class);
-		$this->initialState = $this->createMock(IInitialStateService::class);
+		$this->dateTimeFormatter = $this->createMock(IDateTimeFormatter::class);
+		$this->urlGenerator = $this->createMock(IURLGenerator::class);
+		$this->initialState = $this->createMock(IInitialState::class);
 		$this->userManager = $this->createMock(IUserManager::class);
 		$this->userSession = $this->createMock(IUserSession::class);
 		$this->service = $this->createMock(StatusService::class);
 
-		$this->widget = new UserStatusWidget($this->l10n, $this->initialState, $this->userManager, $this->userSession, $this->service);
+		$this->widget = new UserStatusWidget($this->l10n, $this->dateTimeFormatter, $this->urlGenerator, $this->initialState, $this->userManager, $this->userSession, $this->service);
 	}
 
 	public function testGetId(): void {
@@ -99,7 +109,7 @@ class UserStatusWidgetTest extends TestCase {
 
 		$this->initialState->expects($this->once())
 			->method('provideInitialState')
-			->with('user_status', 'dashboard_data', []);
+			->with('dashboard_data', []);
 
 		$this->service->expects($this->never())
 			->method('findAllRecentStatusChanges');
@@ -195,7 +205,7 @@ class UserStatusWidgetTest extends TestCase {
 
 		$this->initialState->expects($this->once())
 			->method('provideInitialState')
-			->with('user_status', 'dashboard_data', $this->callback(function ($data): bool {
+			->with('dashboard_data', $this->callback(function ($data): bool {
 				$this->assertEquals([
 					[
 						'userId' => 'user_1',