fix(FavoriteButton): pass tool path to favorite button (CorentinTh#1368)

gitmotion · Oct 25, 2024 · ae13639 · ae13639
1 parent c7b80fb
commit ae13639
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/layouts/tool.layout.vue b/src/layouts/tool.layout.vue
@@ -40,7 +40,7 @@ const toolDescription = computed<string>(() => t(`tools.${i18nKey.value}.descrip
           </n-h1>
 
           <div>
-            <FavoriteButton :tool="{ name: route.meta.name } as Tool" />
+            <FavoriteButton :tool="{ name: route.meta.name, path: route.path } as Tool" />
           </div>
         </div>
 

diff --git a/src/tools/tools.store.ts b/src/tools/tools.store.ts
@@ -45,7 +45,10 @@ export const useToolStore = defineStore('tools', () => {
     newTools: computed(() => tools.value.filter(({ isNew }) => isNew)),
 
     addToolToFavorites({ tool }: { tool: MaybeRef<Tool> }) {
-      favoriteToolsName.value.push(get(tool).path);
+      const toolPath = get(tool).path;
+      if (toolPath) {
+        favoriteToolsName.value.push(toolPath);
+      }
     },
 
     removeToolFromFavorites({ tool }: { tool: MaybeRef<Tool> }) {
-Original file line number
+Diff line change
@@ Expand Up @@
               </n-h1>
               <div>
-                <FavoriteButton :tool="{ name: route.meta.name } as Tool" />
+                <FavoriteButton :tool="{ name: route.meta.name, path: route.path } as Tool" />
               </div>
             </div>
@@ Expand Down @@