@nkhang1902/strapi-plugin-export-import-clsx 1.1.15 → 1.1.16

package/admin/src/components/ExportImportButtons/index.jsx

@@ -1,11 +1,5 @@
 import { useState, useRef } from "react";
 import {
-  ModalLayout,
-  ModalHeader,
-  ModalBody,
-  ModalFooter,
-  Typography,
-  Box,
   Button,
 } from "@strapi/design-system";
 import { Download, Upload } from "@strapi/icons";
@@ -14,9 +8,6 @@ import { useNotification } from "@strapi/strapi/admin";
 const ExportImportButtons = (props) => {
   const [isExporting, setIsExporting] = useState(false);
   const [isImporting, setIsImporting] = useState(false);
-  const [showErrorModal, setShowErrorModal] = useState(false);
-  const [importErrors, setImportErrors] = useState([]);
-
   const { toggleNotification } = useNotification();
 
   // Get current content type from props or URL
@@ -188,7 +179,7 @@ const ExportImportButtons = (props) => {
 
     const eventFilter = getEventFilter();
     try {
-      const response = await fetch(`/export-import-clsx/import?eventId=${eventFilter.eventId}`, {
+      const response = await fetch(`/export-import-clsx/import?eventId=${eventFilter?.eventId || ""}`, {
         method: "POST",
         body: formData,
       });
@@ -199,10 +190,14 @@ const ExportImportButtons = (props) => {
         const updated = result.summary?.updated || result.result.updated || 0;
 
         const errorList = result.result?.errors || [];
+        const total = created + updated;
 
         if (errorList.length > 0) {
-          setImportErrors(errorList);
-          setShowErrorModal(true);
+          showLongNotification(toggleNotification, {
+            title: "Import errors",
+            message: errorList.join("\n"),
+            type: "danger",
+          })
         } else if (total > 0) {
           toggleNotification({
             type: "success",
@@ -238,7 +233,6 @@ const ExportImportButtons = (props) => {
   const fileInputRef = useRef(null);
 
   return (
-    <>
     <div
       style={{
         display: "flex",
@@ -276,39 +270,6 @@ const ExportImportButtons = (props) => {
         Import
       </Button>
     </div>
-    {showErrorModal && (
-      <ModalLayout
-        onClose={() => setShowErrorModal(false)}
-        labelledBy="import-errors-title"
-      >
-        <ModalHeader>
-          <Typography id="import-errors-title" fontWeight="bold">
-            Import Errors ({importErrors.length})
-          </Typography>
-        </ModalHeader>
-
-        <ModalBody>
-          <Box padding={4}>
-            {importErrors.map((err, index) => (
-              <Box key={index} paddingBottom={2}>
-                <Typography textColor="danger600">
-                  {index + 1}. {err}
-                </Typography>
-              </Box>
-            ))}
-          </Box>
-        </ModalBody>
-
-        <ModalFooter
-          endActions={
-            <Button onClick={() => setShowErrorModal(false)}>
-              Close
-            </Button>
-          }
-        />
-      </ModalLayout>
-    )}
-    </>
   );
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nkhang1902/strapi-plugin-export-import-clsx",
-  "version": "1.1.15",
+  "version": "1.1.16",
   "description": "A powerful Strapi plugin for exporting and importing data with Excel support and advanced filtering",
   "main": "./strapi-server.js",
   "scripts": {

package/server/controllers/import-controller.js

@@ -15,6 +15,9 @@ module.exports = ({ strapi }) => ({
         .service("import-service");
 
       const { eventId } = ctx.request.query;
+      if (eventId == "") {
+        eventId = null;
+      }
 
       const result = await importService.importData(file, targetContentType, eventId);
 

package/server/services/import-service.js

@@ -147,7 +147,7 @@ module.exports = ({ strapi }) => ({
       .map(([fieldName, attr]) => toCamel(fieldName));
   },
 
-  async handleRelations(entry, contentType, eventId) {
+  async handleRelations(entry, contentType, row) {
     const resolveRelationValue = async (field, value, target) => {
       const targetAttr = strapi.contentTypes[target].attributes;
       for (const field of SHORTCUT_FIELDS) {
@@ -156,7 +156,7 @@ module.exports = ({ strapi }) => ({
           filters: { [field]: { $eq: value } },
         });
         if (existing) return { id: existing.id };
-        throw new Error(`Data with ${field} ${value} not found. Cannot map to ${contentType}`);
+        throw new Error(`Not found.`);
       }
       return null;
     };
@@ -181,11 +181,7 @@ module.exports = ({ strapi }) => ({
         if (relation === "manyToMany" || relation === "oneToMany") {
           updatedEntry[field] = [];
         } else {
-          if (field === 'event') {
-            updatedEntry[field] = { documentId: eventId };
-          } else {
-            updatedEntry[field] = null;
-          }
+          updatedEntry[field] = null;
         }
         continue;
       }
@@ -215,7 +211,7 @@ module.exports = ({ strapi }) => ({
         updatedEntry[field] = Array.isArray(value) ? processed : processed[0];
       } catch (err) {
         throw new Error(
-          `Failed processing field ${field} with value ${JSON.stringify(value)}: ${err.message}`
+          `Error resolving field '${field}' with value '${JSON.stringify(value)}': ${err.message}`
         );
       }
     }
@@ -342,10 +338,15 @@ module.exports = ({ strapi }) => ({
         const f = parseFloat(value);
         return Number.isNaN(f) ? 0 : f;
       }
-      case 'date':
+      case 'date': {
+        const d = new Date(value);
+        if (isNaN(d.getTime())) return null;
+        return d.toISOString().slice(0, 10); // YYYY-MM-DD ONLY
+      }
       case 'datetime': {
         const d = new Date(value);
-        return isNaN(d.getTime()) ? null : d.toISOString();
+        if (isNaN(d.getTime())) return null;
+        return d.toISOString(); // full ISO is valid here
       }
       default:
         return value;
@@ -389,6 +390,9 @@ module.exports = ({ strapi }) => ({
 
   async importEntries(entries, contentType, eventId) {
     const results = { created: 0, updated: 0, errors: [] };
+    const event = eventId ? strapi.documents("api::event.event").findFirst({
+      filters: { documentId: eventId },
+    }) : null;
 
     await strapi.db.transaction(async ({ trx, rollback, onRollback }) => {
       onRollback(() => {
@@ -416,9 +420,10 @@ module.exports = ({ strapi }) => ({
               throw new Error(`Document with id ${id} not found`);
             }
           }
+          data["event"] = event ? event.name : "";
 
           // Handle relations & components
-          data = await this.handleRelations(data, contentType, eventId);
+          data = await this.handleRelations(data, contentType, row, eventName);
           data = await this.handleComponents(data, existing, contentType);
           const sanitizeErrors = [];
           data = this.sanitizeEntryBeforeWrite(data, contentType, '', sanitizeErrors);
@@ -446,9 +451,7 @@ module.exports = ({ strapi }) => ({
           }
         } catch (err) {
           results.errors.push(
-            `Failed ${existing ? "updating" : "creating"} on row ${
-              i + 2
-            }: ${err.message}`
+            `Row ${i + 2}: ${err.message || err.toString()}`
           );
           results.created = 0;
           results.updated = 0;