From eb2a2d78152c7ca689e3aac116c107229a896a66 Mon Sep 17 00:00:00 2001
From: ulrich <undisclosed>
Date: Sat, 03 Mar 2018 19:34:54 +0000
Subject: [PATCH] Bug mit relativen Pfaden entfernt
---
src/java/de/uhilger/filecms/api/FileMgr.java | 361 +++++++++++++++++++++++++++++----------------------
1 files changed, 206 insertions(+), 155 deletions(-)
diff --git a/src/java/de/uhilger/filecms/api/FileMgr.java b/src/java/de/uhilger/filecms/api/FileMgr.java
index 82c5369..8e63002 100644
--- a/src/java/de/uhilger/filecms/api/FileMgr.java
+++ b/src/java/de/uhilger/filecms/api/FileMgr.java
@@ -18,9 +18,10 @@
package de.uhilger.filecms.api;
+import de.uhilger.filecms.data.Bild;
import de.uhilger.filecms.data.FileRef;
-import de.uhilger.filecms.web.Initialiser;
-import de.uhilger.wbx.Bild;
+import de.uhilger.filecms.pub.AbstractComparator;
+import de.uhilger.filecms.pub.FileNameComparator;
import java.awt.Container;
import java.awt.Image;
import java.awt.MediaTracker;
@@ -35,6 +36,7 @@
import java.io.Reader;
import java.security.Principal;
import java.util.ArrayList;
+import java.util.Arrays;
import java.util.Enumeration;
import java.util.Iterator;
import java.util.List;
@@ -45,8 +47,7 @@
import org.apache.commons.io.FileUtils;
/**
- *
- * @author ulrich
+ * Methoden zur Verwaltung von Dateien
*/
public class FileMgr extends Api {
private static final Logger logger = Logger.getLogger(FileMgr.class.getName());
@@ -70,124 +71,164 @@
* @return
*/
public List<FileRef> list(String relPath) {
+ return listInt(relPath, "name", AbstractComparator.ORDER_ASC);
+ }
+
+ public List<FileRef> listOrdered(String relPath, String orderBy, String order) {
+ return listInt(relPath, orderBy, order);
+ }
+
+ /**
+ * Inhalte der WebBox listen. Hier wird nur der relative Pfad
+ * ausgehend von www oder home ausgegeben sowie zudem ausgehend
+ * von $daten und $basis, sofern der Benutzer die Rolle wbxAdmin hat.
+ *
+ * Andere Inhalte werden nicht ausgegeben.
+ *
+ * @param relPath
+ * @param orderBy 'name'
+ * @param order AbstractComparator.ORDER_ASC oder AbstractComparator.ORDER_DESC
+ * @return
+ */
+ private List<FileRef> listInt(String relPath, String orderBy, String order) {
Bild bild = new Bild();
List<FileRef> files = new ArrayList();
- if(relPath.length() == 0) {
- FileRef namedPublicFolder = new FileRef(PUB_DIR_NAME, true);
- logger.finer(namedPublicFolder.getAbsolutePath());
- FileRef namedHomeFolder = new FileRef(HOME_DIR_NAME, true);
- logger.finer(namedHomeFolder.getAbsolutePath());
- files = new ArrayList();
- files.add(namedHomeFolder);
- files.add(namedPublicFolder);
- if(getRequest().isUserInRole(WBX_ADMIN_ROLE)) {
- FileRef namedBaseFolder = new FileRef(WBX_BASE, true);
- FileRef namedDataFolder = new FileRef(WBX_DATA, true);
- files.add(namedBaseFolder);
- files.add(namedDataFolder);
- }
- } else {
- String path = getTargetDir(relPath).getAbsolutePath();
- logger.fine("listing path: " + path);
- File dir = new File(path);
- if(dir.exists()) {
- File[] fileArray = dir.listFiles();
- for(int i = 0; i < fileArray.length; i++) {
- logger.fine(fileArray[i].toURI().toString());
- String fname = fileArray[i].toURI().toString().replace("file:/", "");
- if(fileArray[i].isDirectory()) {
- fname = fname.substring(0, fname.length() - 1);
+ if (!relPath.startsWith(".")) {
+ if (relPath.length() == 0) {
+ FileRef namedPublicFolder = new FileRef(PUB_DIR_NAME, true);
+ logger.finer(namedPublicFolder.getAbsolutePath());
+ FileRef namedHomeFolder = new FileRef(HOME_DIR_NAME, true);
+ logger.finer(namedHomeFolder.getAbsolutePath());
+ FileRef namedDavFolder = new FileRef(DAV_DIR_NAME, true);
+ logger.finer(namedDavFolder.getAbsolutePath());
+ files = new ArrayList();
+ files.add(namedHomeFolder);
+ files.add(namedPublicFolder);
+ files.add(namedDavFolder);
+ if (getRequest().isUserInRole(WBX_ADMIN_ROLE)) {
+ FileRef namedBaseFolder = new FileRef(WBX_BASE, true);
+ FileRef namedDataFolder = new FileRef(WBX_DATA, true);
+ files.add(namedBaseFolder);
+ files.add(namedDataFolder);
+ }
+ } else {
+ String path = getTargetDir(relPath).getAbsolutePath();
+ logger.fine("listing path: " + path);
+ File dir = new File(path);
+ if (dir.exists()) {
+ File[] fileArray = dir.listFiles();
+ if (orderBy != null && orderBy.equalsIgnoreCase("name")) {
+ Arrays.sort(fileArray, new FileNameComparator(order));
+ } else {
+ Arrays.sort(fileArray, new FileNameComparator(AbstractComparator.ORDER_ASC));
}
- logger.fine(fname);
- FileRef ref = new FileRef(fname, fileArray[i].isDirectory());
- ref.setMimetype(bild.getMimeType(fileArray[i]));
- files.add(ref);
+ for (int i = 0; i < fileArray.length; i++) {
+ logger.fine(fileArray[i].toURI().toString());
+ String fname = fileArray[i].toURI().toString().replace("file:/", "");
+ if (fileArray[i].isDirectory()) {
+ fname = fname.substring(0, fname.length() - 1);
+ }
+ logger.fine(fname);
+ FileRef ref = new FileRef(fname, fileArray[i].isDirectory());
+ ref.setMimetype(bild.getMimeType(fileArray[i]));
+ files.add(ref);
+ }
}
}
- }
+ }
return files;
}
+
public FileRef newFolder(String relPath, String folderName) {
- logger.finer(relPath);
- String targetPath = null;
- if(relPath.startsWith(PUB_DIR_NAME)) {
- targetPath = PUB_DIR_PATH + getUserName() + "/" + relPath.substring(PUB_DIR_NAME.length()) + "/" + folderName;
- } else if(relPath.startsWith(HOME_DIR_NAME)) {
- targetPath = HOME_DIR_PATH + getUserName() + "/" + relPath.substring(HOME_DIR_NAME.length()) + "/" + folderName;
+ if (!relPath.startsWith(".")) {
+ logger.finer(relPath);
+ String targetPath = null;
+ if(relPath.startsWith(PUB_DIR_NAME)) {
+ targetPath = PUB_DIR_PATH + getUserName() + "/" + relPath.substring(PUB_DIR_NAME.length()) + "/" + folderName;
+ } else if(relPath.startsWith(HOME_DIR_NAME)) {
+ targetPath = HOME_DIR_PATH + getUserName() + "/" + relPath.substring(HOME_DIR_NAME.length()) + "/" + folderName;
+ } else {
+ // kann eigentlich nicht sein..
+ }
+ logger.finer(targetPath);
+ File targetDir = new File(getBase().getAbsolutePath(), targetPath);
+ targetDir.mkdirs();
+ return new FileRef(targetDir.getAbsolutePath(), true);
} else {
- // kann eigentlich nicht sein..
+ return null;
}
- logger.finer(targetPath);
- File targetDir = new File(getBase().getAbsolutePath(), targetPath);
- targetDir.mkdirs();
- return new FileRef(targetDir.getAbsolutePath(), true);
}
public String getCode(String relPath, String fileName) {
String code = null;
-
- Object p = getRequest().getUserPrincipal();
- if(p instanceof Principal) {
- Reader reader = null;
- try {
- File targetFile = new File(getTargetDir(relPath), fileName);
-
- //reader = new InputStreamReader(new FileInputStream(targetFile), "UTF8");
-
- reader = new FileReader(targetFile);
- StringBuffer buf = new StringBuffer();
- char[] readBuffer = new char[1024];
- int charsRead = reader.read(readBuffer);
- while(charsRead > -1) {
- buf.append(readBuffer, 0, charsRead);
- charsRead = reader.read(readBuffer);
- }
- code = buf.toString();
- } catch (FileNotFoundException ex) {
- Logger.getLogger(FileMgr.class.getName()).log(Level.SEVERE, null, ex);
- } catch (IOException ex) {
- Logger.getLogger(FileMgr.class.getName()).log(Level.SEVERE, null, ex);
- } finally {
+ if (!relPath.startsWith(".")) {
+ Object p = getRequest().getUserPrincipal();
+ if (p instanceof Principal) {
+ Reader reader = null;
try {
- reader.close();
+ File targetFile = new File(getTargetDir(relPath), fileName);
+
+ //reader = new InputStreamReader(new FileInputStream(targetFile), "UTF8");
+ reader = new FileReader(targetFile);
+ StringBuffer buf = new StringBuffer();
+ char[] readBuffer = new char[1024];
+ int charsRead = reader.read(readBuffer);
+ while (charsRead > -1) {
+ buf.append(readBuffer, 0, charsRead);
+ charsRead = reader.read(readBuffer);
+ }
+ code = buf.toString();
+ } catch (FileNotFoundException ex) {
+ Logger.getLogger(FileMgr.class.getName()).log(Level.SEVERE, null, ex);
} catch (IOException ex) {
Logger.getLogger(FileMgr.class.getName()).log(Level.SEVERE, null, ex);
+ } finally {
+ try {
+ reader.close();
+ } catch (IOException ex) {
+ Logger.getLogger(FileMgr.class.getName()).log(Level.SEVERE, null, ex);
+ }
}
+
}
-
- }
-
+ }
return code;
}
public String renameFile(String relPath, String fname, String newName) {
- File targetDir = getTargetDir(relPath);
- File file = new File(targetDir, fname);
- file.renameTo(new File(targetDir, newName));
- return fname + " umbenannt zu " + newName;
+ if (!relPath.startsWith(".")) {
+ File targetDir = getTargetDir(relPath);
+ File file = new File(targetDir, fname);
+ file.renameTo(new File(targetDir, newName));
+ return fname + " umbenannt zu " + newName;
+ } else {
+ return "Pfad nicht erlaubt.";
+ }
}
public String deleteFiles(String relPath, List fileNames) {
String result = null;
try {
logger.fine(fileNames.toString());
- File targetDir = getTargetDir(relPath);
- for(int i=0; i < fileNames.size(); i++) {
- Object o = fileNames.get(i);
- if(o instanceof ArrayList) {
- ArrayList al = (ArrayList) o;
- logger.fine(al.get(0).toString());
- File targetFile = new File(targetDir, al.get(0).toString());
- logger.fine(targetFile.getAbsolutePath());
- if(targetFile.isDirectory()) {
- FileUtils.deleteDirectory(targetFile);
- } else {
- targetFile.delete();
+ if (!relPath.startsWith(".")) {
+ File targetDir = getTargetDir(relPath);
+ for(int i=0; i < fileNames.size(); i++) {
+ Object o = fileNames.get(i);
+ if(o instanceof ArrayList) {
+ ArrayList al = (ArrayList) o;
+ logger.fine(al.get(0).toString());
+ File targetFile = new File(targetDir, al.get(0).toString());
+ logger.fine(targetFile.getAbsolutePath());
+ if(targetFile.isDirectory()) {
+ FileUtils.deleteDirectory(targetFile);
+ } else {
+ targetFile.delete();
+ }
}
}
+ result = "deleted";
}
- result = "deleted";
} catch (Throwable ex) {
logger.log(Level.SEVERE, ex.getLocalizedMessage(), ex);
}
@@ -205,25 +246,27 @@
private String copyOrMoveFiles(String fromPath, String toPath, List fileNames, int operation) {
String result = null;
try {
- File srcDir = getTargetDir(fromPath);
- File targetDir = getTargetDir(toPath);
- Iterator i = fileNames.iterator();
- while(i.hasNext()) {
- Object o = i.next();
- if (o instanceof ArrayList) {
- ArrayList al = (ArrayList) o;
- File srcFile = new File(srcDir, al.get(0).toString());
- if(srcFile.isDirectory()) {
- if(operation == OP_MOVE) {
- FileUtils.moveDirectoryToDirectory(srcFile, targetDir, false);
+ if (!fromPath.startsWith(".")) {
+ File srcDir = getTargetDir(fromPath);
+ File targetDir = getTargetDir(toPath);
+ Iterator i = fileNames.iterator();
+ while(i.hasNext()) {
+ Object o = i.next();
+ if (o instanceof ArrayList) {
+ ArrayList al = (ArrayList) o;
+ File srcFile = new File(srcDir, al.get(0).toString());
+ if(srcFile.isDirectory()) {
+ if(operation == OP_MOVE) {
+ FileUtils.moveDirectoryToDirectory(srcFile, targetDir, false);
+ } else {
+ FileUtils.copyDirectoryToDirectory(srcFile, targetDir);
+ }
} else {
- FileUtils.copyDirectoryToDirectory(srcFile, targetDir);
- }
- } else {
- if(operation == OP_MOVE) {
- FileUtils.moveFileToDirectory(srcFile, targetDir, false);
- } else {
- FileUtils.copyFileToDirectory(srcFile, targetDir);
+ if(operation == OP_MOVE) {
+ FileUtils.moveFileToDirectory(srcFile, targetDir, false);
+ } else {
+ FileUtils.copyFileToDirectory(srcFile, targetDir);
+ }
}
}
}
@@ -237,16 +280,18 @@
public FileRef saveTextFileAs(String relPath, String fileName, String contents) {
FileRef savedFile = null;
logger.fine(relPath + " " + fileName);
- //FileRef datenRef = getBase();
- Object p = getRequest().getUserPrincipal();
- if(p instanceof Principal) {
- File targetFile = new File(getTargetDir(relPath), fileName);
- if(targetFile.exists()) {
- targetFile = getNewFileName(targetFile);
- } else {
- targetFile.getParentFile().mkdirs();
+ if (!relPath.startsWith(".")) {
+ //FileRef datenRef = getBase();
+ Object p = getRequest().getUserPrincipal();
+ if(p instanceof Principal) {
+ File targetFile = new File(getTargetDir(relPath), fileName);
+ if(targetFile.exists()) {
+ targetFile = getNewFileName(targetFile);
+ } else {
+ targetFile.getParentFile().mkdirs();
+ }
+ saveToFile(targetFile, contents);
}
- saveToFile(targetFile, contents);
}
return savedFile;
}
@@ -302,34 +347,36 @@
public FileRef saveTextFile(String relPath, String fileName, String contents) {
FileRef savedFile = null;
logger.fine(relPath + " " + fileName);
- //FileRef datenRef = getBase();
- Object p = getRequest().getUserPrincipal();
- if(p instanceof Principal) {
- File targetFile = new File(getTargetDir(relPath), fileName);
- if(targetFile.exists()) {
- /*
- muss delete() sein?
- pruefen: ueberschreibt der FileWriter den alteen Inhalt oder
- entsteht eine unerwuenschte Mischung aus altem und neuem
- Inhalt?
- */
- targetFile.delete();
- } else {
- targetFile.getParentFile().mkdirs();
+ if (!relPath.startsWith(".")) {
+ //FileRef datenRef = getBase();
+ Object p = getRequest().getUserPrincipal();
+ if(p instanceof Principal) {
+ File targetFile = new File(getTargetDir(relPath), fileName);
+ if(targetFile.exists()) {
+ /*
+ muss delete() sein?
+ pruefen: ueberschreibt der FileWriter den alteen Inhalt oder
+ entsteht eine unerwuenschte Mischung aus altem und neuem
+ Inhalt?
+ */
+ targetFile.delete();
+ } else {
+ targetFile.getParentFile().mkdirs();
+ }
+ saveToFile(targetFile, contents);
}
- saveToFile(targetFile, contents);
}
return savedFile;
}
public String bildVerkleinern(String relPath, String bildName) {
- File dir = getTargetDir(relPath);
- File original = new File(dir, bildName);
- Bild bild = new Bild();
- //for (int i = 0; i < Bild.GR.length; i++) {
-
+ if (!relPath.startsWith(".")) {
+ File dir = getTargetDir(relPath);
+ File original = new File(dir, bildName);
+ Bild bild = new Bild();
+ //for (int i = 0; i < Bild.GR.length; i++) {
+
//int gr = bild.getVariantenGroesse(i);
-
String ext = "";
String nurname = bildName;
int dotpos = bildName.indexOf(".");
@@ -337,47 +384,51 @@
ext = bildName.substring(dotpos);
nurname = bildName.substring(0, dotpos);
}
-
+
Image image = Toolkit.getDefaultToolkit().getImage(original.getAbsolutePath());
MediaTracker mediaTracker = new MediaTracker(new Container());
mediaTracker.addImage(image, 0);
try {
mediaTracker.waitForID(0);
- if(!mediaTracker.isErrorAny()) {
- for(int i = 0; i < Bild.GR.length; i++) {
+ if (!mediaTracker.isErrorAny()) {
+ for (int i = 0; i < Bild.GR.length; i++) {
StringBuffer buf = new StringBuffer();
buf.append(nurname);
buf.append(bild.getVariantenName(i));
buf.append(ext);
File newImgFile = new File(dir, buf.toString());
- if(!newImgFile.exists()) {
+ if (!newImgFile.exists()) {
logger.fine(original.getAbsolutePath() + " " + newImgFile.getAbsolutePath());
bild.writeImageFile(image, bild.getVariantenGroesse(i), bild.getMimeType(original), newImgFile.getAbsolutePath());
//bild.writeImageFile(image, photo.getVariantenGroesse(i), photo.getMimetype(), photo.getAbsolutePath(basisPfad), photo.getVariantenName(basisPfad, i));
}
}
}
- } catch(IOException | InterruptedException ex) {
+ } catch (IOException | InterruptedException ex) {
logger.log(Level.SEVERE, ex.getLocalizedMessage(), ex);
}
-
- return "ok";
+ return "ok";
+ } else {
+ return "Pfad micht erlaubt.";
+ }
}
public String extractZipfile(String relPath, String filename) {
- String result;
- try {
- File targetDir = getTargetDir(relPath);
- File archive = new File(targetDir, filename);
- if(extract(archive)) {
- result = "ok";
- } else {
- result = "error while extracting";
+ String result = null;
+ if (!relPath.startsWith(".")) {
+ try {
+ File targetDir = getTargetDir(relPath);
+ File archive = new File(targetDir, filename);
+ if(extract(archive)) {
+ result = "ok";
+ } else {
+ result = "error while extracting";
+ }
+ } catch(Exception ex) {
+ result = ex.getLocalizedMessage();
+ logger.log(Level.SEVERE, ex.getLocalizedMessage(), ex);
}
- } catch(Exception ex) {
- result = ex.getLocalizedMessage();
- logger.log(Level.SEVERE, ex.getLocalizedMessage(), ex);
}
return result;
}
--
Gitblit v1.9.3