src/controllers/TrackController.ts

@@ -4,7 +4,6 @@ import { EntityFromBody } from 'typeorm-routing-controllers-extensions';
 import { OpenAPI, ResponseSchema } from 'routing-controllers-openapi';
 import { Track } from '../models/Track';
 import { IsInt, IsNotEmpty, IsPositive, IsString } from 'class-validator';
-import {TrackIdChangeNotAllowedError, TrackNotFoundError} from "../errors/TrackErrors";
 
 class CreateTrack {
 	@IsString()
@@ -16,6 +15,12 @@ class CreateTrack {
 	length: number;
 }
 
+export class TrackNotFoundError extends NotFoundError {
+	constructor() {
+		super('Track not found!');
+	}
+}
+
 @JsonController('/tracks')
 @Authorized("TRACKS:read")
 export class TrackController {
@@ -37,7 +42,6 @@ export class TrackController {
 
 	@Get('/:id')
 	@ResponseSchema(Track)
-	@ResponseSchema(TrackNotFoundError, {statusCode: 404})
 	@OnUndefined(TrackNotFoundError)
 	@OpenAPI({ description: "Returns a track of a specified id (if it exists)" })
 	getOne(@Param('id') id: number) {
@@ -56,9 +60,7 @@ export class TrackController {
 
 	@Put('/:id')
 	@ResponseSchema(Track)
-	@ResponseSchema(TrackNotFoundError, {statusCode: 404})
-	@ResponseSchema(TrackIdChangeNotAllowedError, {statusCode: 406})
-	@OpenAPI({description: "Update a track object (id can't be changed)."})
+	@OpenAPI({ description: "Update a track object (id can't be changed)." })
 	async put(@Param('id') id: number, @EntityFromBody() track: Track) {
 		let oldTrack = await this.trackRepository.findOne({ id: id });
 
@@ -66,8 +68,8 @@ export class TrackController {
 			throw new TrackNotFoundError();
 		}
 
-		if(oldTrack.id != track.id){
-			throw new TrackIdChangeNotAllowedError();
+		if (oldTrack.id != track.id) {
+			throw new NotAcceptableError("The id's don't match!");
 		}
 
 		await this.trackRepository.update(oldTrack, track);
@@ -76,8 +78,7 @@ export class TrackController {
 
 	@Delete('/:id')
 	@ResponseSchema(Track)
-	@ResponseSchema(TrackNotFoundError, {statusCode: 404})
-	@OpenAPI({description: "Delete a specified track (if it exists)."})
+	@OpenAPI({ description: "Delete a specified track (if it exists)." })
 	async remove(@Param('id') id: number) {
 		let track = await this.trackRepository.findOne({ id: id });
 

src/errors/TrackErrors.ts

@@ -1,18 +0,0 @@
-import { JsonController, Param, Body, Get, Post, Put, Delete, NotFoundError, OnUndefined, NotAcceptableError } from 'routing-controllers';
-import { IsInt, IsNotEmpty, IsPositive, IsString } from 'class-validator';
-
-export class TrackNotFoundError extends NotFoundError {
-	@IsString()
-	name = "TrackNotFoundError"
-
-	@IsString()
-	message = "Track not found!"
-}
-
-export class TrackIdChangeNotAllowedError extends NotAcceptableError {
-	@IsString()
-	name = "TrackIdChangeNotAllowed"
-
-	@IsString()
-	message = "The id's don't match!!"
-}