🚑 divide by zero

COS301-SE-2023 · Jul 31, 2023 · c1cd5db · c1cd5db
1 parent 15bedfe
commit c1cd5db
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/backend/src/main/java/fellowship/mealmaestro/models/SettingsModel.java b/backend/src/main/java/fellowship/mealmaestro/models/SettingsModel.java
@@ -146,7 +146,11 @@ public int getUserBMI() {
     }
 
     public void setUserBMI(int userHeight, int userWeight) {
-        this.userBMI = userHeight/userWeight;
+        if (userWeight == 0) {
+            this.userBMI = 0;
+        } else {
+            this.userBMI = userHeight/userWeight;
+        }
     }
 
     public void setUserBMI(int userBMI) {

diff --git a/frontend/src/app/pages/profile/profile.page.spec.ts b/frontend/src/app/pages/profile/profile.page.spec.ts
@@ -5,6 +5,8 @@ import { ProfilePage } from './profile.page';
 import { HttpClientTestingModule } from '@angular/common/http/testing';
 import { RouterTestingModule } from '@angular/router/testing';
 import { CUSTOM_ELEMENTS_SCHEMA } from '@angular/core';
+import { AuthenticationService } from '../../services/services';
+import { UserI } from '../../models/interfaces';
 
 
 describe('ProfilePage', () => {

diff --git a/frontend/src/app/pages/profile/profile.page.ts b/frontend/src/app/pages/profile/profile.page.ts
@@ -189,6 +189,7 @@ export class ProfilePage implements OnInit {
 
 
   private updateSettingsOnServer() {
+    // console.log(this.userpreferences);
     this.settingsApiService.updateSettings(this.userpreferences).subscribe(
       (response) => {
         if (response.status === 200) {