[Zope-Checkins] SVN: Zope/branches/2.13/src/ replace map/lambda with list comprehension
Nikolay Kim
fafhrd91 at gmail.com
Mon Jul 18 14:41:35 EDT 2011
Log message for revision 122279:
replace map/lambda with list comprehension
Changed:
U Zope/branches/2.13/src/App/ApplicationManager.py
U Zope/branches/2.13/src/App/CacheManager.py
U Zope/branches/2.13/src/OFS/Cache.py
U Zope/branches/2.13/src/OFS/CopySupport.py
U Zope/branches/2.13/src/OFS/ObjectManager.py
U Zope/branches/2.13/src/OFS/PropertyManager.py
U Zope/branches/2.13/src/OFS/PropertySheets.py
U Zope/branches/2.13/src/Products/Five/browser/__init__.py
U Zope/branches/2.13/src/ZPublisher/BaseRequest.py
U Zope/branches/2.13/src/ZPublisher/HTTPRequest.py
U Zope/branches/2.13/src/webdav/Resource.py
-=-
Modified: Zope/branches/2.13/src/App/ApplicationManager.py
===================================================================
--- Zope/branches/2.13/src/App/ApplicationManager.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/App/ApplicationManager.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -464,7 +464,7 @@
introduced in 2.4.
"""
- meta_types = map(lambda x: x.get('meta_type', None), self._objects)
+ meta_types = [x.get('meta_type', None) for x in self._objects]
if not self.DavLocks.meta_type in meta_types:
Modified: Zope/branches/2.13/src/App/CacheManager.py
===================================================================
--- Zope/branches/2.13/src/App/CacheManager.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/App/CacheManager.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -102,8 +102,7 @@
if REQUEST is not None:
# format as text
REQUEST.RESPONSE.setHeader('Content-Type', 'text/plain')
- return '\n'.join(map(lambda (name, count): '%6d %s' %
- (count, name), detail))
+ return '\n'.join('%6d %s'%(count, name) for count, name in detail)
else:
# raw
return detail
@@ -115,8 +114,7 @@
detail = self._getDB().cacheExtremeDetail()
if REQUEST is not None:
# sort the list.
- lst = map(lambda dict: ((dict['conn_no'], dict['oid']), dict),
- detail)
+ lst = [((dict['conn_no'], dict['oid']), dict) for dict in detail]
# format as text.
res = [
'# Table shows connection number, oid, refcount, state, '
Modified: Zope/branches/2.13/src/OFS/Cache.py
===================================================================
--- Zope/branches/2.13/src/OFS/Cache.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/OFS/Cache.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -439,7 +439,7 @@
ids = getVerifiedManagerIds(container)
id = self.getId()
if id in ids:
- manager_ids = filter(lambda s, id=id: s != id, ids)
+ manager_ids = [s for s in ids if s != id]
if manager_ids:
setattr(container, ZCM_MANAGERS, manager_ids)
elif getattr(aq_base(self), ZCM_MANAGERS, None) is not None:
Modified: Zope/branches/2.13/src/OFS/CopySupport.py
===================================================================
--- Zope/branches/2.13/src/OFS/CopySupport.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/OFS/CopySupport.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -92,7 +92,7 @@
return self._getOb(REQUEST['ids'][0])
def manage_CopyContainerAllItems(self, REQUEST):
- return map(lambda i, s=self: s._getOb(i), tuple(REQUEST['ids']))
+ return [self._getOb(i) for i in REQUEST['ids']]
security.declareProtected(delete_objects, 'manage_cutObjects')
def manage_cutObjects(self, ids=None, REQUEST=None):
Modified: Zope/branches/2.13/src/OFS/ObjectManager.py
===================================================================
--- Zope/branches/2.13/src/OFS/ObjectManager.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/OFS/ObjectManager.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -436,7 +436,7 @@
def objectMap(self):
# Return a tuple of mappings containing subobject meta-data
- return tuple(map(lambda dict: dict.copy(), self._objects))
+ return tuple(d.copy() for d in self._objects)
def objectIds_d(self, t=None):
if hasattr(self, '_reserved_names'): n=self._reserved_names
@@ -700,7 +700,7 @@
globbing = REQUEST.environ.get('GLOBBING','')
if globbing :
- files = filter(lambda x,g=globbing: fnmatch.fnmatch(x[0],g), files)
+ files = [x for x in files if fnmatch.fnmatch(x[0],globbing)]
files.sort()
Modified: Zope/branches/2.13/src/OFS/PropertyManager.py
===================================================================
--- Zope/branches/2.13/src/OFS/PropertyManager.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/OFS/PropertyManager.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -212,27 +212,26 @@
if not self.hasProperty(id):
raise ValueError, 'The property %s does not exist' % escape(id)
self._delPropValue(id)
- self._properties=tuple(filter(lambda i, n=id: i['id'] != n,
- self._properties))
+ self._properties=tuple(i for i in self._properties if i['id'] != id)
security.declareProtected(access_contents_information, 'propertyIds')
def propertyIds(self):
"""Return a list of property ids.
"""
- return map(lambda i: i['id'], self._properties)
+ return [i['id'] for i in self._properties]
security.declareProtected(access_contents_information, 'propertyValues')
def propertyValues(self):
"""Return a list of actual property objects.
"""
- return map(lambda i,s=self: getattr(s,i['id']), self._properties)
+ return [getattr(self, i['id']) for i in self._properties]
security.declareProtected(access_contents_information, 'propertyItems')
def propertyItems(self):
"""Return a list of (id,property) tuples.
"""
- return map(lambda i,s=self: (i['id'],getattr(s,i['id'])),
- self._properties)
+ return [(i['id'], getattr(self, i['id'])) for i in self._properties]
+
def _propertyMap(self):
"""Return a tuple of mappings, giving meta-data for properties.
"""
@@ -244,7 +243,7 @@
Return copies of the real definitions for security.
"""
- return tuple(map(lambda dict: dict.copy(), self._propertyMap()))
+ return tuple(dict.copy() for dict in self._propertyMap())
security.declareProtected(access_contents_information, 'propertyLabel')
def propertyLabel(self, id):
Modified: Zope/branches/2.13/src/OFS/PropertySheets.py
===================================================================
--- Zope/branches/2.13/src/OFS/PropertySheets.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/OFS/PropertySheets.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -260,25 +260,22 @@
raise BadRequest, '%s cannot be deleted.' % escape(id)
delattr(vself, id)
pself=self.p_self()
- pself._properties=tuple(filter(lambda i, n=id: i['id'] != n,
- pself._properties))
+ pself._properties=tuple(i for i in pself._properties if i['id'] != id)
security.declareProtected(access_contents_information, 'propertyIds')
def propertyIds(self):
# Return a list of property ids.
- return map(lambda i: i['id'], self._propertyMap())
+ return [i['id'] for i in self._propertyMap()]
security.declareProtected(access_contents_information, 'propertyValues')
def propertyValues(self):
# Return a list of property values.
- return map(lambda i, s=self: s.getProperty(i['id']),
- self._propertyMap())
+ return [self.getProperty(i['id']) for i in self._propertyMap()]
security.declareProtected(access_contents_information, 'propertyItems')
def propertyItems(self):
# Return a list of (id, property) tuples.
- return map(lambda i, s=self: (i['id'], s.getProperty(i['id'])),
- self._propertyMap())
+ return [(i['id'], self.getProperty(i['id'])) for i in self._propertyMap()]
security.declareProtected(access_contents_information, 'propertyInfo')
def propertyInfo(self, id):
@@ -294,7 +291,7 @@
security.declareProtected(access_contents_information, 'propertyMap')
def propertyMap(self):
# Returns a secure copy of the property definitions.
- return tuple(map(lambda dict: dict.copy(), self._propertyMap()))
+ return tuple(dict.copy() for dict in self._propertyMap())
def _propdict(self):
dict={}
@@ -329,8 +326,7 @@
attrs=item.get('meta', {}).get('__xml_attrs__', None)
if attrs is not None:
# It's a xml property. Don't escape value.
- attrs=map(lambda n: ' %s="%s"' % n, attrs.items())
- attrs=''.join(attrs)
+ attrs=''.join(' %s="%s"' % n for n in attrs.items())
else:
# It's a non-xml property. Escape value.
attrs=''
@@ -383,8 +379,7 @@
attrs=item.get('meta', {}).get('__xml_attrs__', None)
if attrs is not None:
# It's a xml property. Don't escape value.
- attrs=map(lambda n: ' %s="%s"' % n, attrs.items())
- attrs=''.join(attrs)
+ attrs=''.join(' %s="%s"' % n for n in attrs.items())
else:
# It's a non-xml property. Escape value.
attrs=''
@@ -541,7 +536,7 @@
return self.pm
def propertyMap(self):
- return map(lambda dict: dict.copy(), self._propertyMap())
+ return [dict.copy() for dict in self._propertyMap()]
def dav__creationdate(self):
return iso8601_date(43200.0)
@@ -650,7 +645,7 @@
security.declareProtected(access_contents_information, 'values')
def values(self):
propsets=self.__propsets__()
- return map(lambda n, s=self: n.__of__(s), propsets)
+ return [n.__of__(self) for n in propsets]
security.declareProtected(access_contents_information, 'items')
def items(self):
Modified: Zope/branches/2.13/src/Products/Five/browser/__init__.py
===================================================================
--- Zope/branches/2.13/src/Products/Five/browser/__init__.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/Products/Five/browser/__init__.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -25,7 +25,8 @@
# Use an explicit __init__ to work around problems with magically inserted
# super classes when using BrowserView as a base for viewlets.
def __init__(self, context, request):
- zope.publisher.browser.BrowserView.__init__(self, context, request)
+ self.context = context
+ self.request = request
# Classes which are still based on Acquisition and access
# self.context in a method need to call aq_inner on it, or get a
Modified: Zope/branches/2.13/src/ZPublisher/BaseRequest.py
===================================================================
--- Zope/branches/2.13/src/ZPublisher/BaseRequest.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/ZPublisher/BaseRequest.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -312,7 +312,7 @@
def __str__(self):
L1 = self.items()
L1.sort()
- return '\n'.join(map(lambda item: "%s:\t%s" % item, L1))
+ return '\n'.join("%s:\t%s" % item for item in L1)
__repr__=__str__
Modified: Zope/branches/2.13/src/ZPublisher/HTTPRequest.py
===================================================================
--- Zope/branches/2.13/src/ZPublisher/HTTPRequest.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/ZPublisher/HTTPRequest.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -1726,14 +1726,14 @@
def __str__(self):
L1 = self.__dict__.items()
L1.sort()
- return ", ".join(map(lambda item: "%s: %s" % item, L1))
+ return ", ".join("%s: %s" % item for item in L1)
def __repr__(self):
#return repr( self.__dict__ )
L1 = self.__dict__.items()
L1.sort()
return '{%s}' % ', '.join(
- map(lambda item: "'%s': %s" % (item[0], repr(item[1])), L1))
+ "'%s': %s" % (item[0], repr(item[1])) for item in L1)
def __cmp__(self, other):
return (cmp(type(self), type(other)) or
Modified: Zope/branches/2.13/src/webdav/Resource.py
===================================================================
--- Zope/branches/2.13/src/webdav/Resource.py 2011-07-18 12:41:24 UTC (rev 122278)
+++ Zope/branches/2.13/src/webdav/Resource.py 2011-07-18 18:41:34 UTC (rev 122279)
@@ -149,7 +149,6 @@
# the final part of the URL (ie '/a/b/foo.html' becomes '/a/b/')
if col: url = url[:url.rfind('/')+1]
- havetag = lambda x, self=self: self.wl_hasLock(x)
found = 0; resourcetagged = 0
taglist = IfParser(ifhdr)
for tag in taglist:
@@ -157,7 +156,7 @@
if not tag.resource:
# There's no resource (url) with this tag
tag_list = map(tokenFinder, tag.list)
- wehave = filter(havetag, tag_list)
+ wehave = [tag for tag in tag_list if self.wl_hasLock(tag)]
if not wehave: continue
if tag.NOTTED: continue
@@ -168,7 +167,7 @@
elif urlbase(tag.resource) == url:
resourcetagged = 1
tag_list = map(tokenFinder, tag.list)
- wehave = filter(havetag, tag_list)
+ wehave = [tag for tag in tag_list if self.wl_hasLock(tag)]
if not wehave: continue
if tag.NOTTED: continue
More information about the Zope-Checkins
mailing list