Changeset 1462 in products for quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/browser/views.py

Timestamp:

Dec 18, 2009 2:06:24 PM (14 years ago)

Author:

liebster

Message:

Cleaned code

File:

• quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/browser/views.py (modified) (4 diffs)

quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/browser/views.py

@@ -310 +310 @@
         """
         return condition and first or second 
+      
+    def setProperty(self, property, value, type='string'):
+        context = aq_inner(self.context)
+        if context.hasProperty(property):
+            context.manage_changeProperties({property: value})
+        else:
+            context.manage_addProperty(property, value, type)
 
     def manageSEOProps(self, **kw):
@@ -328 +335 @@
         if delete_list: context.manage_delProperties(delete_list)
 
-    def delSEOCustomMetaTags(self):
+    def setSEOCustomMetaTags(self, custommetatags):
+        context = aq_inner(self.context)
+        for k, v in custommetatags.items():
+            self.setProperty('%s%s' % (PROP_CUSTOM_PREFIX, meta_name), meta_content)   
+
+    def delAllSEOCustomMetaTagsByNames(self):
         context = self.context
         delete_list = []
@@ -336 +348 @@
         if delete_list: context.manage_delProperties(delete_list)        
 
-    def delSEOCustomMetaTag(self, custommetatag):
+    def delSEOCustomMetaTagByName(self, custommetatagname):
         context = self.context
-        if context.hasProperty(custommetatag):
-            context.manage_delProperties(custommetatag)
-      
-    def manageSEOCustomMetaTags(self, **kw):
-        context = aq_inner(self.context)
-        if kw.has_key('seo_custommetatags_override'):
-            if kw.get('seo_custommetatags_override'):
-                custommetatags = kw.get('seo_custommetatags', {})
-                self.updateSEOCustomMetaTags(custommetatags)
-            else:
-                self.delSEOCustomMetaTags()
-        elif kw.get('seo_custommetatags'):
-            self.delSEOCustomMetaTags()
+        seo_custom_prop = PROP_CUSTOM_PREFIX + custommetatagname
+        if context.hasProperty(seo_custom_prop):
+            context.manage_delProperties([seo_custom__prop])
 
     def updateSEOCustomMetaTags(self, custommetatags):
@@ -366 +368 @@
   
         for tag in custommetatags:
+            metalist = []
             meta_name, meta_content = tag['meta_name'], tag['meta_content']
             if meta_name:
                 if not [gmt for gmt in globalCustomMetaTags if (gmt['meta_name']==meta_name and gmt['meta_content']==meta_content)]:
-                    self.setProperty('%s%s' % (PROP_CUSTOM_PREFIX, meta_name), meta_content)
-
-    def setSEOCustomMetaTags(self, custommetatags=None):
-        context = aq_inner(self.context)
-        if custommetatags is not None:
-            if len(custommetatags):
-                self.delSEOCustomMetaTags()
+                    metalist.append(tag)
+                self.setSEOCustomMetaTags(metalist)
+      
+    def manageSEOCustomMetaTags(self, **kw):
+        context = aq_inner(self.context)
+        if kw.has_key('seo_custommetatags_override'):
+            if kw.get('seo_custommetatags_override'):
+                custommetatags = kw.get('seo_custommetatags', {})
+                self.updateSEOCustomMetaTags(custommetatags)
             else:
-                for k, v in custommetatags.items():
-                    self.setProperty('%s%s' % (PROP_CUSTOM_PREFIX, meta_name), meta_content)   
-      
-    def setProperty(self, property, value, type='string'):
-        context = aq_inner(self.context)
-        if context.hasProperty(property):
-            context.manage_changeProperties({property: value})
-        else:
-            context.manage_addProperty(property, value, type)
+                self.delAllSEOCustomMetaTagsByNames()
+        elif kw.get('seo_custommetatags'):
+            self.delAllSEOCustomMetaTagsByNames()
 
     def __call__( self ):