From e785e838dd2749f19eb4f53296e6c2f6ca732c55 Mon Sep 17 00:00:00 2001
From: MuslemRahimi <moslem_rahimi@hotmail.de>
Date: Tue, 29 Oct 2024 21:50:43 +0100
Subject: [PATCH] bugfixing

---
 src/routes/stocks/[tickerID]/+layout.svelte   |  5 +-
 src/routes/stocks/[tickerID]/+layout.ts       |  6 +--
 .../[tickerID]/dividends/+layout.svelte       |  6 ++-
 .../stocks/[tickerID]/financials/+page.svelte |  7 +--
 .../stocks/[tickerID]/insider/+layout.svelte  |  6 ++-
 .../stocks/[tickerID]/insider/+page.svelte    | 49 ++++++++-----------
 6 files changed, 34 insertions(+), 45 deletions(-)

diff --git a/src/routes/stocks/[tickerID]/+layout.svelte b/src/routes/stocks/[tickerID]/+layout.svelte
index e78acc2a..d902bf35 100644
--- a/src/routes/stocks/[tickerID]/+layout.svelte
+++ b/src/routes/stocks/[tickerID]/+layout.svelte
@@ -22,7 +22,6 @@
   import { onMount, onDestroy, afterUpdate } from "svelte";
   import { page } from "$app/stores";
   import toast from "svelte-french-toast";
-  import Sidecard from "$lib/components/Sidecard.svelte";
   import Markethour from "$lib/components/Markethour.svelte";
   import AIScore from "$lib/components/AIScore.svelte";
 
@@ -53,7 +52,7 @@
   let y;
 
   let userWatchList = data?.getUserWatchlist ?? [];
-  let isTickerIncluded;
+  let isTickerIncluded = false;
   //let userPortfolio = data?.getUserPortfolio ?? [];
   //let holdingShares = 0;
   //let availableCash = 0;
@@ -315,7 +314,7 @@ function handleTypeOfTrade(state:string)
   }
 
   $: {
-    if ($page.url.pathname) {
+    if ($page?.url?.pathname && typeof window !== "undefined") {
       const parts = $page?.url?.pathname?.split("/");
       const sectionMap = {
         statistics: "statistics",
diff --git a/src/routes/stocks/[tickerID]/+layout.ts b/src/routes/stocks/[tickerID]/+layout.ts
index 7d937346..933902b0 100644
--- a/src/routes/stocks/[tickerID]/+layout.ts
+++ b/src/routes/stocks/[tickerID]/+layout.ts
@@ -53,8 +53,8 @@ const checkMarketHour = async () => {
 };
 
 export const load = async ({ params, data }) => {
-  stockTicker.update((value) => params.tickerID?.toUpperCase());
-  assetType.update((value) => "stock");
-  displayCompanyName.update((value) => data?.companyName);
+  stockTicker.update((value) => (value = params.tickerID?.toUpperCase()));
+  assetType.update((value) => (value = "stock"));
+  displayCompanyName.update((value) => (value = data?.companyName));
   await checkMarketHour();
 };
diff --git a/src/routes/stocks/[tickerID]/dividends/+layout.svelte b/src/routes/stocks/[tickerID]/dividends/+layout.svelte
index f5d261be..52e300d1 100644
--- a/src/routes/stocks/[tickerID]/dividends/+layout.svelte
+++ b/src/routes/stocks/[tickerID]/dividends/+layout.svelte
@@ -4,6 +4,8 @@
 
   export let data;
 
+  let newsList = data?.getNews ?? [];
+
   const formatDate = (dateString) => {
     // Create a date object for the input dateString
     const inputDate = new Date(dateString);
@@ -64,7 +66,7 @@
             </div>
           {/if}
 
-          {#if data?.getNews?.length !== 0}
+          {#if newsList?.length !== 0}
             <div
               class="w-full bg-[#27272A] duration-100 ease-out sm:hover:text-white text-gray-400 sm:hover:border-gray-700 border border-gray-800 rounded-md h-fit pb-4 mt-4 cursor-pointer"
             >
@@ -73,7 +75,7 @@
                   {$stockTicker} News
                 </h3>
                 <ul class="text-gray-200">
-                  {#each data?.getNews?.slice(0, 10) as item}
+                  {#each newsList?.slice(0, 10) as item}
                     <li class="mb-3 last:mb-1">
                       {formatDate(item?.publishedDate)} ago -
                       <a
diff --git a/src/routes/stocks/[tickerID]/financials/+page.svelte b/src/routes/stocks/[tickerID]/financials/+page.svelte
index 3c4aa914..c5718516 100644
--- a/src/routes/stocks/[tickerID]/financials/+page.svelte
+++ b/src/routes/stocks/[tickerID]/financials/+page.svelte
@@ -15,12 +15,11 @@
   import { LineChart, BarChart } from "echarts/charts";
   import { GridComponent, TooltipComponent } from "echarts/components";
   import { CanvasRenderer } from "echarts/renderers";
-  import { onMount } from "svelte";
   use([LineChart, BarChart, GridComponent, TooltipComponent, CanvasRenderer]);
 
   export let data;
 
-  let isLoaded = false;
+  let isLoaded = true;
   let optionsData;
   let tableList = [];
 
@@ -32,10 +31,6 @@
   $coolMode = false;
   let timeFrame = "10Y";
 
-  onMount(async () => {
-    isLoaded = true;
-  });
-
   const statementConfig = [
     {
       propertyName: "revenue",
diff --git a/src/routes/stocks/[tickerID]/insider/+layout.svelte b/src/routes/stocks/[tickerID]/insider/+layout.svelte
index 4041cdb2..797a2c04 100644
--- a/src/routes/stocks/[tickerID]/insider/+layout.svelte
+++ b/src/routes/stocks/[tickerID]/insider/+layout.svelte
@@ -6,6 +6,8 @@
 
   export let data;
 
+  let newsList = data?.getNews ?? [];
+
   const formatDate = (dateString) => {
     // Create a date object for the input dateString
     const inputDate = new Date(dateString);
@@ -161,7 +163,7 @@
             </div>
           {/if}
 
-          {#if data?.getNews?.length !== 0}
+          {#if newsList?.length !== 0}
             <div
               class="w-full bg-[#27272A] duration-100 ease-out sm:hover:text-white text-gray-400 sm:hover:border-gray-700 border border-gray-800 rounded-md h-fit pb-4 mt-4 cursor-pointer"
             >
@@ -170,7 +172,7 @@
                   {$stockTicker} News
                 </h3>
                 <ul class="text-gray-200">
-                  {#each data?.getNews?.slice(0, 10) as item}
+                  {#each newsList?.slice(0, 10) as item}
                     <li class="mb-3 last:mb-1">
                       {formatDate(item?.publishedDate)} ago -
                       <a
diff --git a/src/routes/stocks/[tickerID]/insider/+page.svelte b/src/routes/stocks/[tickerID]/insider/+page.svelte
index 0f9b64fb..38b67359 100644
--- a/src/routes/stocks/[tickerID]/insider/+page.svelte
+++ b/src/routes/stocks/[tickerID]/insider/+page.svelte
@@ -12,14 +12,32 @@
   import TableHeader from "$lib/components/Table/TableHeader.svelte";
 
   export let data;
-  let isLoaded = false;
+  let isLoaded = true;
+
+  let rawData = data?.getInsiderTrading?.sort(
+    (a, b) => new Date(b?.transactionDate) - new Date(a?.transactionDate),
+  );
+
+  let insiderTradingList = rawData?.slice(0, 50);
+  function backToTop() {
+    window.scrollTo({
+      top: 0,
+    });
+  }
 
   let statistics = {};
   let buySellRatio = 0;
   const now = new Date();
   let year = now.getFullYear();
   let quarter = Math.floor(now.getMonth() / 3) + 1;
-  let yearRange = [];
+  let yearRange = Array.from(
+    new Set(
+      rawData?.map((item) => new Date(item?.transactionDate)?.getFullYear()),
+    ),
+  )?.sort((a, b) => b - a);
+  if (yearRange?.length > 0) {
+    year = yearRange?.slice(0)?.at(0);
+  }
 
   function calculateInsiderTradingStatistics(data, year, quarter) {
     // Helper function to check if the transaction date is within the current quarter
@@ -66,17 +84,6 @@
     return statistics;
   }
 
-  let rawData = data?.getInsiderTrading?.sort(
-    (a, b) => new Date(b?.transactionDate) - new Date(a?.transactionDate),
-  );
-
-  let insiderTradingList = rawData?.slice(0, 50);
-  function backToTop() {
-    window.scrollTo({
-      top: 0,
-    });
-  }
-
   function extractOfficeInfo(inputString) {
     const indexOfficer = inputString?.toLowerCase()?.indexOf("officer:");
     const indexOther = inputString?.toLowerCase()?.indexOf("other:");
@@ -104,22 +111,6 @@
   }
 
   onMount(() => {
-    statistics = calculateInsiderTradingStatistics(rawData, year, quarter);
-    buySellRatio =
-      statistics?.soldShares !== 0
-        ? statistics?.buyShares / statistics?.soldShares
-        : 0;
-
-    yearRange = Array.from(
-      new Set(
-        rawData?.map((item) => new Date(item?.transactionDate)?.getFullYear()),
-      ),
-    )?.sort((a, b) => b - a);
-    if (yearRange?.length > 0) {
-      year = yearRange?.slice(0)?.at(0);
-    }
-
-    isLoaded = true;
     window.addEventListener("scroll", handleScroll);
     return () => {
       window.removeEventListener("scroll", handleScroll);