History: Add history service and usage

Add history service to allow for CRUD operations.
Use history service in controllers to:
  1. Allow manipulating of history entries
  2. Guaranty the correct existence of history entries

Signed-off-by: Philip Molares <philip.molares@udo.edu>

src/api/public/me/me.controller.ts

@@ -17,15 +17,16 @@ import {
   Request,
 } from '@nestjs/common';
 import { HistoryEntryUpdateDto } from '../../../history/history-entry-update.dto';
-import { HistoryEntryDto } from '../../../history/history-entry.dto';
 import { HistoryService } from '../../../history/history.service';
 import { ConsoleLoggerService } from '../../../logger/console-logger.service';
 import { NoteMetadataDto } from '../../../notes/note-metadata.dto';
 import { NotesService } from '../../../notes/notes.service';
-import { UserInfoDto } from '../../../users/user-info.dto';
 import { UsersService } from '../../../users/users.service';
 import { TokenAuthGuard } from '../../../auth/token-auth.guard';
 import { ApiSecurity } from '@nestjs/swagger';
+import { HistoryEntryDto } from '../../../history/history-entry.dto';
+import { UserInfoDto } from '../../../users/user-info.dto';
+import { NotInDBError } from '../../../errors/errors';
 
 @ApiSecurity('token')
 @Controller('me')
@@ -49,19 +50,37 @@ export class MeController {
 
   @UseGuards(TokenAuthGuard)
   @Get('history')
-  getUserHistory(@Request() req): HistoryEntryDto[] {
+  async getUserHistory(@Request() req): Promise<HistoryEntryDto[]> {
-    return this.historyService.getUserHistory(req.user.userName);
+    const foundEntries = await this.historyService.getEntriesByUser(req.user);
+    return Promise.all(
+      foundEntries.map(
+        async (entry) => await this.historyService.toHistoryEntryDto(entry),
+      ),
+    );
   }
 
   @UseGuards(TokenAuthGuard)
   @Put('history/:note')
-  updateHistoryEntry(
+  async updateHistoryEntry(
     @Request() req,
     @Param('note') note: string,
     @Body() entryUpdateDto: HistoryEntryUpdateDto,
-  ): HistoryEntryDto {
+  ): Promise<HistoryEntryDto> {
     // ToDo: Check if user is allowed to pin this history entry
-    return this.historyService.updateHistoryEntry(note, entryUpdateDto);
+    try {
+      return this.historyService.toHistoryEntryDto(
+        await this.historyService.updateHistoryEntry(
+          note,
+          req.user,
+          entryUpdateDto,
+        ),
+      );
+    } catch (e) {
+      if (e instanceof NotInDBError) {
+        throw new NotFoundException(e.message);
+      }
+      throw e;
+    }
   }
 
   @UseGuards(TokenAuthGuard)
@@ -70,10 +89,13 @@ export class MeController {
   deleteHistoryEntry(@Request() req, @Param('note') note: string) {
     // ToDo: Check if user is allowed to delete note
     try {
-      return this.historyService.deleteHistoryEntry(note);
+      return this.historyService.deleteHistoryEntry(note, req.user);
     } catch (e) {
+      if (e instanceof NotInDBError) {
         throw new NotFoundException(e.message);
       }
+      throw e;
+    }
   }
 
   @UseGuards(TokenAuthGuard)

src/api/public/notes/notes.controller.ts

@@ -28,6 +28,7 @@ import { RevisionsService } from '../../../revisions/revisions.service';
 import { MarkdownBody } from '../../utils/markdownbody-decorator';
 import { TokenAuthGuard } from '../../../auth/token-auth.guard';
 import { ApiSecurity } from '@nestjs/swagger';
+import { HistoryService } from '../../../history/history.service';
 import { NoteDto } from '../../../notes/note.dto';
 import { NoteMetadataDto } from '../../../notes/note-metadata.dto';
 import { RevisionMetadataDto } from '../../../revisions/revision-metadata.dto';
@@ -40,6 +41,7 @@ export class NotesController {
     private readonly logger: ConsoleLoggerService,
     private noteService: NotesService,
     private revisionsService: RevisionsService,
+    private historyService: HistoryService,
   ) {
     this.logger.setContext(NotesController.name);
   }
@@ -57,6 +59,22 @@ export class NotesController {
     );
   }
 
+  @UseGuards(TokenAuthGuard)
+  @Get(':noteIdOrAlias')
+  async getNote(@Request() req, @Param('noteIdOrAlias') noteIdOrAlias: string) {
+    // ToDo: check if user is allowed to view this note
+    try {
+      const note = await this.noteService.getNoteByIdOrAlias(noteIdOrAlias);
+      await this.historyService.createOrUpdateHistoryEntry(note, req.user);
+      return this.noteService.toNoteDto(note);
+    } catch (e) {
+      if (e instanceof NotInDBError) {
+        throw new NotFoundException(e.message);
+      }
+      throw e;
+    }
+  }
+
   @UseGuards(TokenAuthGuard)
   @Post(':noteAlias')
   async createNamedNote(
@@ -71,25 +89,6 @@ export class NotesController {
     );
   }
 
-  @UseGuards(TokenAuthGuard)
-  @Get(':noteIdOrAlias')
-  async getNote(
-    @Request() req,
-    @Param('noteIdOrAlias') noteIdOrAlias: string,
-  ): Promise<NoteDto> {
-    // ToDo: check if user is allowed to view this note
-    try {
-      return this.noteService.toNoteDto(
-        await this.noteService.getNoteByIdOrAlias(noteIdOrAlias),
-      );
-    } catch (e) {
-      if (e instanceof NotInDBError) {
-        throw new NotFoundException(e.message);
-      }
-      throw e;
-    }
-  }
-
   @UseGuards(TokenAuthGuard)
   @Delete(':noteIdOrAlias')
   async deleteNote(

src/history/history-entry.dto.ts

@@ -4,15 +4,33 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
-import { IsBoolean, ValidateNested } from 'class-validator';
+import { IsArray, IsBoolean, IsDate, IsString } from 'class-validator';
-import { NoteMetadataDto } from '../notes/note-metadata.dto';
 
 export class HistoryEntryDto {
   /**
-   * Metadata of this note
+   * ID or Alias of the note
    */
-  @ValidateNested()
+  @IsString()
-  metadata: NoteMetadataDto;
+  identifier: string;
+
+  /**
+   * Title of the note
+   * Does not contain any markup but might be empty
+   * @example "Shopping List"
+   */
+  @IsString()
+  title: string;
+
+  /**
+   * Datestring of the last time this note was updated
+   * @example "2020-12-01 12:23:34"
+   */
+  @IsDate()
+  lastVisited: Date;
+
+  @IsArray()
+  @IsString({ each: true })
+  tags: string[];
 
   /**
    * True if this note is pinned

src/history/history-entry.entity.ts

@@ -4,12 +4,7 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
-import {
+import { Column, Entity, ManyToOne, UpdateDateColumn } from 'typeorm';
-  Column,
-  Entity,
-  ManyToOne,
-  UpdateDateColumn,
-} from 'typeorm';
 import { User } from '../users/user.entity';
 import { Note } from '../notes/note.entity';
 

src/history/history.module.ts

@@ -7,10 +7,19 @@
 import { Module } from '@nestjs/common';
 import { LoggerModule } from '../logger/logger.module';
 import { HistoryService } from './history.service';
+import { TypeOrmModule } from '@nestjs/typeorm';
+import { HistoryEntry } from './history-entry.entity';
+import { UsersModule } from '../users/users.module';
+import { NotesModule } from '../notes/notes.module';
 
 @Module({
   providers: [HistoryService],
   exports: [HistoryService],
-  imports: [LoggerModule],
+  imports: [
+    LoggerModule,
+    TypeOrmModule.forFeature([HistoryEntry]),
+    UsersModule,
+    NotesModule,
+  ],
 })
 export class HistoryModule {}

src/history/history.service.ts

@@ -8,90 +8,98 @@ import { Injectable } from '@nestjs/common';
 import { ConsoleLoggerService } from '../logger/console-logger.service';
 import { HistoryEntryUpdateDto } from './history-entry-update.dto';
 import { HistoryEntryDto } from './history-entry.dto';
+import { InjectRepository } from '@nestjs/typeorm';
+import { Repository } from 'typeorm';
+import { HistoryEntry } from './history-entry.enity';
+import { UsersService } from '../users/users.service';
+import { NotesService } from '../notes/notes.service';
+import { User } from '../users/user.entity';
+import { Note } from '../notes/note.entity';
+import { NotInDBError } from '../errors/errors';
 
 @Injectable()
 export class HistoryService {
-  constructor(private readonly logger: ConsoleLoggerService) {
+  constructor(
+    private readonly logger: ConsoleLoggerService,
+    @InjectRepository(HistoryEntry)
+    private historyEntryRepository: Repository<HistoryEntry>,
+    private usersService: UsersService,
+    private notesService: NotesService,
+  ) {
     this.logger.setContext(HistoryService.name);
   }
 
-  getUserHistory(username: string): HistoryEntryDto[] {
+  async getEntriesByUser(user: User): Promise<HistoryEntry[]> {
-    //TODO: Use the database
+    return await this.historyEntryRepository.find({
-    this.logger.warn('Using hardcoded data!');
+      where: { user: user },
-    return [
+      relations: ['note'],
-      {
+    });
-        metadata: {
-          alias: null,
-          createTime: new Date(),
-          description: 'Very descriptive text.',
-          editedBy: [],
-          id: 'foobar-barfoo',
-          permissions: {
-            owner: {
-              displayName: 'foo',
-              userName: 'fooUser',
-              email: 'foo@example.com',
-              photo: '',
-            },
-            sharedToUsers: [],
-            sharedToGroups: [],
-          },
-          tags: [],
-          title: 'Title!',
-          updateTime: new Date(),
-          updateUser: {
-            displayName: 'foo',
-            userName: 'fooUser',
-            email: 'foo@example.com',
-            photo: '',
-          },
-          viewCount: 42,
-        },
-        pinStatus: false,
-      },
-    ];
   }
 
-  updateHistoryEntry(
+  private async getEntryByNoteIdOrAlias(
-    noteId: string,
+    noteIdOrAlias: string,
+    user: User,
+  ): Promise<HistoryEntry> {
+    const note = await this.notesService.getNoteByIdOrAlias(noteIdOrAlias);
+    return await this.getEntryByNote(note, user);
+  }
+
+  private async getEntryByNote(note: Note, user: User): Promise<HistoryEntry> {
+    return await this.historyEntryRepository.findOne({
+      where: {
+        note: note,
+        user: user,
+      },
+      relations: ['note', 'user'],
+    });
+  }
+
+  async createOrUpdateHistoryEntry(
+    note: Note,
+    user: User,
+  ): Promise<HistoryEntry> {
+    let entry = await this.getEntryByNote(note, user);
+    if (!entry) {
+      entry = HistoryEntry.create(user, note);
+    } else {
+      entry.updatedAt = new Date();
+    }
+    return await this.historyEntryRepository.save(entry);
+  }
+
+  async updateHistoryEntry(
+    noteIdOrAlias: string,
+    user: User,
     updateDto: HistoryEntryUpdateDto,
-  ): HistoryEntryDto {
+  ): Promise<HistoryEntry> {
-    //TODO: Use the database
+    const entry = await this.getEntryByNoteIdOrAlias(noteIdOrAlias, user);
-    this.logger.warn('Using hardcoded data!');
+    if (!entry) {
+      throw new NotInDBError(
+        `User '${user.userName}' has no HistoryEntry for Note with id '${noteIdOrAlias}'`,
+      );
+    }
+    entry.pinStatus = updateDto.pinStatus;
+    return this.historyEntryRepository.save(entry);
+  }
+
+  async deleteHistoryEntry(noteIdOrAlias: string, user: User): Promise<void> {
+    const entry = await this.getEntryByNoteIdOrAlias(noteIdOrAlias, user);
+    if (!entry) {
+      throw new NotInDBError(
+        `User '${user.userName}' has no HistoryEntry for Note with id '${noteIdOrAlias}'`,
+      );
+    }
+    await this.historyEntryRepository.remove(entry);
+    return;
+  }
+
+  async toHistoryEntryDto(entry: HistoryEntry): Promise<HistoryEntryDto> {
     return {
-      metadata: {
+      identifier: entry.note.alias ? entry.note.alias : entry.note.id,
-        alias: null,
+      lastVisited: entry.updatedAt,
-        createTime: new Date(),
+      tags: this.notesService.toTagList(entry.note),
-        description: 'Very descriptive text.',
+      title: entry.note.title,
-        editedBy: [],
+      pinStatus: entry.pinStatus,
-        id: 'foobar-barfoo',
-        permissions: {
-          owner: {
-            displayName: 'foo',
-            userName: 'fooUser',
-            email: 'foo@example.com',
-            photo: '',
-          },
-          sharedToUsers: [],
-          sharedToGroups: [],
-        },
-        tags: [],
-        title: 'Title!',
-        updateTime: new Date(),
-        updateUser: {
-          displayName: 'foo',
-          userName: 'fooUser',
-          email: 'foo@example.com',
-          photo: '',
-        },
-        viewCount: 42,
-      },
-      pinStatus: updateDto.pinStatus,
     };
   }
-
-  deleteHistoryEntry(note: string) {
-    //TODO: Use the database and actually do stuff
-    throw new Error('Not implemented');
-  }
 }

src/notes/notes.service.ts

@@ -177,6 +177,10 @@ export class NotesService {
     return this.getCurrentContent(note);
   }
 
+  toTagList(note: Note): string[] {
+    return note.tags.map((tag) => tag.name);
+  }
+
   async toNotePermissionsDto(note: Note): Promise<NotePermissionsDto> {
     return {
       owner: this.usersService.toUserDto(note.owner),
@@ -204,7 +208,7 @@ export class NotesService {
       ),
       // TODO: Extract into method
       permissions: await this.toNotePermissionsDto(note),
-      tags: note.tags.map((tag) => tag.name),
+      tags: this.toTagList(note),
       updateTime: (await this.getLatestRevision(note)).createdAt,
       // TODO: Get actual updateUser
       updateUser: {

test/public-api/users.e2e-spec.ts

@@ -85,7 +85,7 @@ describe('Notes', () => {
       .delete('/me/history/test3')
       .expect(204);
     expect(response.body.content).toBeNull();
-    const history = historyService.getUserHistory('testuser');
+    const history = historyService.getEntriesByUser('testuser');
     let historyEntry: HistoryEntryDto = null;
     for (const e of history) {
       if (e.metadata.alias === noteName) {
@@ -106,7 +106,7 @@ describe('Notes', () => {
       .send(historyEntryUpdateDto)
       .expect(200);
     // TODO parameter is not used for now
-    const history = historyService.getUserHistory('testuser');
+    const history = historyService.getEntriesByUser('testuser');
     let historyEntry: HistoryEntryDto;
     for (const e of <any[]>response.body.content) {
       if ((<HistoryEntryDto>e).metadata.alias === noteName) {