# HG changeset patch
# User Paul Boddie <paul@boddie.org.uk>
# Date 1396559169 -7200
# Node ID bb86e3e374a3845d20c11bf7193feb6d75d876ed
# Parent  bca53f2afce89390d47534a8491c9867833656a5
Some tidying and commenting.

diff -r bca53f2afce8 -r bb86e3e374a3 MoinShare.py
--- a/MoinShare.py	Thu Apr 03 22:25:14 2014 +0200
+++ b/MoinShare.py	Thu Apr 03 23:06:09 2014 +0200
@@ -120,24 +120,30 @@
     "A feed update entry."
 
     def __init__(self):
-        self.title = None
-        self.link = None
         self.content = None
         self.content_type = None
         self.updated = None
         self.author = None
 
+        # Message-related attributes.
+
+        self.parts = None
+
+        # Feed-related attributes.
+
+        self.title = None
+        self.link = None
+
         # Page-related attributes.
 
         self.fragment = None
         self.preferred = None
 
-        # Message-related attributes.
+        # Store-related attributes.
 
         self.message_number = None
-        self.parts = None
 
-        # Message- and page-related attributes.
+        # Store- and page-related attributes.
 
         self.page = None
 
@@ -146,6 +152,11 @@
         self.path = []
 
     def unique_id(self):
+
+        """
+        A unique identifier used for anchors to parts of presented updates.
+        """
+
         return "moinshare-tab-%s-%s" % (self.message_number, "-".join(map(str, self.path)))
 
     def __cmp__(self, other):
@@ -380,7 +391,7 @@
 
     for message_number, part in enumerate(mailbox.get_payload()):
         update = Update()
-        update.fragment = update.updated = getDateTimeFromRFC2822(part.get("date"))
+        update.updated = getDateTimeFromRFC2822(part.get("date"))
         update.title = part.get("subject", "Update #%d" % message_number)
         update.message_number = message_number
 
@@ -495,7 +506,7 @@
 
     # Produce a fragment identifier.
 
-    update.fragment = update.updated = getDateTimeFromRFC2822(message.get("date"))
+    update.updated = getDateTimeFromRFC2822(message.get("date"))
     update.title = message.get("subject", "Update #%d" % message_number)
     update.author = message.get("moin-user")
 
@@ -643,6 +654,7 @@
     single = len(updates) == 1
 
     # Format some navigation tabs.
+    # This only occurs for multipart updates.
 
     if not single:
         append(fmt.div(on=1, css_class="moinshare-alternatives"))