github/hedgedoc
1
0
Fork 0
mirror of https://github.com/hedgedoc/hedgedoc.git synced 2025-02-25 02:03:05 +00:00
Code Issues Projects Releases Packages Wiki Activity

NotesService: Rename getCurrentContent to getNoteContentByNote

Browse source 
The new name should better explain what this functions does.

Signed-off-by: Philip Molares <philip.molares@udo.edu>
This commit is contained in:
Philip Molares 2021-02-19 13:36:01 +01:00
parent 0131297fbb
commit 336117cef5
3 changed files with 16 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/api/public/notes/notes.controller.ts
View file

@ -164,7 +164,7 @@ export class NotesController {
if (!this.permissionsService.mayRead(req.user, note)) {
throw new UnauthorizedException('Reading note denied!');
}
return await this.noteService.getNoteContent(noteIdOrAlias);
return await this.noteService.getNoteContentByNote(note);
} catch (e) {
if (e instanceof NotInDBError) {
throw new NotFoundException(e.message);

15
src/notes/notes.service.ts
View file

@ -74,10 +74,19 @@ export class NotesService {
newNote.historyEntries = [HistoryEntry.create(owner)];
newNote.owner = owner;
}
return this.noteRepository.save(newNote);
try {
return await this.noteRepository.save(newNote);
} catch {
this.logger.debug(
`A note with the alias '${alias}' already exists.`,
'createNote',
);
throw new AlreadyInDBError(
`A note with the alias '${alias}' already exists.`,
);
}
}
async getCurrentContent(note: Note): Promise<string> {
async getNoteContentByNote(note: Note): Promise<string> {
return (await this.getLatestRevision(note)).content;
}

6
test/public-api/notes.e2e-spec.ts
View file

@ -74,7 +74,7 @@ describe('Notes', () => {
.expect(201);
expect(response.body.metadata?.id).toBeDefined();
expect(
await notesService.getCurrentContent(
await notesService.getNoteContentByNote(
await notesService.getNoteByIdOrAlias(response.body.metadata.id),
),
).toEqual(content);
@ -109,7 +109,7 @@ describe('Notes', () => {
.expect(201);
expect(response.body.metadata?.id).toBeDefined();
return expect(
await notesService.getCurrentContent(
await notesService.getNoteContentByNote(
await notesService.getNoteByIdOrAlias(response.body.metadata?.id),
),
).toEqual(content);
@ -141,7 +141,7 @@ describe('Notes', () => {
.send(changedContent)
.expect(200);
await expect(
await notesService.getCurrentContent(
await notesService.getNoteContentByNote(
await notesService.getNoteByIdOrAlias('test4'),
),
).toEqual(changedContent);