[#1959] Missing contribution bar for merged groups after refresh (#1960)

Currently, upon refreshing the page, the contribution bar disappears 
due to getContributionBars returning NaN instead of the expected array. 
This was caused by division by 0, due to the user's 
checkedFileTypeContribution not being initialized properly. 

Let's fix this checkedFileTypeContribution not being initialized 
properly and add guard clauses against division by 0.

frontend/src/components/c-summary-charts.vue

@@ -295,7 +295,11 @@ export default {
       let currentBarWidth = 0;
       const fullBarWidth = 100;
       const contributionPerFullBar = (this.avgContributionSize * 2);
+
       const allFileTypesContributionBars = {};
+      if (contributionPerFullBar === 0) {
+        return allFileTypesContributionBars;
+      }
 
       Object.keys(fileTypeContribution)
         .filter((fileType) => this.checkedFileTypes.includes(fileType))
@@ -318,7 +322,7 @@ export default {
               contributionBars.push(fullBarWidth);
             }
             const remainingBarWidth = barWidth % fullBarWidth;
-            if (remainingBarWidth !== 0) {
+            if (remainingBarWidth > 0) {
               contributionBars.push(remainingBarWidth);
             }
             currentBarWidth = remainingBarWidth;
@@ -349,7 +353,9 @@ export default {
     getContributionBars(totalContribution) {
       const res = [];
       const contributionLimit = (this.avgContributionSize * 2);
-
+      if (contributionLimit === 0) {
+        return res;
+      }
       const cnt = Math.floor(totalContribution / contributionLimit);
       for (let cntId = 0; cntId < cnt; cntId += 1) {
         res.push(100);

frontend/src/types/types.ts

@@ -34,7 +34,7 @@ export function isCommit(commit: Commit | DailyCommit): commit is Commit {
 }
 
 export interface User {
-  checkedFileTypeContribution: number;
+  checkedFileTypeContribution: number | undefined;
   commits?: Commit[];
   dailyCommits: DailyCommit[];
   displayName: string;

frontend/src/utils/api.ts

@@ -269,7 +269,7 @@ window.api = {
           searchPath: searchParams.join('_').toLowerCase(),
           repoName: `${repo.displayName}`,
           location: `${repo.location.location}`,
-          checkedFileTypeContribution: 0,
+          checkedFileTypeContribution: undefined,
         });
 
         res.push(user);

frontend/src/utils/user.ts

@@ -2,7 +2,7 @@ import { Commit, DailyCommit, User as UserType } from '../types/types';
 import { AuthorFileTypeContributions } from '../types/zod/commits-type';
 
 export default class User implements UserType {
-  checkedFileTypeContribution: number;
+  checkedFileTypeContribution: number | undefined;
 
   commits: Commit[];
 
@@ -25,7 +25,7 @@ export default class User implements UserType {
   variance: number;
 
   constructor(userObj: User) {
-    this.checkedFileTypeContribution = userObj.checkedFileTypeContribution || 0;
+    this.checkedFileTypeContribution = userObj.checkedFileTypeContribution;
     this.commits = userObj.commits || [];
     this.dailyCommits = userObj.dailyCommits || [];
     this.displayName = userObj.displayName || '';

frontend/src/views/c-summary.vue

@@ -219,13 +219,16 @@ export default defineComponent({
           if (user.checkedFileTypeContribution === undefined) {
             this.updateCheckedFileTypeContribution(user);
           }
-          if (user.checkedFileTypeContribution > 0) {
+
+          if (user.checkedFileTypeContribution && user.checkedFileTypeContribution > 0) {
             totalCount += 1;
             totalLines += user.checkedFileTypeContribution;
           }
         });
       });
-
+      if (totalCount === 0) {
+        return 0;
+      }
       return totalLines / totalCount;
     },
 
@@ -544,7 +547,7 @@ export default defineComponent({
 
         this.mergeFileTypeContribution(user, mergedFileTypeContribution);
 
-        totalMergedCheckedFileTypeCommits += user.checkedFileTypeContribution;
+        totalMergedCheckedFileTypeCommits += user.checkedFileTypeContribution || 0;
         mergedVariance += user.variance;
       });
       mergedCommits.sort(window.comparator((ele) => ele.date));