Merge pull request #111 from TheRefraction/hotfix-binding

Hotfix con issue

Author: TheRefraction

backend/src/controllers/account.controller.ts

@@ -12,7 +12,7 @@ import { HttpStatus } from '../utils/httpStatus';
 export class AccountController extends BaseController {
     constructor(private service: AccountService) { super(); }
 
-    async register(req: Request, res: Response, next: NextFunction) {
+    register = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const result = await this.service.register(req.body);
 
@@ -22,7 +22,7 @@ export class AccountController extends BaseController {
         }
     }
 
-    async login(req: Request, res: Response, next: NextFunction) {
+    login = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const { email, password } = req.body;
 
@@ -34,7 +34,7 @@ export class AccountController extends BaseController {
         }
     }
 
-    async getProfile(req: AuthRequest, res: Response, next: NextFunction) {
+    getProfile = async (req: AuthRequest, res: Response, next: NextFunction): Promise<void> => {
         try {
             const userId = req.user!.userId;
 
@@ -46,7 +46,7 @@ export class AccountController extends BaseController {
         }
     }
 
-    async getAccountById(req: Request, res: Response, next: NextFunction) {
+    getAccountById = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -58,7 +58,7 @@ export class AccountController extends BaseController {
         }
     }
 
-    async getAllAccounts(req: AuthRequest, res: Response, next: NextFunction) {
+    getAllAccounts = async (req: AuthRequest, res: Response, next: NextFunction): Promise<void> => {
         try {
             if (req.user!.role !== Role.Admin) {
                 this.sendResponse(res, HttpStatus.FORBIDDEN, 'Permission denied');
@@ -82,13 +82,14 @@ export class AccountController extends BaseController {
         }
     }
 
-    async updateAccount(req: AuthRequest, res: Response, next: NextFunction) {
+    updateAccount = async (req: AuthRequest, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
             const userId = req.user?.userId;
 
             if (!userId) {
-                return res.status(HttpStatus.UNAUTHORIZED).json({ message: 'User not authenticated' });
+                res.status(HttpStatus.UNAUTHORIZED).json({ message: 'User not authenticated' });
+                return;
             }
 
             const result = await this.service.update(id, req.body, userId);
@@ -99,13 +100,14 @@ export class AccountController extends BaseController {
         }
     }
 
-    async deleteAccount(req: AuthRequest, res: Response, next: NextFunction) {
+    deleteAccount = async (req: AuthRequest, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
             const userId = req.user?.userId;
 
             if (!userId) {
-                return res.status(HttpStatus.UNAUTHORIZED).json({ message: 'User not authenticated' });
+                res.status(HttpStatus.UNAUTHORIZED).json({ message: 'User not authenticated' });
+                return;
             }
 
             await this.service.delete(id, userId);

backend/src/controllers/category.controller.ts

@@ -6,7 +6,7 @@ import { HttpStatus } from '../utils/httpStatus';
 export class CategoryController extends BaseController {
     constructor(private categorySvc: CategoryService) { super(); }
 
-    async getAll(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getAll = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const categories = await this.categorySvc.getAll();
 
@@ -16,7 +16,7 @@ export class CategoryController extends BaseController {
         }
     }
 
-    async getById(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getById = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -28,7 +28,7 @@ export class CategoryController extends BaseController {
         }
     }
 
-    async create(req: Request, res: Response, next: NextFunction): Promise<void> {
+    create = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const newCategory = await this.categorySvc.create(req.body);
 
@@ -38,7 +38,7 @@ export class CategoryController extends BaseController {
         }
     }
 
-    async update(req: Request, res: Response, next: NextFunction): Promise<void> {
+    update = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -50,7 +50,7 @@ export class CategoryController extends BaseController {
         }
     }
 
-    async delete(req: Request, res: Response, next: NextFunction): Promise<void> {
+    delete = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 

backend/src/controllers/customization.controller.ts

@@ -12,7 +12,7 @@ export class CustomizationController extends BaseController {
         super(); 
     }
 
-    async getAll(req: Request, res : Response, next : NextFunction): Promise<void> {
+    getAll = async(req: Request, res : Response, next : NextFunction): Promise<void> => {
         try {
             const slots = await this.slotSvc.getAll();
 
@@ -22,7 +22,7 @@ export class CustomizationController extends BaseController {
         }
     }
 
-    async getAllByProductId(req: Request, res: Response, next : NextFunction): Promise<void>{
+    getAllByProductId = async (req: Request, res: Response, next : NextFunction): Promise<void> => {
         try {
             const productId = parseInt(req.params.id);
 
@@ -34,7 +34,7 @@ export class CustomizationController extends BaseController {
         }
     }
 
-    async getFullSlotsByProductId(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getFullSlotsByProductId = async(req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const productId = parseInt(req.params.id);
 
@@ -46,7 +46,7 @@ export class CustomizationController extends BaseController {
         }
     }
 
-    async getbyId(req: Request, res : Response, next : NextFunction): Promise<void> {
+    getbyId = async(req: Request, res : Response, next : NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -58,7 +58,7 @@ export class CustomizationController extends BaseController {
         }
     }
 
-    async getFullSlotById(req: Request, res: Response, next : NextFunction): Promise<void> {
+    getFullSlotById = async (req: Request, res: Response, next : NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -70,7 +70,7 @@ export class CustomizationController extends BaseController {
         }
     }   
 
-    async create(req: Request, res : Response, next: NextFunction): Promise<void>{
+    create = async (req: Request, res : Response, next: NextFunction): Promise<void> => {
         try {
             const slot = await this.slotSvc.create(req.body);
 
@@ -80,7 +80,7 @@ export class CustomizationController extends BaseController {
         }
     }
 
-    async update(req: Request, res: Response, next: NextFunction): Promise<void> {
+    update = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -92,7 +92,7 @@ export class CustomizationController extends BaseController {
         }
     }
 
-    async delete(req: Request, res: Response, next: NextFunction): Promise<void>{
+    delete = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 

backend/src/controllers/invoice.controller.ts

@@ -6,7 +6,7 @@ const invoiceService = new InvoiceService();
 export class InvoiceController {
 
 
-    async create(req: Request, res: Response, next: NextFunction) {
+    create = async (req: Request, res: Response, next: NextFunction) => {
         try {
             const invoiceData = req.body;
             const createdInvoice = await invoiceService.createInvoice(invoiceData);
@@ -24,9 +24,9 @@ export class InvoiceController {
     }
 
 
-    async getById(req: Request, res: Response, next: NextFunction) {
+    getById = async(req: Request, res: Response, next: NextFunction) => {
         try {
-            const invoiceId = parseInt(req.params.id, 10);
+            const invoiceId = parseInt(req.params.id);
             const fullInvoice = await invoiceService.getInvoiceData(invoiceId);
 
             res.status(200).json({

backend/src/controllers/option.controller.ts

@@ -12,7 +12,7 @@ export class OptionController extends BaseController {
         super(); 
     }
 
-    async getAllBySlotId(req: Request, res: Response, next : NextFunction): Promise<void>{
+    getAllBySlotId = async (req: Request, res: Response, next : NextFunction): Promise<void> => {
         try {
             const slotId = parseInt(req.params.id);
 
@@ -24,7 +24,7 @@ export class OptionController extends BaseController {
         }
     }
 
-    async getBySlotAndProduct(req: Request, res: Response, next : NextFunction): Promise<void>{
+    getBySlotAndProduct = async (req: Request, res: Response, next : NextFunction): Promise<void> => {
         try {
             const slotId = parseInt(req.params.id);
 
@@ -36,7 +36,7 @@ export class OptionController extends BaseController {
         }
     }
 
-    async create(req: Request, res: Response, next: NextFunction): Promise <void>{
+    create = async (req: Request, res: Response, next: NextFunction): Promise <void> => {
         try{
             const result = await this.slotFcd.addOptionToSlot(req.body);
 
@@ -46,7 +46,7 @@ export class OptionController extends BaseController {
         }
     }
 
-    async update(req: Request, res: Response, next: NextFunction): Promise<void> {
+    update = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const slotId = parseInt (req.params.id);
 
@@ -58,7 +58,7 @@ export class OptionController extends BaseController {
         }
     }
 
-    async delete(req: Request, res: Response, next: NextFunction): Promise<void>{
+    delete = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const slotId = parseInt(req.params.id);
 

backend/src/controllers/product.controller.ts

@@ -12,7 +12,7 @@ export class ProductController extends BaseController {
         super();
     }
 
-    async getAll(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getAll = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const products = await this.productSvc.getAll();
 
@@ -22,7 +22,7 @@ export class ProductController extends BaseController {
         }
     }
 
-    async getById(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getById = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -34,7 +34,7 @@ export class ProductController extends BaseController {
         }
     }
 
-    async getFullProduct(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getFullProduct = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -46,7 +46,7 @@ export class ProductController extends BaseController {
         }
     }
 
-    async create(req: Request, res: Response, next: NextFunction): Promise<void> {
+    create = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const result = await this.productSvc.create(req.body);
 
@@ -56,7 +56,7 @@ export class ProductController extends BaseController {
         }
     }
 
-    async update(req: Request, res: Response, next: NextFunction): Promise<void> {
+    update = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 
@@ -68,7 +68,7 @@ export class ProductController extends BaseController {
         }
     }
 
-    async delete(req: Request, res: Response, next: NextFunction): Promise<void> {
+    delete = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = parseInt(req.params.id);
 

backend/src/controllers/supplier.controller.ts

@@ -6,7 +6,7 @@ import { HttpStatus } from '../utils/httpStatus';
 export class SupplierController extends BaseController {
     constructor(private supplierSvc: SupplierService) { super(); }
 
-    async getAll(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getAll = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const result = await this.supplierSvc.getAll();
 
@@ -16,7 +16,7 @@ export class SupplierController extends BaseController {
         }
     }
 
-    async getById(req: Request, res: Response, next: NextFunction): Promise<void> {
+    getById = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const id = req.params.id;
             const sanId : number = parseInt(id);
@@ -29,7 +29,7 @@ export class SupplierController extends BaseController {
         }
     }
 
-    async create(req: Request, res: Response, next: NextFunction): Promise<void> {
+    create = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const result = await this.supplierSvc.create(req.body);
 
@@ -39,7 +39,7 @@ export class SupplierController extends BaseController {
         }
     }
 
-    async update(req: Request, res: Response, next: NextFunction): Promise<void> {
+    update = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const { id } = req.params;
             const sanId = parseInt(id);
@@ -52,7 +52,7 @@ export class SupplierController extends BaseController {
         }
     }
 
-    async delete(req: Request, res: Response, next: NextFunction): Promise<void> {
+    delete = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
         try {
             const { id } = req.params;
             const sanId = parseInt(id);