Addressed Mateo's comment of filterByPrice

cuappdev · Sep 20, 2023 · ab9db56 · ab9db56
1 parent dc8ffc1
commit ab9db56
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/repositories/PostRepository.ts b/src/repositories/PostRepository.ts
@@ -95,8 +95,8 @@ export class PostRepository extends AbstractRepository<PostModel> {
     return await this.repository
       .createQueryBuilder("post")
       .leftJoinAndSelect("post.user", "user")
-      .where("post.original_price >= :lowerBound", {lowerBound: lowerBound})
-      .andWhere("post.original_price <= :upperBound", {upperBound: upperBound})
+      .where("CASE WHEN post.altered_price = -1 THEN post.original_price ELSE post.altered_price END >= :lowerBound", { lowerBound: lowerBound })
+      .andWhere("CASE WHEN post.altered_price = -1 THEN post.original_price ELSE post.altered_price END <= :upperBound", { upperBound: upperBound })
       .andWhere("post.archive = false")
       .getMany();
   }
@@ -123,7 +123,7 @@ export class PostRepository extends AbstractRepository<PostModel> {
     return await this.repository.save(post)
   }
 
-  public async editPostPrice(post: PostModel, new_price: number) : Promise<PostModel> {
+  public async editPostPrice(post: PostModel, new_price: number): Promise<PostModel> {
     post.altered_price = new_price
     return await this.repository.save(post)
   }

diff --git a/src/services/UserService.ts b/src/services/UserService.ts
@@ -18,7 +18,7 @@ export class UserService {
   }
 
   public async getAllUsers(user: UserModel): Promise<UserModel[]> {
-    // if (!user.admin) throw new UnauthorizedError('User does not have permission to get all users')
+    if (!user.admin) throw new UnauthorizedError('User does not have permission to get all users')
     return this.transactions.readOnly(async (transactionalEntityManager) => {
       const userRepository = Repositories.user(transactionalEntityManager);
       return userRepository.getAllUsers();