--- ImageArchive/ImageArchive.py	2006/02/07 18:36:07	1.74
+++ ImageArchive/ImageArchive.py	2006/03/29 18:48:00	1.76
@@ -22,6 +22,7 @@ import time
 from threading import Thread
 import Queue
 import cgi
+import zLOG
 
 from AccessControl import ClassSecurityInfo, getSecurityManager
 
@@ -297,6 +298,11 @@ class ImageDigilib(Folder,Persistent,Imp
         user=getSecurityManager().getUser()
 
         roles=user.getRoles()
+
+	if self.getRightsQuery()=="":
+            #query empty then always true
+	    return True
+
         if 'Authenticated' in roles:
             return True
         
@@ -481,7 +487,8 @@ class ImageDigilib(Folder,Persistent,Imp
     def updateImage(self,file,rename=None,RESPONSE=None):
             """lade neues Version des Bildes"""
             #teste ob Dokumenten ordner schon vorhanden
-            imagePath=os.path.join(self.ImageStoragePath,self.getId())
+            #imagePath=os.path.join(self.ImageStoragePath,self.getId())
+            imagePath=os.path.join(self.ImageStoragePath,self.filename)
             path=os.path.join(self.ImageStoragePath,"."+self.getId()+".dir")
             
             if not os.path.exists(path):
@@ -492,16 +499,20 @@ class ImageDigilib(Folder,Persistent,Imp
 
             #teste ob version schon existiert
 
-            while os.path.exists(os.path.join(path,"V%i_%s"%(versionNumber,self.getId()))):
+            while os.path.exists(os.path.join(path,"V%i_%s"%(versionNumber,self.filename))):
                     versionNumber+=1
 
             #kopieren der bestehenden Version in den Versions ordner.
-            imageNewPath=os.path.join(path,"V%i_%s"%(versionNumber,self.getId()))
-            os.rename(imagePath,imageNewPath)
+            imageNewPath=os.path.join(path,"V%i_%s"%(versionNumber,self.filename))
+	    try:
+		#zLOG.LOG("ImageArchive:updateImage", zLOG.INFO, "rename: %s -> %s"%(imagePath,imageNewPath))
+		os.rename(imagePath,imageNewPath)
+	    except:
+		zLOG.LOG("ImageArchive:updateImage", zLOG.ERROR, "rename: %s -> %s didn't work!"%(imagePath,imageNewPath))	
 
             #lesen des upload files und schreiben
             filedata=file.read()
-            f=open(imagePath,"w")
+            f=open(imagePath,"w") # if we wanted to have filename=id we should do it here!
             f.write(filedata)
             f.close()
             try:
@@ -1598,7 +1609,7 @@ class ImageCollection(Folder, Persistent
         
         
     security.declareProtected('View','index_html')
-    def index_html(self,fn=None,generic=None):
+    def index_html(self,fn=None,generic='No'):
         """main template collection"""
     
         if fn:
@@ -1617,7 +1628,8 @@ class ImageCollection(Folder, Persistent
           
             overview=self.ZopeFind(self,obj_ids=['overview_selected.html'])
             
-            if overview and (generic is None):
+
+            if overview and (generic =='No'):
                 return overview[0][1]()
             else:
                 pt=PageTemplateFile(os.path.join(package_home(globals()),'zpt','overview_selected.zpt')).__of__(self)
@@ -1638,7 +1650,7 @@ class ImageCollection(Folder, Persistent
             
 
         overview=self.ZopeFind(self,obj_ids=['overview.html'])
-        if overview and (generic is None):
+        if overview and (generic == 'No'):
             return overview[0][1]()
         elif hasattr(self,'templates'):
             pt=self.templates.imgcoll_main.__of__(self)