Updated issue body with current progress and split into two phases based on codebase analysis (July 13):
Phase 1 — Fill remaining REST Client gaps (Media, Folder, Jobs, Tag CRUD, Settings…
<div @onclick> instead of <a> breaks "open in new tab"
Addressed the validation review points:
- AlbumController, TagController: reverted
PageSize < 1back to originalPageSize < 0 - PersonController: removed
PageSize < 1, kept only…
Fixed the 4 failing tests (23, 24, 25, 45, 46):
- Root cause:
SharedWith.Any()navigation through the many-to-manyAlbumAssetjoin table couldn't be translated by EF Core inside a SQL…
Rebased & addressed all review feedback:
- Pagination zero-based standard (closes #95):
- AssetRepository: all Skip() →
page * size(3 occurrences) - JobRecordRepository: same formula…
- AssetRepository: all Skip() →
Added performance index for visibility queries (commit 48b6d4f)
Created a partial index IX_Assets_VisibleForSearch on Assets(IsPubliclyShared, OwnerId) WHERE "DeletedAt" IS NULL to…
Refactored AlbumRepository.SearchQuery to match the new PersonRepository pattern (commit 89edf29)
Same approach as the Person repository — projects AlbumPreviewDto directly with…
Addressed review feedback on TotalVisibleAlbums (commit c77096b)
Removed the [NotMapped] TotalVisibleAlbums property from Person.cs and refactored PersonRepository.SearchQuery to…