[Checkins] SVN: lovely.tag/trunk/src/lovely/tag/tag.py fixed naming of the 'name' attribute, this should not affect current implementations which uses only the 'name' attribute as defined in the interface, implementations which use the undocumented 'tag' attribute are now broken

Bernd Dorn bernd.dorn at lovelysystems.com
Mon Jan 8 08:29:08 EST 2007


Log message for revision 71803:
  fixed naming of the 'name' attribute, this should not affect current implementations which uses only the 'name' attribute as defined in the interface, implementations which use the undocumented 'tag' attribute are now broken

Changed:
  U   lovely.tag/trunk/src/lovely/tag/tag.py

-=-
Modified: lovely.tag/trunk/src/lovely/tag/tag.py
===================================================================
--- lovely.tag/trunk/src/lovely/tag/tag.py	2007-01-08 13:25:43 UTC (rev 71802)
+++ lovely.tag/trunk/src/lovely/tag/tag.py	2007-01-08 13:29:07 UTC (rev 71803)
@@ -33,21 +33,21 @@
 
     item = FieldProperty(interfaces.ITag['item'])
     user = FieldProperty(interfaces.ITag['user'])
-    tag = FieldProperty(interfaces.ITag['name'])
+    name = FieldProperty(interfaces.ITag['name'])
     timestamp = FieldProperty(interfaces.ITag['timestamp'])
-
-    def __init__(self, item, user, tag):
+    
+    def __init__(self, item, user, name):
         self.item = item
         if not isinstance(user, unicode):
             user = unicode(user, 'utf-8')
         self.user = user
-        self.tag = tag
+        self.name = name
         self.timestamp = datetime.datetime.now()
 
     def __cmp__(self, other):
-        return cmp((self.item, self.user, self.tag),
-                   (other.item, other.user, other.tag))
+        return cmp((self.item, self.user, self.name),
+                   (other.item, other.user, other.name))
 
     def __repr__(self):
         return '<%s %r for %i by %r>' %(
-            self.__class__.__name__, self.tag, self.item, self.user)
+            self.__class__.__name__, self.name, self.item, self.user)



More information about the Checkins mailing list