fix: suggestions about separation

Author: kelvinsb

src/decorators/UserDecorator/index.ts

@@ -41,4 +41,4 @@ async function isRightSessionRole(
   return false;
 }
 
-export { canActivate, isRightSessionRole };
+export { canActivate };

src/guards/utils.ts

@@ -16,7 +16,7 @@ import { ShelterService } from './shelter.service';
 import { ServerResponse } from '../utils';
 import { StaffGuard } from '@/guards/staff.guard';
 import { ApplyUser } from '@/guards/apply-user.guard';
-import { UserDecorator } from '@/decorators/UserDecorator';
+import { UserDecorator } from '@/decorators/UserDecorator/user.decorator';
 
 @ApiTags('Abrigos')
 @Controller('shelters')
@@ -40,7 +40,9 @@ export class ShelterController {
   @UseGuards(ApplyUser)
   async show(@UserDecorator() user: any, @Param('id') id: string) {
     try {
-      const data = await this.shelterService.show(id, user);
+      const isLogged =
+        Boolean(user) && Boolean(user?.sessionId) && Boolean(user?.userId);
+      const data = await this.shelterService.show(id, isLogged);
       return new ServerResponse(200, 'Successfully get shelter', data);
     } catch (err: any) {
       this.logger.error(`Failed to get shelter: ${err}`);

src/shelter/shelter.controller.ts

@@ -60,10 +60,7 @@ export class ShelterService {
     });
   }
 
-  async show(id: string, user: any) {
-    const isLogged =
-      Boolean(user) && Boolean(user?.sessionId) && Boolean(user?.userId);
-
+  async show(id: string, shouldShowContact: boolean) {
     const data = await this.prismaService.shelter.findFirst({
       where: {
         id,
@@ -75,7 +72,7 @@ export class ShelterService {
         pix: true,
         shelteredPeople: true,
         capacity: true,
-        contact: isLogged,
+        contact: shouldShowContact,
         petFriendly: true,
         prioritySum: true,
         latitude: true,