New getAllAlbumsSlim endpoint

- Add new endpoint
- Add slim option to albumService.getAll (default false)
- Use getAllAlbumsSlim in search-albums-section

server/src/controllers/album.controller.spec.ts

@@ -37,6 +37,25 @@ describe(AlbumController.name, () => {
     });
   });
 
+  describe('GET /albums/slim', () => {
+    it('should be an authenticated route', async () => {
+      await request(ctx.getHttpServer()).post('/albums/slim');
+      expect(ctx.authenticate).toHaveBeenCalled();
+    });
+
+    it('should reject an invalid shared param', async () => {
+      const { status, body } = await request(ctx.getHttpServer()).get('/albums/slim?shared=invalid');
+      expect(status).toEqual(400);
+      expect(body).toEqual(factory.responses.badRequest(['shared must be a boolean value']));
+    });
+
+    it('should reject an invalid assetId param', async () => {
+      const { status, body } = await request(ctx.getHttpServer()).get('/albums/slim?assetId=invalid');
+      expect(status).toEqual(400);
+      expect(body).toEqual(factory.responses.badRequest(['assetId must be a UUID']));
+    });
+  });
+
   describe('GET /albums/:id', () => {
     it('should be an authenticated route', async () => {
       await request(ctx.getHttpServer()).get(`/albums/${factory.uuid()}`);

server/src/controllers/album.controller.ts

@@ -28,6 +28,13 @@ export class AlbumController {
     return this.service.getAll(auth, query);
   }
 
+  @Get('slim')
+  @Authenticated({ permission: Permission.ALBUM_READ })
+  getAllAlbumsSlim(@Auth() auth: AuthDto, @Query() query: GetAlbumsDto): Promise<AlbumResponseDto[]> {
+    return this.service.getAll(auth, query, true);
+    //asdf
+  }
+
   @Post()
   @Authenticated({ permission: Permission.ALBUM_CREATE })
   createAlbum(@Auth() auth: AuthDto, @Body() dto: CreateAlbumDto): Promise<AlbumResponseDto> {

server/src/services/album.service.ts

@@ -37,7 +37,11 @@ export class AlbumService extends BaseService {
     };
   }
 
-  async getAll({ user: { id: ownerId } }: AuthDto, { assetId, shared }: GetAlbumsDto): Promise<AlbumResponseDto[]> {
+  async getAll(
+    { user: { id: ownerId } }: AuthDto,
+    { assetId, shared }: GetAlbumsDto,
+    slim: boolean = false,
+  ): Promise<AlbumResponseDto[]> {
     await this.albumRepository.updateThumbnails();
 
     let albums: MapAlbumDto[];
@@ -53,20 +57,24 @@ export class AlbumService extends BaseService {
 
     // Get asset count for each album. Then map the result to an object:
     // { [albumId]: assetCount }
-    const results = await this.albumRepository.getMetadataForIds(albums.map((album) => album.id));
     const albumMetadata: Record<string, AlbumAssetCount> = {};
-    for (const metadata of results) {
-      albumMetadata[metadata.albumId] = metadata;
+    if (!slim) {
+      const results = await this.albumRepository.getMetadataForIds(albums.map((album) => album.id));
+      for (const metadata of results) {
+        albumMetadata[metadata.albumId] = metadata;
+      }
     }
 
     return albums.map((album) => ({
       ...mapAlbumWithoutAssets(album),
       sharedLinks: undefined,
-      startDate: albumMetadata[album.id]?.startDate ?? undefined,
-      endDate: albumMetadata[album.id]?.endDate ?? undefined,
-      assetCount: albumMetadata[album.id]?.assetCount ?? 0,
-      // lastModifiedAssetTimestamp is only used in mobile app, please remove if not need
-      lastModifiedAssetTimestamp: albumMetadata[album.id]?.lastModifiedAssetTimestamp ?? undefined,
+      ...(!slim && {
+        startDate: albumMetadata[album.id]?.startDate ?? undefined,
+        endDate: albumMetadata[album.id]?.endDate ?? undefined,
+        assetCount: albumMetadata[album.id]?.assetCount ?? 0,
+        // lastModifiedAssetTimestamp is only used in mobile app, please remove if not need
+        lastModifiedAssetTimestamp: albumMetadata[album.id]?.lastModifiedAssetTimestamp ?? undefined,
+      }),
     }));
   }