fix: Make sure that nodeType is called using numeric values

Signed-off-by: Julius Härtl <[email protected]>

appinfo/routes.php

@@ -126,7 +126,7 @@
 		['name' => 'ApiColumns#createSelectionColumn', 'url' => '/api/2/columns/selection', 'verb' => 'POST'],
 		['name' => 'ApiColumns#createDatetimeColumn', 'url' => '/api/2/columns/datetime', 'verb' => 'POST'],
 
-		['name' => 'ApiFavorite#create', 'url' => '/api/2/favorites/{nodeType}/{nodeId}', 'verb' => 'POST'],
-		['name' => 'ApiFavorite#destroy', 'url' => '/api/2/favorites/{nodeType}/{nodeId}', 'verb' => 'DELETE'],
+		['name' => 'ApiFavorite#create', 'url' => '/api/2/favorites/{nodeType}/{nodeId}', 'verb' => 'POST', 'requirements' => ['nodeType' => '(\d)', 'nodeId' => '(\d)']],
+		['name' => 'ApiFavorite#destroy', 'url' => '/api/2/favorites/{nodeType}/{nodeId}', 'verb' => 'DELETE', 'requirements' => ['nodeType' => '(\d)', 'nodeId' => '(\d)']],
 	]
 ];

lib/Controller/ApiFavoriteController.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace OCA\Tables\Controller;
 
 use Exception;

src/shared/constants.js

@@ -0,0 +1,23 @@
+/*
+ * @copyright Copyright (c) 2024 Julius Härtl <[email protected]>
+ *
+ * @author Julius Härtl <[email protected]>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+export const NODE_TYPE_TABLE = 0
+export const NODE_TYPE_VIEW = 1

src/store/store.js

@@ -6,6 +6,7 @@ import { showError } from '@nextcloud/dialogs'
 import '@nextcloud/dialogs/dist/index.css'
 import data from './data.js'
 import displayError from '../shared/utils/displayError.js'
+import { NODE_TYPE_TABLE, NODE_TYPE_VIEW } from '../shared/constants.js'
 
 Vue.use(Vuex)
 
@@ -236,7 +237,7 @@ export default new Vuex.Store({
 		},
 		async favoriteView({ state, commit, dispatch }, { id }) {
 			try {
-				await axios.post(generateOcsUrl('/apps/tables/api/2/favorites/views/' + id))
+				await axios.post(generateOcsUrl(`/apps/tables/api/2/favorites/${NODE_TYPE_VIEW}/${id}`))
 			} catch (e) {
 				displayError(e, t('tables', 'Could not favorite view'))
 				return false
@@ -251,7 +252,7 @@ export default new Vuex.Store({
 		},
 		async removeFavoriteView({ state, commit, dispatch }, { id }) {
 			try {
-				await axios.delete(generateOcsUrl('/apps/tables/api/2/favorites/views/' + id))
+				await axios.delete(generateOcsUrl(`/apps/tables/api/2/favorites/${NODE_TYPE_VIEW}/${id}`))
 			} catch (e) {
 				displayError(e, t('tables', 'Could not remove view from favorites'))
 				return false
@@ -266,7 +267,7 @@ export default new Vuex.Store({
 		},
 		async favoriteTable({ state, commit, dispatch }, { id }) {
 			try {
-				await axios.post(generateOcsUrl('/apps/tables/api/2/favorites/tables/' + id))
+				await axios.post(generateOcsUrl(`/apps/tables/api/2/favorites/${NODE_TYPE_TABLE}/${id}`))
 			} catch (e) {
 				displayError(e, t('tables', 'Could not favorite table'))
 				return false
@@ -281,7 +282,7 @@ export default new Vuex.Store({
 		},
 		async removeFavoriteTable({ state, commit, dispatch }, { id }) {
 			try {
-				await axios.delete(generateOcsUrl('/apps/tables/api/2/favorites/tables/' + id))
+				await axios.delete(generateOcsUrl(`/apps/tables/api/2/favorites/${NODE_TYPE_TABLE}/${id}`))
 			} catch (e) {
 				displayError(e, t('tables', 'Could not remove table from favorites'))
 				return false