diff --git a/build.xml b/build.xml
index 457be59..d033c56 100755
--- a/build.xml
+++ b/build.xml
@@ -11,7 +11,6 @@
-
@@ -78,6 +77,10 @@
path="/Zallery"
update="true"
war="file:${releaseDir}/Zallery.war" />
+
+
+
diff --git a/src/zall/UploadServlet.java b/src/zall/UploadServlet.java
index e814c4c..e3e1731 100644
--- a/src/zall/UploadServlet.java
+++ b/src/zall/UploadServlet.java
@@ -30,9 +30,8 @@ public class UploadServlet extends AjaxFileUpload{
private static final Logger logger = LogUtil.getLogger();
private static final long serialVersionUID = 1L;
- public static final Set VIDEO_EXT = Collections.unmodifiableSet(new HashSet(Arrays.asList(new String[]{
- "avi","mp4","mpeg","mpeg","divx","xvid","wmv","mov","flv","m4v"
- })));
+ private static final Set VIDEO_EXT = Collections.unmodifiableSet(new HashSet(Arrays.asList(
+ "avi","mp4","mpeg","mpeg","divx","xvid","wmv","mov","flv","m4v")));
@Override
public String getProgressHTML() {
@@ -76,6 +75,7 @@ public class UploadServlet extends AjaxFileUpload{
vid.setUser( user );
vid.setFile( item );
vid.save(db);
+ logger.info("Video upload successful: "+vid.getFolder().getPath());
}
else{
Image img = new Image();
@@ -83,7 +83,8 @@ public class UploadServlet extends AjaxFileUpload{
img.setFolder( folder );
img.setUser( user );
img.setFile( item );
- img.save(db);
+ img.save(db);
+ logger.info("Image upload successful: "+img.getFolder().getPath());
}
} catch (Exception e) {
logger.log(Level.WARNING, "Error: Creating new Media(\""+item.getName()+"\")", e);
diff --git a/src/zall/Zallery.java b/src/zall/Zallery.java
index 9205dac..d9c6d7a 100755
--- a/src/zall/Zallery.java
+++ b/src/zall/Zallery.java
@@ -32,7 +32,7 @@ import zutil.log.LogUtil;
public class Zallery extends HttpServlet{
private static Logger logger = LogUtil.getLogger();
- public static final String VERSION = "1.0.0";
+ public static final String VERSION = "1.0.2";
public static String WEBSITE_NAME = "Example.com";
public static String WEBSITE_URL = "http://example.com";
@@ -54,7 +54,7 @@ public class Zallery extends HttpServlet{
Context context = new InitialContext();
// Check if Zallery has been properly configured
if ("C:\\\\data".equals(context.lookup("java:comp/env/DATA_PATH")))
- throw new ServletException("Zallery has not been properly configured, set proper configuration in Zaller.xml context file.");
+ throw new ServletException("Zallery has not been properly configured, set proper configuration in Zallery.xml context file.");
WEBSITE_NAME = (String)context.lookup("java:comp/env/WEBSITE_NAME");
WEBSITE_URL = (String)context.lookup("java:comp/env/WEBSITE_URL");
@@ -211,7 +211,7 @@ public class Zallery extends HttpServlet{
}
}
List list = Media.load(db, folder);
- List subFolders = Folder.loadSubFolders(db, folder.getId(), user);
+ List subFolders = Folder.loadSubFolders(db, folder, user);
//session.setAttribute("user", user);
request.setAttribute("folder", folder);
diff --git a/src/zall/bean/Folder.java b/src/zall/bean/Folder.java
index fdb585a..6f4cc36 100755
--- a/src/zall/bean/Folder.java
+++ b/src/zall/bean/Folder.java
@@ -4,8 +4,10 @@ import java.io.File;
import java.sql.PreparedStatement;
import java.sql.SQLException;
import java.sql.Timestamp;
+import java.util.Collections;
import java.util.LinkedList;
import java.util.List;
+import java.util.logging.Level;
import java.util.logging.Logger;
import java.util.regex.Matcher;
@@ -21,25 +23,24 @@ import zall.Zallery;
public class Folder extends DBBean{
private static final Logger logger = LogUtil.getLogger();
- protected transient String name;
- protected User user;
- protected String path;
+ private transient String name;
+ protected long user = -1;
+ protected transient User userInstance;
+ private String path;
protected Folder parent;
- protected Timestamp date;
- protected boolean isPrivate;
+ protected Timestamp date;
+ private boolean isPrivate;
- //@DBLinkTable(name="FolderTree", beanClass=Folder.class, idColumn="parent")
- //protected LinkedList subFolders;
public static Folder load(DBConnection db, Long id) throws SQLException{
return load(db, Folder.class, id);
}
- public static List loadSubFolders(DBConnection db, Long id, User user) throws SQLException{
+ public static List loadSubFolders(DBConnection db, Folder folder, User requestingUser) throws SQLException{
PreparedStatement sql = db.getPreparedStatement("SELECT * FROM Folder WHERE parent=? AND (isPrivate=0 OR user=? OR ?)");
- sql.setLong(1, id);
- sql.setLong(2, user.getId());
- sql.setBoolean(3, user.isSuperUser());
+ sql.setLong(1, folder.getId());
+ sql.setLong(2, folder.user);
+ sql.setBoolean(3, requestingUser.isSuperUser());
return DBConnection.exec(sql, DBBeanSQLResultHandler.createList(Folder.class, db));
}
@@ -60,15 +61,14 @@ public class Folder extends DBBean{
public static List load(DBConnection db, User user) throws SQLException{
if( user.getId() == null )
- return new LinkedList();
+ return Collections.emptyList();
PreparedStatement sql = db.getPreparedStatement("SELECT * FROM Folder WHERE user=?");
sql.setLong(1, user.getId() );
return DBConnection.exec(sql, DBBeanSQLResultHandler.createList(Folder.class, db));
}
public Folder(){
- //subFolders = new LinkedList();
- date = new Timestamp( System.currentTimeMillis() );
+ date = new Timestamp( System.currentTimeMillis() );
}
public String getName(){
@@ -76,20 +76,25 @@ public class Folder extends DBBean{
String[] tmp = path.split("/");
name = tmp[tmp.length-1];
}
- String userName = user != null ? user.getName() : "UNKNOWN";
+ String userName = getUser() != null ? getUser().getName() : "UNKNOWN";
String tmp = name.replaceAll("\\{NAME\\}", userName);
return tmp;
}
- public String getUnModName(){
- if(name == null)
- getName();
- return name;
- }
public User getUser(){
- return user;
+ if (userInstance == null) {
+ try {
+ DBConnection db = Zallery.getDB();
+ userInstance = User.load(db, user);
+ db.close();
+ } catch (Exception e) {
+ logger.log(Level.WARNING, null, e);
+ }
+ }
+ return userInstance;
}
public void setUser(User user){
- this.user = user;
+ this.user = user.getId();
+ this.userInstance = null;
}
public boolean isPrivate(){
return isPrivate;
@@ -98,12 +103,9 @@ public class Folder extends DBBean{
this.isPrivate = priv;
}
public String getPath(){
- String tmp = path.replaceAll("\\{NAME\\}", user.getName());
+ String tmp = path.replaceAll("\\{NAME\\}", getUser().getName());
return tmp;
}
- public String getUnModPath(){
- return path;
- }
public void setName(String name){
this.name = name;
if( parent.path.endsWith("/") )
@@ -125,20 +127,10 @@ public class Folder extends DBBean{
return parent;
}
- /*public void addSubFolder(Folder f){
- if( !subFolders.contains(f) ){
- subFolders.add( f );
- f.setParent( this );
- }
- }*/
+ public Timestamp getDate(){
+ return date;
+ }
- /*public List getSubFolders(){
- return subFolders;
- }*/
-
- public Timestamp getDate(){
- return date;
- }
/**
* @param filename is the name of the file
@@ -153,7 +145,7 @@ public class Folder extends DBBean{
return null;
}
- if( user == null || filename == null )
+ if( user < 0 || filename == null )
return null;
StringBuilder tmp = new StringBuilder();
@@ -166,7 +158,7 @@ public class Folder extends DBBean{
tmp.append( File.separatorChar );
// Add UserID and this folders path
- String tmp_path = path.replaceAll("\\{NAME\\}", ""+user.getId());
+ String tmp_path = path.replaceAll("\\{NAME\\}", ""+user);
tmp_path = tmp_path.replaceAll("/", Matcher.quoteReplacement(File.separator));
tmp.append( tmp_path );
@@ -195,6 +187,6 @@ public class Folder extends DBBean{
}
public boolean isEmpty(DBConnection db) throws SQLException {
- return Folder.loadSubFolders(db, this.getId(), user).isEmpty() && Media.load(db, this).isEmpty();
+ return Folder.loadSubFolders(db, this, getUser()).isEmpty() && Media.load(db, this).isEmpty();
}
}