Add rating per user and other fixes. (#3899)

Author: ildyria

app/Actions/Album/Create.php

@@ -150,6 +150,8 @@ private function setStatistics(Album $album): void
 			'download_count' => 0,
 			'favourite_count' => 0,
 			'shared_count' => 0,
+			'rating_sum' => 0,
+			'rating_count' => 0,
 		]);
 	}
 }

app/Actions/Album/PositionData.php

@@ -36,6 +36,7 @@ public function get(AbstractAlbum $album, bool $include_sub_albums = false): Pos
 				},
 				'palette',
 				'tags',
+				'rating',
 			])
 			->whereNotNull('latitude')
 			->whereNotNull('longitude');

app/Actions/Albums/Flow.php

@@ -113,7 +113,7 @@ private function getQuery(Album|null $base, bool $with_relations): AlbumBuilder
 
 		$base_query = Album::query();
 		if ($with_relations) {
-			$base_query->with(['cover', 'cover.size_variants', 'statistics', 'photos', 'photos.statistics', 'photos.size_variants', 'photos.palette', 'photos.tags']);
+			$base_query->with(['cover', 'cover.size_variants', 'statistics', 'photos', 'photos.statistics', 'photos.size_variants', 'photos.palette', 'photos.tags', 'photos.rating']);
 		}
 
 		// Only join what we need for ordering.

app/Actions/Albums/PositionData.php

@@ -47,6 +47,7 @@ public function do(): PositionDataResource
 					},
 					'palette',
 					'tags',
+					'rating',
 				])
 				->whereNotNull('latitude')
 				->whereNotNull('longitude'),

app/Actions/Photo/Pipes/Shared/SaveStatistics.php

@@ -26,6 +26,8 @@ public function handle(PhotoDTO $state, \Closure $next): PhotoDTO
 				'download_count' => 0,
 				'favourite_count' => 0,
 				'shared_count' => 0,
+				'rating_sum' => 0,
+				'rating_count' => 0,
 			]);
 
 			$state->getPhoto()->setRelation('statistics', $stats);

app/Actions/Photo/Rating.php

@@ -0,0 +1,103 @@
+<?php
+
+/**
+ * SPDX-License-Identifier: MIT
+ * Copyright (c) 2017-2018 Tobias Reich
+ * Copyright (c) 2018-2025 LycheeOrg.
+ */
+
+namespace App\Actions\Photo;
+
+use App\Exceptions\ConflictingPropertyException;
+use App\Models\Photo;
+use App\Models\PhotoRating;
+use App\Models\Statistics;
+use App\Models\User;
+use Illuminate\Support\Facades\DB;
+
+class Rating
+{
+	/**
+	 * Set or update the rating for the photo.
+	 *
+	 * Handles:
+	 * - Creating new ratings (rating > 0, no existing rating)
+	 * - Updating existing ratings (rating > 0, existing rating)
+	 * - Removing ratings (rating == 0)
+	 * - Atomic statistics updates
+	 *
+	 * @param Photo $photo  The photo to rate
+	 * @param User  $user   The user rating the photo
+	 * @param int   $rating The rating value (0-5, where 0 removes the rating)
+	 *
+	 * @return Photo the photo with refreshed statistics
+	 *
+	 * @throws ConflictingPropertyException if a database conflict occurs during the transaction
+	 */
+	public function do(Photo $photo, User $user, int $rating): Photo
+	{
+		try {
+			DB::transaction(function () use ($photo, $user, $rating): void {
+				// Ensure statistics record exists atomically (Q001-07)
+				$statistics = Statistics::firstOrCreate(
+					['photo_id' => $photo->id],
+					[
+						'album_id' => null,
+						'visit_count' => 0,
+						'download_count' => 0,
+						'favourite_count' => 0,
+						'shared_count' => 0,
+						'rating_sum' => 0,
+						'rating_count' => 0,
+					]
+				);
+
+				if ($rating > 0) {
+					// Find existing rating by this user for this photo
+					$existing_rating = PhotoRating::where('photo_id', $photo->id)
+						->where('user_id', $user->id)
+						->first();
+
+					if ($existing_rating !== null) {
+						// Update: adjust statistics delta
+						$delta = $rating - $existing_rating->rating;
+						$statistics->rating_sum += $delta;
+						$existing_rating->rating = $rating;
+						$existing_rating->save();
+					} else {
+						// Insert: create new rating and increment statistics
+						PhotoRating::create([
+							'photo_id' => $photo->id,
+							'user_id' => $user->id,
+							'rating' => $rating,
+						]);
+						$statistics->rating_sum += $rating;
+						$statistics->rating_count++;
+					}
+
+					$statistics->save();
+				} else {
+					// Rating == 0: remove rating (idempotent, Q001-06)
+					$existing_rating = PhotoRating::where('photo_id', $photo->id)
+						->where('user_id', $user->id)
+						->first();
+
+					if ($existing_rating !== null) {
+						$statistics->rating_sum -= $existing_rating->rating;
+						$statistics->rating_count--;
+						$statistics->save();
+						$existing_rating->delete();
+					}
+					// If no existing rating, do nothing (idempotent)
+				}
+			});
+
+			// Reload photo with fresh statistics
+			$photo->refresh();
+
+			return $photo;
+		} catch (\Throwable $e) {
+			throw new ConflictingPropertyException('Failed to update photo rating due to a conflict. Please try again.', $e);
+		}
+	}
+}

app/Actions/Photo/Timeline.php

@@ -52,7 +52,7 @@ public function do(): Builder
 		// @codeCoverageIgnoreEnd
 
 		return $this->photo_query_policy->applySearchabilityFilter(
-			query: Photo::query()->with(['statistics', 'size_variants', 'statistics', 'palette', 'tags']),
+			query: Photo::query()->with(['statistics', 'size_variants', 'statistics', 'palette', 'tags', 'rating']),
 			origin: null,
 			include_nsfw: !$this->config_manager->getValueAsBool('hide_nsfw_in_timeline')
 		)->orderBy($order->value, OrderSortingType::DESC->value);

app/Actions/Search/PhotoSearch.php

@@ -56,7 +56,7 @@ public function query(array $terms): Collection
 	public function sqlQuery(array $terms, ?Album $album = null): Builder
 	{
 		$query = $this->photo_query_policy->applySearchabilityFilter(
-			query: Photo::query()->with(['albums', 'statistics', 'size_variants', 'palette', 'tags']),
+			query: Photo::query()->with(['albums', 'statistics', 'size_variants', 'palette', 'tags', 'rating']),
 			origin: $album,
 			include_nsfw: !$this->config_manager->getValueAsBool('hide_nsfw_in_search')
 		);

app/Actions/Tag/GetTagWithPhotos.php

@@ -45,7 +45,7 @@ public function do(Tag $tag): TagWithPhotosResource
 		$user = Auth::user();
 
 		$base_query = Photo::query()
-			->with(['size_variants', 'statistics', 'palette', 'tags'])
+			->with(['size_variants', 'statistics', 'palette', 'tags', 'rating'])
 			->when(
 				$user->may_administrate === false,
 				fn ($q) => $q->where('photos.owner_id', Auth::id())

app/Contracts/Http/Requests/RequestAttribute.php

@@ -83,6 +83,7 @@ class RequestAttribute
 	public const FILE_ATTRIBUTE = 'file';
 	public const SHALL_OVERRIDE_ATTRIBUTE = 'shall_override';
 	public const IS_STARRED_ATTRIBUTE = 'is_starred';
+	public const RATING_ATTRIBUTE = 'rating';
 	public const DIRECTION_ATTRIBUTE = 'direction';
 
 	public const SINGLE_PATH_ATTRIBUTE = 'path';