[PR #16131] [CLOSED] fix(server): assets in multiple albums from being counted more than once #14758

Closed
opened 2026-02-05 15:37:17 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/16131
Author: @jbcodemonkey
Created: 2/16/2025
Status: Closed

Base: mainHead: patch-2


📝 Commits (1)

📊 Changes

1 file changed (+2 additions, -0 deletions)

View changed files

📝 server/src/repositories/map.repository.ts (+2 -0)

📄 Description

Description

Fixes assets in multiple albums from being counted more than once. Query returns only one row per unique asset.
Tested and working.

Fixes # (issue)
https://github.com/immich-app/immich/issues/16129

How Has This Been Tested?

Replaced file in current deployed container, restarted, verified fix.

Screenshots (if appropriate)

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/immich-app/immich/pull/16131 **Author:** [@jbcodemonkey](https://github.com/jbcodemonkey) **Created:** 2/16/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `patch-2` --- ### 📝 Commits (1) - [`9750ab6`](https://github.com/immich-app/immich/commit/9750ab62fb2c2985b84fae9fd26122860b3246e2) Update map.repository.ts ### 📊 Changes **1 file changed** (+2 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `server/src/repositories/map.repository.ts` (+2 -0) </details> ### 📄 Description ## Description Fixes assets in multiple albums from being counted more than once. Query returns only one row per unique asset. Tested and working. Fixes # (issue) https://github.com/immich-app/immich/issues/16129 ## How Has This Been Tested? Replaced file in current deployed container, restarted, verified fix. <details><summary><h2>Screenshots (if appropriate)</h2></summary> <!-- Images go below this line. --> </details> <!-- API endpoint changes (if relevant) ## API Changes The `/api/something` endpoint is now `/api/something-else` --> ## Checklist: - [x] I have performed a self-review of my own code - [x] I have made corresponding changes to the documentation if applicable - [x] I have no unrelated changes in the PR. - [x] I have confirmed that any new dependencies are strictly necessary. - [x] I have written tests for new code (if applicable) - [x] I have followed naming conventions/patterns in the surrounding code - [x] All code in `src/services` uses repositories implementations for database calls, filesystem operations, etc. - [x] All code in `src/repositories/` is pretty basic/simple and does not have any immich specific logic (that belongs in `src/services`) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 15:37:17 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#14758