changeset 208:6d6076e28430

fix deleteField for additionalLink (still ugly though)
author casties
date Mon, 01 Jul 2013 18:57:22 +0200
parents 938add25f81b
children e0cc46205395
files MPIWGStaff.py zpt/staff/edit_additionalLinks.zpt
diffstat 2 files changed, 9 insertions(+), 12 deletions(-) [+]
line wrap: on
line diff
--- a/MPIWGStaff.py	Mon Jul 01 12:59:10 2013 +0200
+++ b/MPIWGStaff.py	Mon Jul 01 18:57:22 2013 +0200
@@ -476,7 +476,7 @@
 
 
     def getAdditionalLinks(self):        
-        return self.folder.executeZSQL("SELECT oid,* FROM additionalLink  WHERE  key_main = %s ORDER BY priority",[self.content.key]) 
+        return self.folder.executeZSQL("SELECT oid,* FROM additionallink  WHERE  key_main = %s ORDER BY priority",[self.content.key]) 
 
     
     def getPathStyle(self, path, selected, style=""):
@@ -751,14 +751,11 @@
 
             
 
-    security.declareProtected('View management screens','deleteField') 
-   
+    security.declareProtected('View management screens','deleteField')    
     def deleteField(self,REQUEST):
         """delete entry"""
-        
-        CHANGEABLE=['talks','taching','pubmanbiblio']
-        
-        
+        # TODO: this is ugly
+        CHANGEABLE=['talks','teaching','pubmanbiblio','additionallink']
         
         table = REQUEST.form.get('table',None);
         oid = REQUEST.form.get('oid',None);
@@ -772,9 +769,9 @@
         if table is None or oid is None:
             return
         
-        query="DELETE FROM %s WHERE oid = '%s'"%(table,oid)
-
-        self.executeZSQL(query)
+        query="DELETE FROM %s WHERE oid = %%s"%(table)
+        self.executeZSQL(query, [oid])
+        
         REQUEST.response.redirect(self.REQUEST['HTTP_REFERER'])
  
    
--- a/zpt/staff/edit_additionalLinks.zpt	Mon Jul 01 12:59:10 2013 +0200
+++ b/zpt/staff/edit_additionalLinks.zpt	Mon Jul 01 18:57:22 2013 +0200
@@ -13,7 +13,7 @@
     <input type="hidden" name="main_fields" value="additionalLink__title">
     <table tal:define="links python:here.getAdditionalLinks()">
     <tal:x tal:repeat="link links">
-    <tr class="fliesstext">
+    <tr>
       <td>Text over Link</td>
       <td><input tal:attributes="name python:'additionalLink__title__'+str(link.oid); 
 				     value python:link.title" size="80" /> </td>
@@ -32,7 +32,7 @@
         <span tal:define="name string:additionalLink__published__${link/oid}; value link/published;">
           <tal:block metal:use-macro="here/common_template/macros/yesno_input_radio"/>
         </span>
-      &nbsp;&nbsp;<a tal:attributes="href string:$root/deleteField?table=additionalLink&oid=${link/oid}">delete this link</a>
+      &nbsp;&nbsp;<a tal:attributes="href string:$root/deleteField?table=additionallink&oid=${link/oid}">delete this link</a>
       </td>
     </tr>
     <tr>