proje-react-panel 1.11.0-beta.0 → 1.11.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "proje-react-panel",
-  "version": "1.11.0-beta.0",
+  "version": "1.11.0",
   "type": "module",
   "description": "",
   "author": "SEFA DEMİR",
@@ -73,7 +73,7 @@
     "axios": ">=1.0.0",
     "react": ">=19.0.0",
     "react-hook-form": ">=7.54.2",
-    "react-router": "7.3.0",
+    "react-router": "^7.3.0",
     "react-select": "^5.10.1",
     "use-sync-external-store": ">=1.4.0",
     "zustand": ">=5.0.3"

package/src/__tests__/components/list/LinkCell.test.tsx

@@ -0,0 +1,70 @@
+/**
+ * @jest-environment jsdom
+ */
+import { TextDecoder, TextEncoder } from 'util';
+// react-router reads TextEncoder at import time and jsdom has none — set it up before the
+// imports below pull the router in.
+Object.assign(globalThis, { TextEncoder, TextDecoder });
+
+import 'reflect-metadata';
+import React from 'react';
+import { describe, expect, it } from '@jest/globals';
+import { render } from '@testing-library/react';
+import { MemoryRouter } from 'react-router';
+import { LinkCell } from '../../../components/list/cells/LinkCell';
+import { CellConfiguration } from '../../../decorators/list/Cell';
+
+// `@LinkCell({ path: '/users/:id' })` satirin verisiyle dolmalı. Ikame bozulursa
+// hucredeki her open/edit linki gorunuste calisir ama tiklaninca 404 verir —
+// tarayicida fark edilmesi zor, o yuzden burada kilitli.
+const hrefOf = (configuration: CellConfiguration, item: Record<string, unknown>) =>
+  render(
+    <MemoryRouter>
+      <table>
+        <tbody>
+          <tr>
+            <td>
+              <LinkCell configuration={configuration} item={item} />
+            </td>
+          </tr>
+        </tbody>
+      </table>
+    </MemoryRouter>
+  )
+    .container.querySelector('a')
+    ?.getAttribute('href');
+
+const linkCell = (over: Partial<CellConfiguration> = {}): CellConfiguration =>
+  ({
+    name: 'open',
+    type: 'link',
+    placeHolder: 'open',
+    ...over,
+  }) as CellConfiguration;
+
+describe('LinkCell path parameters', () => {
+  it('substitutes a single :param from the row', () => {
+    expect(hrefOf(linkCell({ path: '/users/:id' } as never), { id: 42 })).toBe(
+      '/users/42'
+    );
+  });
+
+  it('substitutes every :param in the path', () => {
+    expect(
+      hrefOf(linkCell({ path: '/workspaces/:workspaceId/users/:id' } as never), {
+        workspaceId: 'ws-1',
+        id: 7,
+      })
+    ).toBe('/workspaces/ws-1/users/7');
+  });
+
+  it('leaves an unmatched :param in place instead of writing undefined', () => {
+    expect(hrefOf(linkCell({ path: '/users/:missing' } as never), { id: 1 })).toBe(
+      '/users/:missing'
+    );
+  });
+
+  it('passes a path with no parameters through untouched', () => {
+    expect(hrefOf(linkCell({ path: '/users' } as never), { id: 1 })).toBe('/users');
+  });
+});

package/src/api/CrudApi.ts

@@ -64,7 +64,7 @@ export function update<T>(endpoint: string, key = 'id'): OnSubmitFN<T> {
 export function updateFormData<T>(endpoint: string, key = 'id'): OnSubmitFN<T> {
   return async (data: T | FormData): Promise<T> => {
     const axiosInstance = getAxiosInstance();
-    const id = (data as any)[key];
+    const id = (data as Record<string, unknown>)[key];
     const response = await axiosInstance.put<T>(`/${endpoint}/${id}`, data, {
       headers: {
         'Content-Type': 'multipart/form-data',
@@ -89,7 +89,7 @@ export function remove<T>(endpoint: string, key = 'id'): (data: T) => Promise<vo
     const id = (data as any)[key];
     await axiosInstance
       .delete<T>(`/${endpoint}/${id}`)
-      .then((res: any) => res.data)
+      .then(res => res.data)
       .catch((err: AxiosError) => {
         const messageError = err.response?.data as { message: string };
         if (messageError?.message) {

package/src/components/list/cells/LinkCell.tsx

@@ -8,12 +8,32 @@ interface LinkCellProps<T> {
   configuration: CellConfiguration;
 }
 
+/**
+ * `@LinkCell({ path: '/users/:id' })` yazan model sahibi, satirin `id`'siyle dolmus
+ * bir adres bekliyor — router path'i zaten oyle okunuyor. Ikame edilmezse `:id`
+ * harfi harfine adrese giriyor ve hucredeki her open/edit linki 404'e cikiyor:
+ * gorunuste calisan, tiklaninca bozuk bir link. Eslesmeyen bir parametre oldugu
+ * gibi birakilir (silinmez) — boylece eksik alan gorunur kalir, sessizce
+ * `/users/undefined` uretilmez.
+ */
+function resolvePath<T>(path: string, item: T): string {
+  return path.replace(/:(\w+)/g, (match, key: string) => {
+    const value = (item as Record<string, unknown>)[key];
+    return value != null ? String(value) : match;
+  });
+}
+
 export function LinkCell<T>({ item, configuration }: LinkCellProps<T>) {
   const linkConfiguration = configuration as LinkCellConfiguration<T>;
   const value = item[configuration.name as keyof T] ?? 'Link';
 
   return (
-    <Link to={linkConfiguration.path ?? linkConfiguration.url ?? ''}>
+    <Link
+      to={resolvePath(
+        linkConfiguration.path ?? linkConfiguration.url ?? '',
+        item
+      )}
+    >
       {linkConfiguration.onClick ? (
         <a
           className="util-cell-link"