refactor(dashboard): Use attributes for routing

Signed-off-by: provokateurin <kate@provokateurin.de>

2 files changed, 10 insertions, 0 deletions

diff --git a/apps/dashboard/lib/Controller/DashboardApiController.php b/apps/dashboard/lib/Controller/DashboardApiController.php
index bd7cb402600..b6ebb4625b5 100644
--- a/apps/dashboard/lib/Controller/DashboardApiController.php
+++ b/apps/dashboard/lib/Controller/DashboardApiController.php
@@ -31,6 +31,7 @@ namespace OCA\Dashboard\Controller;
 use OCA\Dashboard\ResponseDefinitions;
 use OCA\Dashboard\Service\DashboardService;
 use OCP\AppFramework\Http;
+use OCP\AppFramework\Http\Attribute\ApiRoute;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\AppFramework\OCSController;
 use OCP\Dashboard\IAPIWidget;
@@ -98,6 +99,7 @@ class DashboardApiController extends OCSController {
 	 *
 	 * 200: Widget items returned
 	 */
+	#[ApiRoute(verb: 'GET', url: '/api/v1/widget-items')]
 	public function getWidgetItems(array $sinceIds = [], int $limit = 7, array $widgets = []): DataResponse {
 		$items = [];
 		$widgets = $this->getShownWidgets($widgets);
@@ -126,6 +128,7 @@ class DashboardApiController extends OCSController {
 	 *
 	 * 200: Widget items returned
 	 */
+	#[ApiRoute(verb: 'GET', url: '/api/v2/widget-items')]
 	public function getWidgetItemsV2(array $sinceIds = [], int $limit = 7, array $widgets = []): DataResponse {
 		$items = [];
 		$widgets = $this->getShownWidgets($widgets);
@@ -150,6 +153,7 @@ class DashboardApiController extends OCSController {
 	 *
 	 * 200: Widgets returned
 	 */
+	#[ApiRoute(verb: 'GET', url: '/api/v1/widgets')]
 	public function getWidgets(): DataResponse {
 		$widgets = $this->dashboardManager->getWidgets();
 
@@ -200,6 +204,7 @@ class DashboardApiController extends OCSController {
 	 *
 	 * 200: Layout returned
 	 */
+	#[ApiRoute(verb: 'GET', url: '/api/v3/layout')]
 	public function getLayout(): DataResponse {
 		return new DataResponse(['layout' => $this->service->getLayout()]);
 	}
@@ -213,6 +218,7 @@ class DashboardApiController extends OCSController {
 	 *
 	 * 200: Statuses updated successfully
 	 */
+	#[ApiRoute(verb: 'POST', url: '/api/v3/layout')]
 	public function updateLayout(array $layout): DataResponse {
 		$this->config->setUserValue($this->userId, 'dashboard', 'layout', implode(',', $layout));
 		return new DataResponse(['layout' => $layout]);
@@ -226,6 +232,7 @@ class DashboardApiController extends OCSController {
 	 *
 	 * 200: Statuses returned
 	 */
+	#[ApiRoute(verb: 'GET', url: '/api/v3/statuses')]
 	public function getStatuses(): DataResponse {
 		return new DataResponse(['statuses' => $this->service->getStatuses()]);
 	}
@@ -239,6 +246,7 @@ class DashboardApiController extends OCSController {
 	 *
 	 * 200: Statuses updated successfully
 	 */
+	#[ApiRoute(verb: 'POST', url: '/api/v3/statuses')]
 	public function updateStatuses(array $statuses): DataResponse {
 		$this->config->setUserValue($this->userId, 'dashboard', 'statuses', implode(',', $statuses));
 		return new DataResponse(['statuses' => $statuses]);
diff --git a/apps/dashboard/lib/Controller/DashboardController.php b/apps/dashboard/lib/Controller/DashboardController.php
index 8375858cbee..f07fd4938d3 100644
--- a/apps/dashboard/lib/Controller/DashboardController.php
+++ b/apps/dashboard/lib/Controller/DashboardController.php
@@ -33,6 +33,7 @@ namespace OCA\Dashboard\Controller;
 use OCA\Dashboard\Service\DashboardService;
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http;
+use OCP\AppFramework\Http\Attribute\FrontpageRoute;
 use OCP\AppFramework\Http\Attribute\OpenAPI;
 use OCP\AppFramework\Http\TemplateResponse;
 use OCP\AppFramework\Services\IInitialState;
@@ -65,6 +66,7 @@ class DashboardController extends Controller {
 	 * @NoAdminRequired
 	 * @return TemplateResponse
 	 */
+	#[FrontpageRoute(verb: 'GET', url: '/')]
 	public function index(): TemplateResponse {
 		\OCP\Util::addStyle('dashboard', 'dashboard');
 		\OCP\Util::addScript('dashboard', 'main', 'theming');