Feat:Added file icons and fixed download errors

Frontend/src/components/FileList.jsx

@@ -1,25 +1,83 @@
 import React, { useState, useEffect } from "react";
 import PropTypes from "prop-types";
+import { useDispatch } from "react-redux";
+import { setCurrentPath } from "../store/pathSlice";
+import {
+  FileText,
+  FileVideo,
+  FileImage,
+  FileAudio,
+  FileArchive,
+  FileSpreadsheet,
+  FileType2,
+  FileCode2,
+  Presentation,
+  Folder,
+  Download,
+  Trash2,
+  ArrowLeft,
+} from "lucide-react";
 
 const FileTable = ({ initialPath }) => {
-  const [currentPath, setCurrentPath] = useState(initialPath || "/");
+  const [currentPath, setCurrentPathState] = useState(initialPath || "/");
   const [files, setFiles] = useState([]);
+  const dispatch = useDispatch();
 
-  // Helpers to parse entry
   const getType = (entry) =>
     entry.trim().startsWith("📁") ? "Folder" : "File";
+
   const getName = (entry) => entry.trim().replace(/^📁\s*|^📄\s*/, "");
+
   const isFile = (entry) => getType(entry) === "File";
 
-  // Fetch and show only top-level entries (indentation = 0)
+  const getIcon = (name, type) => {
+    if (type === "Folder")
+      return <Folder className="text-yellow-500 w-5 h-5 mr-2" />;
+    const ext = name.split(".").pop().toLowerCase();
+    switch (ext) {
+      case "txt":
+        return <FileText className="text-gray-700 w-5 h-5 mr-2" />;
+      case "mp4":
+      case "mkv":
+        return <FileVideo className="text-purple-500 w-5 h-5 mr-2" />;
+      case "jpg":
+      case "jpeg":
+      case "png":
+      case "gif":
+        return <FileImage className="text-pink-500 w-5 h-5 mr-2" />;
+      case "mp3":
+      case "wav":
+        return <FileAudio className="text-indigo-500 w-5 h-5 mr-2" />;
+      case "zip":
+      case "rar":
+      case "tar":
+      case "gz":
+        return <FileArchive className="text-red-500 w-5 h-5 mr-2" />;
+      case "csv":
+      case "xls":
+      case "xlsx":
+        return <FileSpreadsheet className="text-green-500 w-5 h-5 mr-2" />;
+      case "ppt":
+      case "pptx":
+        return <Presentation className="text-orange-500 w-5 h-5 mr-2" />;
+      case "js":
+      case "html":
+      case "css":
+      case "java":
+      case "py":
+      case "cpp":
+        return <FileCode2 className="text-blue-500 w-5 h-5 mr-2" />;
+      default:
+        return <FileType2 className="text-gray-500 w-5 h-5 mr-2" />;
+    }
+  };
+
   const fetchFiles = async () => {
     try {
       const response = await fetch(
         `http://192.168.29.61:8080/api/hdfs/listFiles?hdfsPath=${currentPath}`
       );
       const data = await response.json();
-
-      // Filter entries: only those without leading spaces
       const filtered = data.filter(
         (entry) => entry.match(/^ */)[0].length === 0
       );
@@ -30,53 +88,130 @@ const FileTable = ({ initialPath }) => {
     }
   };
 
+  const deleteFileOrFolder = async (name, type, event) => {
+    event.stopPropagation(); // Prevent row onClick from firing.
+    try {
+      const hdfsPath =
+        currentPath === "/" ? `/${name}` : `${currentPath}/${name}`;
+      const encodedPath = encodeURIComponent(hdfsPath);
+      let deleteEndpoint = "";
+
+      if (type === "File") {
+        deleteEndpoint = `http://192.168.29.61:8080/api/hdfs/deleteFile?hdfsPath=${encodedPath}`;
+      } else {
+        deleteEndpoint = `http://192.168.29.61:8080/api/hdfs/deleteFolder?hdfsPath=${encodedPath}`;
+      }
+
+      const response = await fetch(deleteEndpoint, { method: "DELETE" });
+
+      if (!response.ok) {
+        const errorText = await response.text();
+        console.error("Deletion failed:", errorText);
+      }
+
+      fetchFiles();
+    } catch (error) {
+      console.error("Failed to delete file/folder:", error);
+    }
+  };
+
   useEffect(() => {
+    dispatch(setCurrentPath(currentPath));
     fetchFiles();
   }, [currentPath]);
 
-  // Navigate into a folder
   const handleOpenFolder = (folderName) => {
     const newPath =
       currentPath === "/" ? `/${folderName}` : `${currentPath}/${folderName}`;
-    setCurrentPath(newPath);
+    setCurrentPathState(newPath);
   };
 
-  // Go up one level
   const goBack = () => {
     if (currentPath === "/") return;
     const parts = currentPath.split("/").filter(Boolean);
     parts.pop();
-    setCurrentPath(parts.length === 0 ? "/" : `/${parts.join("/")}`);
+    setCurrentPathState(parts.length === 0 ? "/" : `/${parts.join("/")}`);
+  };
+
+  const handleFileDownload = async (hdfsPath, name, event) => {
+    event.stopPropagation(); // Prevent row click (if any) for files.
+    try {
+      const formData = new URLSearchParams();
+      formData.append("hdfsPath", hdfsPath);
+      formData.append("username", "kalas");
+
+      const response = await fetch(
+        "http://192.168.29.61:8080/api/hdfs/downloadFile",
+        {
+          method: "POST",
+          headers: {
+            "Content-Type": "application/x-www-form-urlencoded",
+          },
+          body: formData.toString(),
+        }
+      );
+
+      if (!response.ok) throw new Error("Download failed");
+
+      // Extract filename from header OR fallback to name from path
+      const contentDisposition = response.headers.get("Content-Disposition");
+      let fileName = "downloaded_file";
+
+      if (contentDisposition && contentDisposition.includes("filename=")) {
+        const match = contentDisposition.match(
+          /filename[^;=\n]*=((['"]).*?\2|[^;\n]*)/
+        );
+        if (match && match[1]) {
+          fileName = match[1].replace(/['"]/g, "");
+        }
+      } else {
+        // fallback: extract name from path
+        const parts = hdfsPath.split("/");
+        const fallback = parts[parts.length - 1];
+        if (fallback) fileName = fallback;
+      }
+
+      // Create blob and trigger download
+      const blob = await response.blob();
+      const url = window.URL.createObjectURL(blob);
+      const link = document.createElement("a");
+      link.href = url;
+      link.download = fileName;
+      document.body.appendChild(link);
+      link.click();
+      link.remove();
+      window.URL.revokeObjectURL(url);
+    } catch (error) {
+      console.error("Download failed:", error);
+      alert("Something went wrong while downloading the file.");
+    }
   };
 
   return (
-    <div className="relative overflow-x-auto shadow-md rounded-lg">
+    <div className="relative overflow-x-auto rounded-2xl shadow-lg border border-blue-200">
-      <div className="flex items-center justify-between px-6 py-4 bg-blue-100 text-black font-medium">
+      <div className="flex items-center justify-between px-6 py-4 bg-blue-100 text-black font-semibold text-sm">
-        <span>Path: {currentPath}</span>
+        <span className="truncate max-w-[80%]">Path: {currentPath}</span>
         {currentPath !== "/" && (
-          <button onClick={goBack} className="text-blue-600 hover:underline">
+          <button
-            ⬅️ Go Back
+            onClick={goBack}
+            className="flex items-center gap-1 text-blue-600 hover:underline text-sm"
+          >
+            <ArrowLeft className="w-4 h-4" />
+            Go Back
           </button>
         )}
       </div>
       <table className="w-full text-sm text-left text-black">
-        <thead className="text-xs text-black uppercase bg-blue-100">
+        <thead className="text-xs uppercase bg-blue-50 text-blue-800 border-b border-blue-200">
           <tr>
-            <th scope="col" className="px-6 py-3">
+            <th className="px-6 py-3">Name</th>
-              File Name
+            <th className="px-6 py-3">Actions</th>
-            </th>
-            <th scope="col" className="px-6 py-3">
-              Type
-            </th>
-            <th scope="col" className="px-6 py-3">
-              Action
-            </th>
           </tr>
         </thead>
         <tbody>
           {files.length === 0 ? (
             <tr>
-              <td colSpan="3" className="px-6 py-4 text-gray-600">
+              <td colSpan="2" className="px-6 py-4 text-gray-500 text-center">
                 No files found.
               </td>
             </tr>
@@ -84,38 +219,40 @@ const FileTable = ({ initialPath }) => {
             files.map((entry, idx) => {
               const name = getName(entry);
               const type = getType(entry);
-              const encodedPath = encodeURIComponent(`${currentPath}/${name}`);
+              const hdfsPath =
-              const downloadUrl = `http://192.168.29.61:8080/api/hdfs/downloadFile?hdfsPath=${encodedPath}&localPath=E:/testdownload/${name}&kalas=${
+                currentPath === "/" ? `/${name}` : `${currentPath}/${name}`;
-                currentPath.split("/")[1] || "user"
-              }`;
 
               return (
                 <tr
                   key={idx}
-                  className="even:bg-blue-50 odd:bg-white border-b border-blue-200"
+                  onClick={
+                    type === "Folder" ? () => handleOpenFolder(name) : undefined
+                  }
+                  className={`even:bg-blue-50 odd:bg-white border-b border-blue-100 transition hover:bg-blue-100 ${
+                    type === "Folder" ? "cursor-pointer" : ""
+                  }`}
                 >
-                  <td className="px-6 py-4 font-medium whitespace-nowrap">
+                  <td className="px-6 py-4 font-medium flex items-center">
+                    {getIcon(name, type)}
                     {name}
                   </td>
-                  <td className="px-6 py-4">{type}</td>
+                  <td className="px-6 py-4 space-x-3">
-                  <td className="px-6 py-4">
+                    {isFile(entry) && (
-                    {isFile(entry) ? (
-                      <a
-                        href={downloadUrl}
-                        className="font-medium text-blue-600 hover:underline"
-                        target="_blank"
-                        rel="noopener noreferrer"
-                      >
-                        Download
-                      </a>
-                    ) : (
                       <button
-                        onClick={() => handleOpenFolder(name)}
+                        onClick={(e) => handleFileDownload(hdfsPath, name, e)}
-                        className="font-medium text-blue-600 hover:underline"
+                        className="text-blue-600 hover:underline inline-flex items-center"
                       >
-                        Open
+                        <Download className="w-4 h-4 mr-1" />
+                        Download
                       </button>
                     )}
+                    <button
+                      onClick={(e) => deleteFileOrFolder(name, type, e)}
+                      className="text-red-600 hover:underline inline-flex items-center"
+                    >
+                      <Trash2 className="w-4 h-4 mr-1" />
+                      Delete
+                    </button>
                   </td>
                 </tr>
               );