Update variable names, functions, and log messages for clarity and fu…

…nctionality

Renamed `orders` to `photos` in `ProductPostMovePhoto` method in `AdminHub` class for clarity. Updated version numbers in comments across multiple files. Added new `refresh` function in `useSWR` function, replacing previous logic. `AdminProductPhotoAction` function now takes `ProductId` as a prop and updates photo list after a photo is moved. `AdminProductPhoto` function maps data to a new `list` variable used as `Items` prop for `DelegateDataGrid` component. Similar mapping done in `ProductCarousel` function. Changed log message in `PhotoList` method from warning to debug and updated parameters for `useSWR` function in `usePhotoList` method.

SoarCraft.AwaiShop/AdminHub/Product/Post.cs

@@ -38,38 +38,35 @@ public async Task<uint> ProductPostCreate(string name) {
      * <remarks>
      * @author Aloento
      * @since 0.5.0
-     * @version 0.1.0
+     * @version 1.0.0
      * </remarks>
      */
     public async Task<bool> ProductPostMovePhoto(uint photoId, bool up) {
-        var orders = await this.Db.Photos
+        var photos = await this.Db.Photos
             .Where(x => x.ProductId == this.Db.Photos
                 .Where(y => y.PhotoId == photoId)
                 .Select(z => z.ProductId)
                 .Single())
             .OrderBy(x => x.Order)
             .ToListAsync();
 
-        var index = orders.FindIndex(x => x.PhotoId == photoId);
-        var current = orders[index].Order;
+        var index = photos.FindIndex(x => x.PhotoId == photoId);
+        var current = photos[index].Order;
 
         if (up) {
             if (current == 1)
                 throw new HubException("Photo already at top");
 
-            orders[index - 1].Order = current;
-            orders[index].Order = (byte)(current - 1);
+            photos[index - 1].Order = current;
+            photos[index].Order = (byte)(current - 1);
         } else {
-            if (current == orders.Last().Order)
+            if (current == photos.Last().Order)
                 throw new HubException("Photo already at bottom");
 
-            orders[index + 1].Order = current;
-            orders[index].Order = (byte)(current + 1);
+            photos[index + 1].Order = current;
+            photos[index].Order = (byte)(current + 1);
         }
 
-        for (byte i = 0; i < orders.Count; i++)
-            orders[i].Order = (byte)(i + 1);
-
         await this.Db.SaveChangesAsync();
         return true;
     }

SoarCraft.AwaiShop/SoarCraft.AwaiShop.csproj

@@ -39,10 +39,10 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
     <PackageReference Include="Microsoft.Extensions.Hosting.Systemd" Version="8.0.0" />
-    <PackageReference Include="Microsoft.Identity.Web" Version="2.17.0" />
+    <PackageReference Include="Microsoft.Identity.Web" Version="2.17.1" />
     <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="8.0.2" />
     <PackageReference Include="SixLabors.ImageSharp" Version="3.1.2" />
-    <PackageReference Include="Z.EntityFramework.Plus.EFCore" Version="8.102.1" />
+    <PackageReference Include="Z.EntityFramework.Plus.EFCore" Version="8.102.1.1" />
   </ItemGroup>
 
   <ItemGroup>

package.json

@@ -18,7 +18,7 @@
     "@azure/msal-browser": "^3.10.0",
     "@azure/msal-common": "^14.7.1",
     "@azure/msal-react": "^2.0.12",
-    "@fluentui/react-components": "^9.46.5",
+    "@fluentui/react-components": "^9.46.6",
     "@fluentui/react-hooks": "^8.6.36",
     "@fluentui/react-icons": "^2.0.230",
     "@griffel/react": "^1.5.20",
@@ -53,7 +53,7 @@
   },
   "devDependencies": {
     "@types/lodash-es": "^4.17.12",
-    "@types/react": "^18.2.58",
+    "@types/react": "^18.2.60",
     "@types/react-dom": "^18.2.19",
     "@vitejs/plugin-react-swc": "^3.6.0",
     "typescript": "^5.3.3",

src/Helpers/useSWR.ts

@@ -4,7 +4,7 @@ import type { Options, Service } from "ahooks/lib/useRequest/src/types";
 /**
  * @author Aloento
  * @since 1.3.5
- * @version 0.3.0
+ * @version 0.4.0
  */
 export function useSWR<TData, TParams extends any[]>(
   key: string,
@@ -27,15 +27,15 @@ export function useSWR<TData, TParams extends any[]>(
     }
   );
 
+  function refresh() {
+    clearCache(key);
+    localStorage.removeItem(key);
+    return req.refreshAsync();
+  }
+
   return {
     ...req,
-    refresh: () => {
-      if (options.useMemory)
-        clearCache(key);
-      else
-        localStorage.removeItem(key);
-
-      req.refresh();
-    }
+    refresh,
+    refreshAsync: refresh,
   };
 }

src/Pages/Admin/Product/Photo/Action.tsx

@@ -2,19 +2,22 @@ import { Button } from "@fluentui/react-components";
 import { ArrowDownRegular, ArrowUpRegular } from "@fluentui/react-icons";
 import { ICompLog } from "~/Helpers/Logger";
 import { useErrorToast } from "~/Helpers/useToast";
+import { Hub } from "~/ShopNet";
 import { AdminHub } from "~/ShopNet/Admin";
 import { IPhotoItem } from ".";
 import { AdminProductPhotoEdit } from "./Edit";
 
 /**
  * @author Aloento
  * @since 1.4.0
- * @version 0.1.0
+ * @version 0.2.0
  */
 export function AdminProductPhotoAction(props: IPhotoItem & ICompLog) {
-  const { Id, ParentLog } = props;
+  const { Id, ProductId, ParentLog } = props;
   const { dispatch } = useErrorToast(ParentLog);
 
+  const { mutate } = Hub.Product.Get.usePhotoList(ProductId, ParentLog);
+
   const { run } = AdminHub.Product.Post.useMovePhoto({
     manual: true,
     onError(e, params) {
@@ -23,7 +26,24 @@ export function AdminProductPhotoAction(props: IPhotoItem & ICompLog) {
         Request: params,
         Error: e
       });
-    }
+    },
+    onSuccess: (_, [__, up]) => mutate(old => {
+      const list = old![0];
+
+      const index = list.findIndex(x => x.PhotoId === Id);
+      if (index === -1)
+        return old;
+
+      const newIndex = up ? index - 1 : index + 1;
+      if (newIndex < 0 || newIndex >= list.length)
+        return old;
+
+      const temp = list[index];
+      list[index] = list[newIndex];
+      list[newIndex] = temp;
+
+      return old;
+    })
   });
 
   return <>

src/Pages/Admin/Product/Photo/index.tsx

@@ -82,7 +82,15 @@ const columns: TableColumnDefinition<IPhotoItem>[] = [
  * @version 1.0.0
  */
 export function AdminProductPhoto({ ProdId }: { ProdId: number }) {
-  const { data: [list] } = Hub.Product.Get.usePhotoList(ProdId, log);
+  const { data } = Hub.Product.Get.usePhotoList(ProdId, log);
+  const list = data
+    ? data[0].map(x => ({
+      Id: x.PhotoId,
+      Cover: x.ObjectId,
+      Caption: x.Caption,
+      ProductId: x.ProductId
+    }))
+    : undefined;
 
   const { dispatch, dispatchToast } = useErrorToast(log);
 
@@ -137,14 +145,7 @@ export function AdminProductPhoto({ ProdId }: { ProdId: number }) {
     </div>
 
     <DelegateDataGrid
-      Items={
-        list?.map(x => ({
-          Id: x.PhotoId,
-          Cover: x.ObjectId,
-          Caption: x.Caption,
-          ProductId: x.ProductId
-        }))
-      }
+      Items={list}
       Columns={columns}
     />
   </>

src/Pages/Product/Carousel.tsx

@@ -33,7 +33,8 @@ export function ProductCarousel({ Id }: { Id: number; }) {
   const style = useStyles();
   const [imgs, setImgs] = useState<[string, string?][]>([[img]]);
 
-  const { data: [list] } = Hub.Product.Get.usePhotoList(Id, log);
+  const { data } = Hub.Product.Get.usePhotoList(Id, log);
+  const list = data ? data[0] : undefined;
 
   useAsyncEffect(async () => {
     if (!list) return;

src/ShopNet/Product/Get.ts

@@ -141,7 +141,7 @@ export abstract class ProductGet extends ProductData {
     list = list.sort((a, b) => a.Order - b.Order);
 
     if (!cover && list.length > 0) {
-      log.warn(`Product ${prodId} has no cover photo, using first photo instead`);
+      log.debug(`Product ${prodId} has no cover photo, using first photo instead`);
       return [list, list[0].ObjectId];
     }
 
@@ -156,10 +156,10 @@ export abstract class ProductGet extends ProductData {
   public static usePhotoList(prodId: number, pLog: Logger) {
     const req = useSWR(
       this.Index(prodId, this.photoList),
-      () => this.PhotoList(prodId, pLog),
+      (id) => this.PhotoList(id, pLog),
       {
         defaultParams: [prodId],
-        onError: pLog.error,
+        onError: pLog.error
       }
     );