Removed more docstrings.

src/OFS/Cache.py

@@ -144,9 +144,7 @@ class Cacheable:
 
     security.declarePrivate('ZCacheable_getObAndView')
     def ZCacheable_getObAndView(self, view_name):
-        """
-        Returns self and view_name unchanged.
-        """
+        # Returns self and view_name unchanged.
         return self, view_name
 
     security.declarePrivate('ZCacheable_get')

src/OFS/DTMLMethod.py

@@ -93,7 +93,7 @@ class DTMLMethod(RestrictedDTML,
     security.declareProtected(View, '__call__')
     def __call__(self, client=None, REQUEST={}, RESPONSE=None, **kw):
         """Render using the given client object
-        
+
         o If client is not passed, we are being called as a sub-template:
           don't do any error propagation.
 
@@ -201,14 +201,12 @@ class DTMLMethod(RestrictedDTML,
 
     security.declareProtected(change_dtml_methods, 'getCacheNamespaceKeys')
     def getCacheNamespaceKeys(self):
-        """ Return the cacheNamespaceKeys.
-        """
+        # Return the cacheNamespaceKeys.
         return self._cache_namespace_keys
 
     security.declareProtected(change_dtml_methods, 'setCacheNamespaceKeys')
     def setCacheNamespaceKeys(self, keys, REQUEST=None):
-        """ Set the list of names looked up to provide a cache key.
-        """
+        # Set the list of names looked up to provide a cache key.
         ks = []
         for key in keys:
             key = str(key).strip()
@@ -356,14 +354,12 @@ class DTMLMethod(RestrictedDTML,
 
     security.declareProtected(view_management_screens, 'PrincipiaSearchSource')
     def PrincipiaSearchSource(self):
-        """ Support for searching - the document's contents are searched.
-        """
+        # Support for searching - the document's contents are searched.
         return self.read()
 
     security.declareProtected(view_management_screens, 'document_src')
     def document_src(self, REQUEST=None, RESPONSE=None):
-        """ Return unprocessed document source.
-        """
+        # Return unprocessed document source.
         if RESPONSE is not None:
             RESPONSE.setHeader('Content-Type', 'text/plain')
         return self.read()

src/OFS/Image.py

@@ -65,23 +65,23 @@ def manage_addFile(self, id, file='', title='', precondition='',
     precondition = str(precondition)
 
     id, title = cookId(id, title, file)
-    
+
     self=self.this()
 
     # First, we create the file without data:
     self._setObject(id, File(id,title,'',content_type, precondition))
-    
+
     newFile = self._getOb(id)
-    
+
     # Now we "upload" the data.  By doing this in two steps, we
     # can use a database trick to make the upload more efficient.
     if file:
         newFile.manage_upload(file)
     if content_type:
         newFile.content_type=content_type
-    
+
     notify(ObjectCreatedEvent(newFile))
-    
+
     if REQUEST is not None:
         REQUEST['RESPONSE'].redirect(self.absolute_url()+'/manage_main')
 
@@ -436,8 +436,7 @@ class File(Persistent, Implicit, PropertyManager,
 
     security.declareProtected(View, 'PrincipiaSearchSource')
     def PrincipiaSearchSource(self):
-        """ Allow file objects to be searched.
-        """
+        # Allow file objects to be searched.
         if self.content_type.startswith('text/'):
             return str(self.data)
         return ''
@@ -473,9 +472,9 @@ class File(Persistent, Implicit, PropertyManager,
             self.update_data(filedata, content_type, len(filedata))
         else:
             self.ZCacheable_invalidate()
-        
+
         notify(ObjectModifiedEvent(self))
-        
+
         if REQUEST:
             message="Saved changes."
             return self.manage_main(self,REQUEST,manage_tabs_message=message)
@@ -494,9 +493,9 @@ class File(Persistent, Implicit, PropertyManager,
         content_type=self._get_content_type(file, data, self.__name__,
                                             'application/octet-stream')
         self.update_data(data, content_type, size)
-        
+
         notify(ObjectModifiedEvent(self))
-        
+
         if REQUEST:
             message="Saved changes."
             return self.manage_main(self,REQUEST,manage_tabs_message=message)
@@ -600,10 +599,8 @@ class File(Persistent, Implicit, PropertyManager,
 
     security.declareProtected(View, 'get_size')
     def get_size(self):
-        """Get the size of a file or image.
-
-        Returns the size of the file or image.
-        """
+        # Get the size of a file or image.
+        # Returns the size of the file or image.
         size=self.size
         if size is None: size=len(self.data)
         return size
@@ -613,10 +610,8 @@ class File(Persistent, Implicit, PropertyManager,
 
     security.declareProtected(View, 'getContentType')
     def getContentType(self):
-        """Get the content type of a file or image.
-
-        Returns the content type (MIME type) of a file or image.
-        """
+        # Get the content type of a file or image.
+        # Returns the content type (MIME type) of a file or image.
         return self.content_type
 
 
@@ -677,18 +672,18 @@ def manage_addImage(self, id, file, title='', precondition='', content_type='',
 
     # First, we create the image without data:
     self._setObject(id, Image(id,title,'',content_type, precondition))
-    
+
     newFile = self._getOb(id)
-    
+
     # Now we "upload" the data.  By doing this in two steps, we
     # can use a database trick to make the upload more efficient.
     if file:
         newFile.manage_upload(file)
     if content_type:
         newFile.content_type=content_type
-    
+
     notify(ObjectCreatedEvent(newFile))
-    
+
     if REQUEST is not None:
         try:    url=self.DestinationURL()
         except: url=REQUEST['URL1']
@@ -812,7 +807,7 @@ class Image(File):
         if isinstance(data, unicode):
             raise TypeError('Data can only be str or file-like.  '
                             'Unicode objects are expressly forbidden.')
-        
+
         if size is None: size=len(data)
 
         self.size=size

src/OFS/ObjectManager.py

@@ -303,12 +303,11 @@ class ObjectManager(CopyContainer,
 
     security.declareProtected(access_contents_information, 'hasObject')
     def hasObject(self, id):
-        """Indicate whether the folder has an item by ID.
-
-        This doesn't try to be more intelligent than _getOb, and doesn't
-        consult _objects (for performance reasons). The common use case
-        is to check that an object does *not* exist.
-        """
+        # Indicate whether the folder has an item by ID.
+        #
+        # This doesn't try to be more intelligent than _getOb, and doesn't
+        # consult _objects (for performance reasons). The common use case
+        # is to check that an object does *not* exist.
         if (id in ('.', '..') or
             id.startswith('_') or
             id.startswith('aq_') or

src/OFS/OrderSupport.py

@@ -132,8 +132,7 @@ class OrderSupport(object):
     security.declareProtected(manage_properties, 'moveObjectsByDelta')
     def moveObjectsByDelta(self, ids, delta, subset_ids=None,
                            suppress_events=False):
-        """ Move specified sub-objects by delta.
-        """
+        # Move specified sub-objects by delta.
         if isinstance(ids, basestring):
             ids = (ids,)
         min_position = 0
@@ -183,32 +182,27 @@ class OrderSupport(object):
 
     security.declareProtected(manage_properties, 'moveObjectsUp')
     def moveObjectsUp(self, ids, delta=1, subset_ids=None):
-        """ Move specified sub-objects up by delta in container.
-        """
+        # Move specified sub-objects up by delta in container.
         return self.moveObjectsByDelta(ids, -delta, subset_ids)
 
     security.declareProtected(manage_properties, 'moveObjectsDown')
     def moveObjectsDown(self, ids, delta=1, subset_ids=None):
-        """ Move specified sub-objects down by delta in container.
-        """
+        # Move specified sub-objects down by delta in container.
         return self.moveObjectsByDelta(ids, delta, subset_ids)
 
     security.declareProtected(manage_properties, 'moveObjectsToTop')
     def moveObjectsToTop(self, ids, subset_ids=None):
-        """ Move specified sub-objects to top of container.
-        """
+        # Move specified sub-objects to top of container.
         return self.moveObjectsByDelta( ids, -len(self._objects), subset_ids )
 
     security.declareProtected(manage_properties, 'moveObjectsToBottom')
     def moveObjectsToBottom(self, ids, subset_ids=None):
-        """ Move specified sub-objects to bottom of container.
-        """
+        # Move specified sub-objects to bottom of container.
         return self.moveObjectsByDelta( ids, len(self._objects), subset_ids )
 
     security.declareProtected(manage_properties, 'orderObjects')
     def orderObjects(self, key, reverse=None):
-        """ Order sub-objects by key and direction.
-        """
+        # Order sub-objects by key and direction.
         ids = [ id for id, obj in sort( self.objectItems(),
                                         ( (key, 'cmp', 'asc'), ) ) ]
         if reverse:
@@ -218,8 +212,7 @@ class OrderSupport(object):
     security.declareProtected(access_contents_information,
                               'getObjectPosition')
     def getObjectPosition(self, id):
-        """ Get the position of an object by its id.
-        """
+        # Get the position of an object by its id.
         ids = self.objectIds()
         if id in ids:
             return ids.index(id)
@@ -227,22 +220,19 @@ class OrderSupport(object):
 
     security.declareProtected(manage_properties, 'moveObjectToPosition')
     def moveObjectToPosition(self, id, position, suppress_events=False):
-        """ Move specified object to absolute position.
-        """
+        # Move specified object to absolute position.
         delta = position - self.getObjectPosition(id)
         return self.moveObjectsByDelta(id, delta,
                                        suppress_events=suppress_events)
 
     security.declareProtected(access_contents_information, 'getDefaultSorting')
     def getDefaultSorting(self):
-        """ Get default sorting key and direction.
-        """
+        # Get default sorting key and direction.
         return self._default_sort_key, self._default_sort_reverse
 
     security.declareProtected(manage_properties, 'setDefaultSorting')
     def setDefaultSorting(self, key, reverse):
-        """ Set default sorting key and direction.
-        """
+        # Set default sorting key and direction.
         self._default_sort_key = key
         self._default_sort_reverse = reverse and 1 or 0
 

src/OFS/PropertySheets.py

@@ -114,8 +114,8 @@ class PropertySheet(Traversable, Persistent, Implicit):
     __reserved_ids= ('values','items')
 
     def property_extensible_schema__(self):
-        """Return a flag indicating whether new properties may be
-        added or removed."""
+        # Return a flag indicating whether new properties may be
+        # added or removed.
         return self._extensible
 
     def __init__(self, id, md=None):
@@ -171,8 +171,8 @@ class PropertySheet(Traversable, Persistent, Implicit):
 
     security.declareProtected(access_contents_information, 'getPropertyType')
     def getPropertyType(self, id):
-        """Get the type of property 'id', returning None if no
-           such property exists"""
+        # Get the type of property 'id', returning None if no
+        # such property exists.
         pself=self.p_self()
         for md in pself._properties:
             if md['id']==id:
@@ -707,7 +707,7 @@ class PropertySheets(Traversable, Implicit, Tabs):
                 raise BadRequest(
                     'attempt to delete undeletable property sheet: ' + id)
             self.delPropertySheet(id)
-        if REQUEST is not None: 
+        if REQUEST is not None:
             REQUEST.RESPONSE.redirect('%s/manage' % self.absolute_url())
 
     def __len__(self):

src/OFS/SimpleItem.py

@@ -250,7 +250,7 @@ class Item(Base,
                 try:
                     strv = repr(error_value) # quotes tainted strings
                 except:
-                    strv = ('<unprintable %s object>' %
+                    strv = ('<unprintable %s object>' % 
                             str(type(error_value).__name__))
                 v = strv + (
                     (" (Also, the following error occurred while attempting "
@@ -410,13 +410,13 @@ class Item_w__name__(Item):
         self.__name__=id
 
     def getPhysicalPath(self):
-        # Get the physical path of the object.
-        #
-        # Returns a path (an immutable sequence of strings) that can be used to
-        # access this object again later, for example in a copy/paste
-        # operation.  getPhysicalRoot() and getPhysicalPath() are designed to
-        # operate together.
+        """Get the physical path of the object.
 
+        Returns a path (an immutable sequence of strings) that can be used to
+        access this object again later, for example in a copy/paste operation.
+        getPhysicalRoot() and getPhysicalPath() are designed to operate
+        together.
+        """
         path = (self.__name__,)
 
         p = aq_parent(aq_inner(self))

src/OFS/ZDOM.py

@@ -180,27 +180,21 @@ class Document(Explicit, Node):
 
     security.declareProtected(access_contents_information, 'getImplementation')
     def getImplementation(self):
-        """
-        The DOMImplementation object that handles this document.
-        """
+        # The DOMImplementation object that handles this document.
         return DOMImplementation()
 
     security.declareProtected(access_contents_information, 'getDoctype')
     def getDoctype(self):
-        """
-        The Document Type Declaration associated with this document.
-        For HTML documents as well as XML documents without
-        a document type declaration this returns null.
-        """
+        # The Document Type Declaration associated with this document.
+        # For HTML documents as well as XML documents without
+        # a document type declaration this returns null.
         return None
 
     security.declareProtected(access_contents_information,
                               'getDocumentElement')
     def getDocumentElement(self):
-        """
-        This is a convenience attribute that allows direct access to
-        the child node that is the root element of the document.
-        """
+        # This is a convenience attribute that allows direct access to
+        # the child node that is the root element of the document.
         return aq_parent(self)
 
     # Node Methods
@@ -368,12 +362,11 @@ class Element(Node):
     security.declareProtected(access_contents_information,
                               'getElementsByTagName')
     def getElementsByTagName(self, tagname):
-        """ Returns a NodeList of all the Elements with a given tag
-        name in the order in which they would be encountered in a
-        preorder traversal of the Document tree.  Parameter: tagname
-        The name of the tag to match (* = all tags). Return Value: A new
-        NodeList object containing all the matched Elements.
-        """
+        # Returns a NodeList of all the Elements with a given tag
+        # name in the order in which they would be encountered in a
+        # preorder traversal of the Document tree.  Parameter: tagname
+        # The name of the tag to match (* = all tags). Return Value: A new
+        # NodeList object containing all the matched Elements.
         nodeList = []
         for child in self.objectValues():
             if (child.getNodeType()==ELEMENT_NODE and \
@@ -477,12 +470,12 @@ class NodeList:
         return self._data[index]
 
     def item(self, index):
-        """Returns the index-th item in the collection"""
+        # Returns the index-th item in the collection.
         try: return self._data[index]
         except IndexError: return None
 
     def getLength(self):
-        """The length of the NodeList"""
+        # The length of the NodeList.
         return len(self._data)
 
     __len__=getLength
@@ -506,7 +499,7 @@ class NamedNodeMap:
         self._data = data
 
     def item(self, index):
-        """Returns the index-th item in the map"""
+        # Returns the index-th item in the map.
         try: return self._data.values()[index]
         except IndexError: return None
 
@@ -517,17 +510,16 @@ class NamedNodeMap:
             return self._data[key]
 
     def getLength(self):
-        """The length of the NodeList"""
+        # The length of the NodeList.
         return len(self._data)
 
     __len__ = getLength
 
     def getNamedItem(self, name):
-        """Retrieves a node specified by name. Parameters:
-        name Name of a node to retrieve. Return Value A Node (of any
-        type) with the specified name, or None if the specified name
-        did not identify any node in the map.
-        """
+        # Retrieves a node specified by name. Parameters:
+        # name Name of a node to retrieve. Return Value A Node (of any
+        # type) with the specified name, or None if the specified name
+        # did not identify any node in the map.
         if self._data.has_key(name):
             return self._data[name]
         return None
@@ -549,7 +541,7 @@ class Attr(Implicit, Node):
         return self.name
 
     def getName(self):
-        """Returns the name of this attribute."""
+        # Returns the name of this attribute.
         return self.name
 
     def getNodeValue(self):
@@ -561,6 +553,6 @@ class Attr(Implicit, Node):
         return ATTRIBUTE_NODE
 
     def getSpecified(self):
-        """If this attribute was explicitly given a value in the
-        original document, this is true; otherwise, it is false."""
+        # If this attribute was explicitly given a value in the
+        # original document, this is true; otherwise, it is false.
         return self.specified