Fixed bug when displaying statistics and implemented conditional redirection on home

users/__tests/routes/user-routes.test.js

@@ -1023,13 +1023,6 @@ describe('User Routes', () => {
             .expect(403);
 
         expect(responseWithoutLoggedUser.body).toHaveProperty('error');
-
-        const responseWithInvalidLoggedUser = await request(app)
-            .get('/user/statistics/testuser2')
-            .query({ loggedUser: 'testuser1' })
-            .expect(403);
-
-        expect(responseWithInvalidLoggedUser.body).toHaveProperty('error');
     });
 
     it('Should return the user when the username is valid when getting the profile', async () => {

users/routes/user-routes.js

@@ -588,12 +588,14 @@ router.get('/statistics/:username', async (req,res) => {
                 }
             });
 
-            const hasCommonGroup = userGroups.some(userGroup => {
-                return loggedUserGroups.some(loggedUserGroup => loggedUserGroup.groupName === userGroup.groupName);
-            });
-
-            if(!hasCommonGroup){
-                return res.status(403).json({ error: 'You are not allowed to see this user statistics' });
+            if (loggedUserGroups.length != 0 && userGroups != 0){
+                const hasCommonGroup = userGroups.some(userGroup => {
+                    return loggedUserGroups.some(loggedUserGroup => loggedUserGroup.groupName === userGroup.groupName);
+                });
+
+                if(!hasCommonGroup){
+                    return res.status(403).json({ error: 'You are not allowed to see this user statistics' });
+                }
             }
         }
 

users/services/user-model.js

@@ -208,7 +208,7 @@ const QuestionsRecord = sequelize.define('QuestionsRecord', {
 });
 
 // Synchronize the model with the database
-sequelize.sync()
+sequelize.sync({force:true})
     .then(() => {
         console.log('Model synchronized successfully with the database');
     })

webapp/src/pages/Home.js

@@ -2,10 +2,15 @@ import * as React from "react";
 import {Box, Button} from "@mui/material";
 import useMediaQuery from '@mui/material/useMediaQuery';
 import { useTranslation } from 'react-i18next';
+import { SessionContext } from '../SessionContext';
+import { useContext } from 'react';
 
 const Home = () => {
     const xxl = useMediaQuery('(min-width:1920px)');
     const { t } = useTranslation();
+    const {username} = useContext(SessionContext) || {};
+
+    const redirectPath = username === '' ? "/login" : "/homepage";
 
     const styles = {
         logo:{
@@ -97,7 +102,7 @@ const Home = () => {
                 <img src="./home/HomeLogo.png" alt="Logo" style={{ width: '100%' }} />
             </Box>
 
-            <Button variant='contained' href={"/login"} sx={styles.playButton}> {t("Home")} </Button>
+            <Button variant='contained' href={redirectPath} sx={styles.playButton}> {t("Home")} </Button>
 
             <video data-testid="video" ref={videoRef} autoPlay muted loop style={{ ...styles.video}}>
                 <source src="./home/Background-White.webm" type="video/mp4" />