From c204185ae86a98994f80010abf35a190c9406739 Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Sun, 12 Jul 2026 18:08:19 +0000
Subject: [PATCH] =Refactor of Integrations.php. Separated different functionality into traits that classes can use to add that functionality. Hopefully will make maintaining it a little easier. Still have to finish up, as well as refactoring the individual classes to utilize the new system.
---
inc/rest/routes/FavouritesRoutes.php | 32 +++++++++++---------------------
1 files changed, 11 insertions(+), 21 deletions(-)
diff --git a/inc/rest/routes/FavouritesRoutes.php b/inc/rest/routes/FavouritesRoutes.php
index 953e4db..a9d767d 100644
--- a/inc/rest/routes/FavouritesRoutes.php
+++ b/inc/rest/routes/FavouritesRoutes.php
@@ -54,7 +54,6 @@
Route::for('favourites')
->get([$this, 'getFavourites'])
->args([
- 'user' => 'integer|required',
'type' => 'string',
'include_all' => 'boolean',
])
@@ -62,7 +61,6 @@
->rateLimit(30)
->post([$this, 'handleFavourite'])
->args([
- 'user' => 'integer|required',
'action' => 'string|required|enum:add,remove,toggle,batch,note',
'type' => 'string',
'target_id' => 'integer',
@@ -81,7 +79,6 @@
->rateLimit(30)
->post([$this, 'handleList'])
->args([
- 'user' => 'integer|required',
'id' => 'string|required',
'action' => 'string|required|enum:create,update,delete,share,unshare,add_items,remove_items',
'list_id' => 'integer',
@@ -95,7 +92,6 @@
// Favourite counts
Route::for('favourites/counts')
->get([$this, 'getFavouriteCounts'])
- ->args(['user' => 'integer|required'])
->auth(PermissionHandler::combine(['user', ['actionNonce' => 'favourites-']]))
->register();
}
@@ -105,9 +101,9 @@
*/
public function getFavourites(WP_REST_Request $request): WP_REST_Response
{
- $user_id = absint($request->get_param('user'));
+ $user_id = get_current_user_id();
- if (!$this->userCheck($user_id)) {
+ if (!$user_id) {
return $this->unauthorized();
}
@@ -132,11 +128,8 @@
public function handleFavourite(WP_REST_Request $request): WP_REST_Response
{
$params = $request->get_params();
- $user_id = absint($params['user']??0);
+ $user_id = get_current_user_id();
- if (!$this->userCheck($user_id)) {
- return $this->unauthorized();
- }
$action = strtolower(sanitize_text_field($params['action']));
$action = in_array($action, ['add', 'remove']) ? $action : false;
if (!$action) {
@@ -171,11 +164,8 @@
public function getLists(WP_REST_Request $request): WP_REST_Response
{
$params = $request->get_params();
- $user_id = absint($params['user']);
+ $user_id = get_current_user_id();
- if (!$this->userCheck($user_id)) {
- return $this->unauthorized();
- }
$args = $this->buildParams($request);
$args['per_page'] = 20;
@@ -209,11 +199,8 @@
public function getFavouriteCounts(WP_REST_Request $request): WP_REST_Response
{
- $user_id = absint($request->get_param('user'));
+ $user_id = get_current_user_id();
- if (!$this->userCheck($user_id)) {
- return $this->unauthorized();
- }
$counts = JVB()->favourites()->getFavouriteCounts($user_id);
@@ -528,8 +515,11 @@
protected function buildParams(WP_REST_Request $request): array
{
$data = $request->get_params();
-
- $where = ['user_id' => absint($data['user'])];
+ $userID = get_current_user_id();
+ if(!$userID) {
+ return [];
+ }
+ $where = ['user_id' => $userID];
if (!empty($data['content']) && $data['content'] !== 'all') {
$where['type'] = BASE . $data['content'];
}
@@ -1652,7 +1642,7 @@
*/
public function handleList(WP_REST_Request $request): WP_REST_Response
{
- $user_id = absint($request->get_param('user'));
+ $user_id = get_current_user_id();
$operation_id = sanitize_text_field($request->get_param('id'));
$action = sanitize_text_field($request->get_param('action'));
--
Gitblit v1.10.0