Merge pull request #1284 from nextcloud/perf/dashboard/item-api-v2

perf(dashboard): implement widget item api v2

lib/Dashboard/ActivityWidget.php

@@ -5,6 +5,7 @@
  * @copyright Copyright (c) 2021 Jakob Röhrl <[email protected]>
  *
  * @author Jakob Röhrl <[email protected]>
+ * @author Richard Steinmetz <[email protected]>
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -32,14 +33,15 @@
 use OCP\Dashboard\IAPIWidget;
 use OCP\Dashboard\IButtonWidget;
 use OCP\Dashboard\IIconWidget;
+use OCP\Dashboard\IReloadableWidget;
 use OCP\Dashboard\Model\WidgetButton;
 use OCP\Dashboard\Model\WidgetItem;
+use OCP\Dashboard\Model\WidgetItems;
 use OCP\IDateTimeFormatter;
 use OCP\IL10N;
 use OCP\IURLGenerator;
-use OCP\Util;
 
-class ActivityWidget implements IAPIWidget, IButtonWidget, IIconWidget {
+class ActivityWidget implements IAPIWidget, IButtonWidget, IIconWidget, IReloadableWidget {
 	private Data $data;
 	private IL10N $l10n;
 	private GroupHelper $helper;
@@ -111,7 +113,6 @@ public function getUrl(): ?string {
 	 * @inheritDoc
 	 */
 	public function load(): void {
-		Util::addScript('activity', 'activity-dashboard');
 	}
 
 	/**
@@ -142,6 +143,49 @@ public function getItems(string $userId, ?string $since = null, int $limit = 7):
 		}, array_slice($activities['data'], 0, $limit));
 	}
 
+	/**
+	 * @inheritDoc
+	 */
+	public function getItemsV2(string $userId, ?string $since = null, int $limit = 7): WidgetItems {
+		// we set the limit to 50 here because data->get might return less activity entries
+		// in the end we take the first 7 of'em
+		$activities = $this->data->get(
+			$this->helper,
+			$this->settings,
+			$userId,
+			$since ? (int) $since : 0,
+			50,
+			'desc',
+			'by',
+			'',
+			0
+		);
+		$items = array_map(function (array $activity) {
+			$userAvatarUrl = '';
+			if ($activity['user'] !== '') {
+				$userAvatarUrl = $this->urlGenerator->getAbsoluteURL(
+					$this->urlGenerator->linkToRoute('core.avatar.getAvatar', [
+						'userId' => $activity['user'],
+						'size' => 512,
+					])
+				);
+			}
+
+			return new WidgetItem(
+				$activity['subject'],
+				$this->dateTimeFormatter->formatTimeSpan($activity['timestamp']),
+				$activity['link'],
+				$userAvatarUrl,
+				(string) $activity['activity_id'],
+				$activity['icon'],
+			);
+		}, array_slice($activities['data'], 0, $limit));
+		return new WidgetItems(
+			$items,
+			empty($items) ? $this->l10n->t('No activities') : '',
+		);
+	}
+
 	/**
 	 * @inheritDoc
 	 */
@@ -156,4 +200,11 @@ public function getWidgetButtons(string $userId): array {
 			),
 		];
 	}
+
+	/**
+	 * @inheritDoc
+	 */
+	public function getReloadInterval(): int {
+		return 30;
+	}
 }

package.json

@@ -77,7 +77,6 @@
   },
   "dependencies": {
     "@mdi/svg": "^7.2.96",
-    "@nextcloud/auth": "^2.1.0",
     "@nextcloud/axios": "^2.4.0",
     "@nextcloud/dialogs": "^4.1.0",
     "@nextcloud/initial-state": "^2.1.0",

webpack.js

@@ -4,7 +4,6 @@ const webpackConfig = require('@nextcloud/webpack-vue-config')
 
 webpackConfig.entry = {
 	sidebar: path.join(__dirname, 'src', 'sidebar.js'),
-	dashboard: path.join(__dirname, 'src', 'dashboard.js'),
 	personalSettings: path.join(__dirname, 'src', 'settings-personal.js'),
 	adminSettings: path.join(__dirname, 'src', 'settings-admin.js'),
 }