From 2ef0c8e9c38ffefd9c7112e84a0bffc43890b603 Mon Sep 17 00:00:00 2001
From: Chris Hunt <chris.hunt@kasmweb.com>
Date: Tue, 10 Jan 2023 12:00:16 +0000
Subject: [PATCH] Fix missing image recursion Fix add app link not being
 highlighted

---
 site/components/header.js       |  2 +-
 site/pages/addapp/[[...app]].js | 10 ++++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/site/components/header.js b/site/components/header.js
index 9fa4a42..c0b19e5 100644
--- a/site/components/header.js
+++ b/site/components/header.js
@@ -47,7 +47,7 @@ export default function Header({ searchText, changeSearch }) {
       </div>
       <nav className='relative z-10 mx-12'>
         <Link href="/" className={'p-4 inline-block rounded-full border border-solid' + (router.pathname == "/" ? ' border-white/30' : ' border-transparent')}>Library</Link>
-        <Link href="/addapp" className={'p-4 inline-block rounded-full border border-solid' + (router.pathname == "/addapp" ? ' bg-black/10 border-white/30' : ' border-transparent')}>Add App</Link>
+        <Link href="/addapp" className={'p-4 inline-block rounded-full border border-solid' + (router.pathname.startsWith("/addapp") ? ' bg-black/10 border-white/30' : ' border-transparent')}>Add App</Link>
       </nav>
       <div className="grow flex justify-center relative z-10">
         <div className='bg-black/10 shadow border border-1 border-white/30 rounded flex w-full max-w-md'>
diff --git a/site/pages/addapp/[[...app]].js b/site/pages/addapp/[[...app]].js
index dbdccf9..9a807f3 100644
--- a/site/pages/addapp/[[...app]].js
+++ b/site/pages/addapp/[[...app]].js
@@ -31,7 +31,7 @@ export async function getStaticProps({ params }) {
   }
 }
 
-export default function AddApp({ app = null }) {
+export default function AddApp({ app }) {
 
   function friendlyUrl(url) {
     // make the url lowercase         
@@ -98,12 +98,14 @@ export default function AddApp({ app = null }) {
   // const { app } = router.query
 
   useEffect(() => {
+    console.log(app)
     if(app === null) {
       description.current.value = ''
       name.current.value = ''
       friendly_name.current.value = ''
       setCategories(null)
       setArchitecture(null)
+      setIcon(null)
       setApplication(defaultState)
     }
     else if (app && app[0]) {
@@ -211,6 +213,7 @@ export default function AddApp({ app = null }) {
         file: event.target.files[0]
       })
       setExt(event.target.value.substr(event.target.value.lastIndexOf('.') + 1))
+      setInlineImage(null)
       // return
     }
 
@@ -257,6 +260,7 @@ export default function AddApp({ app = null }) {
 
             <label className='mb-2 font-medium'>Categories</label>
             <CreatableSelect
+              instanceId="1"
               name="categories"
               isMulti
               options={options}
@@ -276,6 +280,7 @@ export default function AddApp({ app = null }) {
 
             <label className='mb-2 font-medium'>Architecture</label>
             <Select
+              instanceId="2"
               name="architecture"
               isMulti
               options={[
@@ -330,7 +335,8 @@ function App({ app, icon, inlineImage }) {
     <div className={"rounded-xl group w-full shadow max-w-xs relative overflow-hidden h-[100px] border border-solid flex flex-col justify-between bg-slate-300 border-slate-400/50"}>
       <div className={"absolute top-0 left-0 right-0 h-[200px] transition-all" + (showDescription ? ' -translate-y-1/2' : '')}>
         <div onClick={() => setShowDescription(true)} className={"h-[100px] p-4 relative overflow-hidden cursor-pointer"}>
-          <img className="h-[90px] group-hover:scale-150 transition-all absolute left-2 top-1" src={app.image_src} onError={(e) => e.target.src = inlineImage} alt={app.friendly_name} />
+          <img className="h-[90px] group-hover:scale-150 transition-all absolute left-2 top-1" src={app.image_src} onError={(e) => { 
+            if ( inlineImage !== null) { e.target.src = inlineImage }}} alt={app.friendly_name} />
           <div className="flex-col pl-28">
             <div className="font-bold">{app.friendly_name || 'Friendly Name'}</div>
             <div className="text-xs mb-2 flex gap-2">{process.env.name || 'Unknown'} <span>{official()}</span></div>