fix: problems not getting readded when the dateselect is updated

Author: zubyj

src/problems-by-company/company.ts

@@ -1,3 +1,5 @@
+// Create a new variable to store all solutions
+const allSolutions = [] as { id: number, title: string, url: string }[];
 const solutions = [] as { id: number, title: string, url: string }[];
 
 let companyName = 'Amazon';
@@ -72,6 +74,25 @@ function addNavbarLinks() {
     });
 }
 
+interface Company {
+    name: string;
+}
+
+interface Question {
+    title: string;
+    frontend_id: number;
+    companies?: Company[];
+}
+
+interface LeetcodeProblems {
+    questions: Question[];
+}
+
+let minFrequency = Number.MAX_SAFE_INTEGER;
+let maxFrequency = 0;
+
+
+
 async function updateFrequency(selectedFrequency: string) {
     // Clear the existing table
     const table = document.getElementById('solutionTable') as HTMLTableElement;
@@ -83,6 +104,9 @@ async function updateFrequency(selectedFrequency: string) {
     minFrequency = Number.MAX_SAFE_INTEGER;
     maxFrequency = 0;
 
+    // Create a new array to hold the solutions with the selected frequency
+    const updatedSolutions = [];
+
     // Update the frequency values in the solutions array
     const data = await new Promise<{ companyProblems: any }>((resolve) => {
         chrome.storage.local.get('companyProblems', function (data) {
@@ -92,7 +116,7 @@ async function updateFrequency(selectedFrequency: string) {
 
     const companyProblems = data.companyProblems[companyName];
     if (Array.isArray(companyProblems)) {
-        solutions.forEach((solution, index) => {
+        allSolutions.forEach((solution, index) => { // Iterate over allSolutions
             // Check if the selected frequency value exists for the problem
             if (companyProblems[index].hasOwnProperty(selectedFrequency)) {
                 const freqValue = companyProblems[index][selectedFrequency];
@@ -101,32 +125,22 @@ async function updateFrequency(selectedFrequency: string) {
                 // Update min and max frequency for the selected range
                 if (freqValue < minFrequency) minFrequency = freqValue;
                 if (freqValue > maxFrequency) maxFrequency = freqValue;
+
+                // Add the solution to the updated solutions array
+                updatedSolutions.push(solution);
             }
         });
     }
 
+    // Replace the solutions array with the updated solutions
+    solutions.length = 0;
+    solutions.push(...updatedSolutions);
+
     // Rebuild the table with updated frequency values
     rebuildTable();
 }
 
 
-interface Company {
-    name: string;
-}
-
-interface Question {
-    title: string;
-    frontend_id: number;
-    companies?: Company[];
-}
-
-interface LeetcodeProblems {
-    questions: Question[];
-}
-
-let minFrequency = Number.MAX_SAFE_INTEGER;
-let maxFrequency = 0;
-
 function addCompanyProblems(sortMethod: string) {
     chrome.storage.local.get(['companyProblems', 'leetcodeProblems'], function (data) {
         const companyProblems = data.companyProblems[companyName];
@@ -139,14 +153,15 @@ function addCompanyProblems(sortMethod: string) {
         // Check if companyProblems is an array before proceeding
         if (Array.isArray(companyProblems)) {
             companyProblems.forEach((problem) => {
-                const correspondingLeetcodeProblem = leetcodeProblems.find(q => q.frontend_id === problem.id);
-                solutions.push({
+                const correspondingLeetcodeProblem = leetcodeProblems.find(q => q.frontend_id === problem.id); // Find the corresponding problem
+                // Populate allSolutions instead of solutions
+                allSolutions.push({
                     id: problem.id,
                     title: problem.title,
                     url: `https://leetcode.com/problems/${problem.title.replace(/\s/g, '-')}/`,
                     frequency: problem.freq_alltime,
-                    difficulty: correspondingLeetcodeProblem?.difficulty_lvl, // Add difficulty
-                    acceptance: correspondingLeetcodeProblem?.acceptance, // Add acceptance
+                    difficulty: correspondingLeetcodeProblem?.difficulty_lvl, // Use the defined variable
+                    acceptance: correspondingLeetcodeProblem?.acceptance, // Use the defined variable
                 });
 
                 // Update min and max frequency
@@ -155,64 +170,17 @@ function addCompanyProblems(sortMethod: string) {
             });
         }
 
+        // Initialize solutions with all problems initially
+        solutions.length = 0;
+        solutions.push(...allSolutions);
+
         console.log(solutions);
 
         // Rebuild the table with sorted solutions
         rebuildTable();
-
     });
 }
 
-async function addCompaniesToSelect() {
-    const companySearch = document.getElementById('companySearch') as HTMLInputElement;
-    const companyList = document.getElementById('companyList') as HTMLDataListElement;
-    let companies = [];
-
-    const data = await new Promise<{ companyProblems: any }>((resolve) => {
-        chrome.storage.local.get('companyProblems', function (data) {
-            resolve(data);
-        });
-    });
-
-    const companyProblems = data.companyProblems;
-    // add all the keys to the set
-    Object.keys(companyProblems).forEach((company) => {
-        if (company) {
-            console.log(company);
-            companies.push(company);
-        }
-    });
-
-    // Event when the "Enter" key is pressed or an option is selected from the dropdown
-    const handleSelection = () => {
-        const inputValue = companySearch.value;
-        // Find the selected company in a case-insensitive manner
-        const selectedCompany = Array.from(companies).find(
-            (company) => company.toLowerCase() === inputValue.toLowerCase()
-        );
-        if (selectedCompany) {
-            chrome.storage.local.set({ clickedCompany: selectedCompany }, () => {
-                location.reload();
-            });
-        }
-    };
-
-    companySearch.addEventListener('keydown', (event) => {
-        if (event.key === 'Enter') {
-            handleSelection();
-        }
-    });
-
-    companySearch.addEventListener('change', handleSelection);
-
-    const sortedCompanies = companies.sort();
-
-    sortedCompanies.forEach((company) => {
-        const option = document.createElement('option');
-        option.value = company;
-        companyList.appendChild(option);
-    });
-}
 
 // Function to rebuild the table with sorted solutions
 function rebuildTable() {
@@ -312,7 +280,6 @@ function sortBy(column: string) {
 
     // Rebuild the table with sorted solutions
     rebuildTable();
-
 }
 
 /* Run the script */