Log message for revision 68835: Replace bulk of uses of 'zLOG' with equivalent Python 'logging' module usage. Exceptions: - 'ZServer' code which uses the 'register_subsystem' stuff. - Code which imports custom log levels (BLATHER, TRACE), now pulled from 'zLOG.EventLogger'. - Test jigs which play with zLOG's 'write_log'. - Usage in 'svn:externals' modules (ZODB, etc.)
Changed: U Zope/branches/2.9/lib/python/AccessControl/ImplPython.py U Zope/branches/2.9/lib/python/AccessControl/SecurityInfo.py U Zope/branches/2.9/lib/python/App/ApplicationManager.py U Zope/branches/2.9/lib/python/App/Hotfixes.py U Zope/branches/2.9/lib/python/App/ProductContext.py U Zope/branches/2.9/lib/python/App/RefreshFuncs.py U Zope/branches/2.9/lib/python/App/special_dtml.py U Zope/branches/2.9/lib/python/OFS/Cache.py U Zope/branches/2.9/lib/python/OFS/ObjectManager.py U Zope/branches/2.9/lib/python/OFS/Uninstalled.py U Zope/branches/2.9/lib/python/Products/PageTemplates/ZopePageTemplate.py U Zope/branches/2.9/lib/python/Products/PluginIndexes/DateIndex/DateIndex.py U Zope/branches/2.9/lib/python/Products/PluginIndexes/TextIndex/TextIndex.py U Zope/branches/2.9/lib/python/Products/Sessions/SessionDataManager.py U Zope/branches/2.9/lib/python/Products/Sessions/stresstests/stresstestMultiThread.py U Zope/branches/2.9/lib/python/Products/ZCatalog/ProgressHandler.py U Zope/branches/2.9/lib/python/Products/ZCatalog/ZCatalog.py U Zope/branches/2.9/lib/python/Shared/DC/ZRDB/Connection.py U Zope/branches/2.9/lib/python/TAL/XMLParser.py U Zope/branches/2.9/lib/python/ZPublisher/BeforeTraverse.py U Zope/branches/2.9/lib/python/tempstorage/TemporaryStorage.py -=- Modified: Zope/branches/2.9/lib/python/AccessControl/ImplPython.py =================================================================== --- Zope/branches/2.9/lib/python/AccessControl/ImplPython.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/AccessControl/ImplPython.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -15,13 +15,15 @@ import os import string +import logging +logger = logging.getLogger('Zope Security Policy') from Acquisition import aq_base from Acquisition import aq_parent from Acquisition import aq_inner from Acquisition import aq_acquire from ExtensionClass import Base -from zLOG import LOG, BLATHER, PROBLEM +from zLOG.EventLogger import CUSTOM_BLATHER from zope.interface import implements # This is used when a permission maps explicitly to no permission. We @@ -346,8 +348,8 @@ return 1 except TypeError: # 'roles' isn't a sequence - LOG('Zope Security Policy', PROBLEM, "'%s' passed as roles" - " during validation of '%s' is not a sequence." % ( + logger.warn("'%s' passed as roles" + " during validation of '%s' is not a sequence." % ( `roles`, name)) raise @@ -804,7 +806,7 @@ info.append(s + '.') text = ' '.join(info) - LOG('Zope Security Policy', BLATHER, 'Unauthorized: %s' % text) + logger.log(CUSTOM_BLATHER, 'Unauthorized: %s' % text) raise Unauthorized(text) def getUserRolesInContext(user, context): Modified: Zope/branches/2.9/lib/python/AccessControl/SecurityInfo.py =================================================================== --- Zope/branches/2.9/lib/python/AccessControl/SecurityInfo.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/AccessControl/SecurityInfo.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -39,13 +39,13 @@ """ import sys +import logging +logger = logging.getLogger('SecurityInfo') import Acquisition from AccessControl.ImplPython import _what_not_even_god_should_do -from zLOG import LOG, WARNING - # Security constants - these are imported into the AccessControl # namespace and can be referenced as AccessControl.PUBLIC etc. @@ -69,8 +69,8 @@ def _setaccess(self, names, access): for name in names: if self.names.get(name, access) != access: - LOG('SecurityInfo', WARNING, 'Conflicting security ' - 'declarations for "%s"' % name) + logger.warn('Conflicting security declarations for "%s"' + % name) self._warnings = 1 self.names[name] = access @@ -111,8 +111,8 @@ for role in roles: rdict[role] = 1 if self.roles.get(permission_name, rdict) != rdict: - LOG('SecurityInfo', WARNING, 'Conflicting default role' - 'declarations for permission "%s"' % permission_name) + logger.warn('Conflicting default role ' + 'declarations for permission "%s"' % permission_name) self._warnings = 1 self.roles[permission_name] = rdict @@ -193,8 +193,8 @@ access) if getattr(self, '_warnings', None): - LOG('SecurityInfo', WARNING, 'Class "%s" had conflicting ' - 'security declarations' % classobj.__name__) + logger.warn('Class "%s" had conflicting security declarations' + % classobj.__name__) class ClassSecurityInformation(ClassSecurityInfo): # Default policy is disallow @@ -273,8 +273,8 @@ dict['__allow_access_to_unprotected_subobjects__'] = self if getattr(self, '_warnings', None): - LOG('SecurityInfo', WARNING, 'Module "%s" had conflicting ' - 'security declarations' % dict['__name__']) + logger.warn('Module "%s" had conflicting security declarations' + % dict['__name__']) declareProtected__roles__=ACCESS_PRIVATE def declareProtected(self, permission_name, *names): Modified: Zope/branches/2.9/lib/python/App/ApplicationManager.py =================================================================== --- Zope/branches/2.9/lib/python/App/ApplicationManager.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/App/ApplicationManager.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -15,6 +15,9 @@ __version__='$Revision: 1.94 $'[11:-2] import sys,os,time,Globals, Acquisition, os, Undo +import logging +logger = logging.getLogger('ApplicationManager') + from Globals import DTMLFile from OFS.ObjectManager import ObjectManager from OFS.Folder import Folder @@ -31,7 +34,6 @@ from zExceptions import Redirect from Products.PageTemplates.PageTemplateFile import PageTemplateFile from cgi import escape -import zLOG import Lifetime try: import thread @@ -391,8 +393,7 @@ user = '"%s"' % getSecurityManager().getUser().getUserName() except: user = 'unknown user' - zLOG.LOG("ApplicationManager", zLOG.INFO, - "Restart requested by %s" % user) + logger.info("Restart requested by %s" % user) #for db in Globals.opened: db.close() Lifetime.shutdown(1) return """<html> @@ -407,8 +408,7 @@ user = '"%s"' % getSecurityManager().getUser().getUserName() except: user = 'unknown user' - zLOG.LOG("ApplicationManager", zLOG.INFO, - "Shutdown requested by %s" % user) + logger.info("Shutdown requested by %s" % user) #for db in Globals.opened: db.close() Lifetime.shutdown(0) return """<html> Modified: Zope/branches/2.9/lib/python/App/Hotfixes.py =================================================================== --- Zope/branches/2.9/lib/python/App/Hotfixes.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/App/Hotfixes.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -11,8 +11,10 @@ # ############################################################################## +import logging +logger = logging.getLogger('Hotfixes') + from version_txt import getZopeVersion -from zLOG import LOG, INFO, WARNING merged_hotfixes = { 'Hotfix_2001-09-28': 1, @@ -33,13 +35,13 @@ def logHotfix(id, apply_hotfix): if apply_hotfix: - LOG('Hotfixes', INFO, 'Applying %s' % id) + logger.info('Applying %s' % id) elif apply_hotfix is OUTDATED_ZOPE: - LOG('Hotfixes', WARNING, 'Not applying %s. It is not designed for ' + logger.warn('Not applying %s. It is not designed for ' 'this version of Zope. Please uninstall the hotfix product.' % id) else: # ALREADY_MERGED - LOG('Hotfixes', WARNING, 'Not applying %s. The fix has already been ' + logger.warn('Not applying %s. The fix has already been ' 'merged into Zope. Please uninstall the hotfix product.' % id) Modified: Zope/branches/2.9/lib/python/App/ProductContext.py =================================================================== --- Zope/branches/2.9/lib/python/App/ProductContext.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/App/ProductContext.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -17,6 +17,8 @@ import os.path, re import stat +import logging +logger = logging.getLogger('Zope') from AccessControl.PermissionRole import PermissionRole import Globals, os, OFS.ObjectManager, OFS.misc_, Products @@ -25,7 +27,6 @@ from HelpSys import HelpTopic, APIHelpTopic from HelpSys.HelpSys import ProductHelp from FactoryDispatcher import FactoryDispatcher -from zLOG import LOG, WARNING from DateTime import DateTime from Interface.Implements import instancesOfObjectImplements from zope.interface import implementedBy @@ -312,7 +313,7 @@ try: dir_mod_time=DateTime(os.stat(path)[stat.ST_MTIME]) except OSError, (errno, text): - LOG("Zope", WARNING, '%s: %s' % (text, path)) + logger.warn('%s: %s' % (text, path)) return # test to see if nothing has changed since last registration Modified: Zope/branches/2.9/lib/python/App/RefreshFuncs.py =================================================================== --- Zope/branches/2.9/lib/python/App/RefreshFuncs.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/App/RefreshFuncs.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -17,11 +17,14 @@ import os, sys from time import time +from traceback import format_exception +import logging +logger = logging.getLogger('Refresh') + import transaction import Products from ExtensionClass import Base from Globals import PersistentMapping -from zLOG import format_exception, LOG, ERROR, INFO global_classes_timestamp = 0 products_mod_times = {} @@ -136,8 +139,8 @@ def logBadRefresh(productid): exc = sys.exc_info() try: - LOG('Refresh', ERROR, 'Exception while refreshing %s' - % productid, error=exc) + logger.error('Exception while refreshing %s' + % productid, error=exc) if hasattr(exc[0], '__name__'): error_type = exc[0].__name__ else: @@ -179,7 +182,7 @@ def performSafeRefresh(jar, productid): try: - LOG('Refresh', INFO, 'Refreshing product %s' % productid) + logger.info('Refreshing product %s' % productid) if not performRefresh(jar, productid): return 0 except: Modified: Zope/branches/2.9/lib/python/App/special_dtml.py =================================================================== --- Zope/branches/2.9/lib/python/App/special_dtml.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/App/special_dtml.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -10,10 +10,11 @@ # FOR A PARTICULAR PURPOSE # ############################################################################## +from types import InstanceType +import logging +logger = logging.getLogger('ZPublisher') import DocumentTemplate, Common, Persistence, MethodObject, Globals, os, sys -from types import InstanceType -from zLOG import LOG,WARNING from App.config import getConfiguration class HTML(DocumentTemplate.HTML,Persistence.Persistent,): @@ -175,8 +176,10 @@ try: result = render_blocks(self._v_blocks, ns) except DTReturn, v: result = v.v except AttributeError: - if type(sys.exc_value)==InstanceType and sys.exc_value.args[0]=="_v_blocks": - LOG("ZPublisher",WARNING,"DTML file '%s' could not be read" % self.raw) + if (type(sys.exc_value) == InstanceType + and sys.exc_value.args[0]=="_v_blocks"): + logger.warn("DTML file '%s' could not be read" + % self.raw) raise ValueError, ("DTML file error: " "Check logfile for details") else: Modified: Zope/branches/2.9/lib/python/OFS/Cache.py =================================================================== --- Zope/branches/2.9/lib/python/OFS/Cache.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/OFS/Cache.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -15,10 +15,12 @@ $Id$ """ import time, sys +import logging +logger = logging.getLogger('Cache') + import Globals from Globals import DTMLFile from Acquisition import aq_get, aq_acquire, aq_inner, aq_parent, aq_base -from zLOG import LOG, WARNING from AccessControl import getSecurityManager from AccessControl.Role import _isBeingUsedAsAMethod from AccessControl import Unauthorized @@ -195,8 +197,7 @@ mtime_func, default) return val except: - LOG('Cache', WARNING, 'ZCache_get() exception', - error=sys.exc_info()) + logger.warn('ZCache_get() exception', error=sys.exc_info()) return default return default @@ -213,8 +214,7 @@ c.ZCache_set(ob, data, view_name, keywords, mtime_func) except: - LOG('Cache', WARNING, 'ZCache_set() exception', - error=sys.exc_info()) + logger.warn('ZCache_set() exception', error=sys.exc_info()) def ZCacheable_invalidate(self, view_name='', REQUEST=None): '''Called after a cacheable object is edited. Causes all @@ -231,8 +231,7 @@ except: exc = sys.exc_info() try: - LOG('Cache', WARNING, 'ZCache_invalidate() exception', - error=exc) + logger.warn('ZCache_invalidate() exception', error=exc) message = 'An exception occurred: %s: %s' % exc[:2] finally: exc = None Modified: Zope/branches/2.9/lib/python/OFS/ObjectManager.py =================================================================== --- Zope/branches/2.9/lib/python/OFS/ObjectManager.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/OFS/ObjectManager.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -20,6 +20,8 @@ import sys, fnmatch, copy, os, re from cgi import escape from cStringIO import StringIO +import logging +logger = logging.getLogger('FTP') from types import StringType, UnicodeType import App.Common @@ -36,7 +38,6 @@ from webdav.Lockable import ResourceLockedError from webdav.NullResource import NullResource from zExceptions import BadRequest -from zLOG import LOG, ERROR from ZODB.POSException import ConflictError from zope.interface import implements @@ -673,8 +674,8 @@ try: stat=marshal.loads(v.manage_FTPstat(REQUEST)) except: - LOG("FTP", ERROR, "Failed to stat file '%s'" % k, - error=sys.exc_info()) + logger.error("Failed to stat file '%s'" % k, + error=sys.exc_info()) stat=None if stat is not None: out=out+((k,stat),) Modified: Zope/branches/2.9/lib/python/OFS/Uninstalled.py =================================================================== --- Zope/branches/2.9/lib/python/OFS/Uninstalled.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/OFS/Uninstalled.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -13,11 +13,13 @@ """ Objects for packages that have been uninstalled. """ -import SimpleItem, Globals, Acquisition +from thread import allocate_lock +import logging +logger = logging.getLogger('ZODB') # ??? + +import SimpleItem, Globals, Acquisition from Acquisition import Acquired import Persistence -from thread import allocate_lock -from zLOG import LOG, WARNING from cgi import escape broken_klasses={} @@ -71,8 +73,9 @@ klass.info=( 'This object\'s class was %s in module %s.' % (klass.__name__, klass.__module__)) - LOG('ZODB', WARNING, 'Could not import class %s ' - 'from module %s' % (`klass.__name__`, `klass.__module__`)) + logger.warn('Could not import class %s ' + 'from module %s' + % (`klass.__name__`, `klass.__module__`)) finally: broken_klasses_lock.release() if oid is None: return klass Modified: Zope/branches/2.9/lib/python/Products/PageTemplates/ZopePageTemplate.py =================================================================== --- Zope/branches/2.9/lib/python/Products/PageTemplates/ZopePageTemplate.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Products/PageTemplates/ZopePageTemplate.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -20,7 +20,6 @@ import os, AccessControl, Acquisition, sys, types from types import StringType from Globals import DTMLFile, ImageFile, MessageDialog, package_home -from zLOG import LOG, ERROR, INFO from OFS.SimpleItem import SimpleItem from DateTime.DateTime import DateTime from Shared.DC.Scripts.Script import Script, BindingsUI Modified: Zope/branches/2.9/lib/python/Products/PluginIndexes/DateIndex/DateIndex.py =================================================================== --- Zope/branches/2.9/lib/python/Products/PluginIndexes/DateIndex/DateIndex.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Products/PluginIndexes/DateIndex/DateIndex.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -18,6 +18,8 @@ import time from datetime import date, datetime from datetime import tzinfo, timedelta +import logging +logger = logging.getLogger('UnIndex') from types import StringType, FloatType, IntType import BTrees.Length @@ -28,7 +30,6 @@ from Globals import DTMLFile from OFS.PropertyManager import PropertyManager from ZODB.POSException import ConflictError -from zLOG import LOG, ERROR from zope.interface import implements from Products.PluginIndexes.common import safe_callable @@ -144,7 +145,7 @@ except ConflictError: raise except: - LOG('UnIndex', ERROR, + logger.error( ("Should not happen: ConvertedDate was there," " now it's not, for document with id %s" % documentId)) Modified: Zope/branches/2.9/lib/python/Products/PluginIndexes/TextIndex/TextIndex.py =================================================================== --- Zope/branches/2.9/lib/python/Products/PluginIndexes/TextIndex/TextIndex.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Products/PluginIndexes/TextIndex/TextIndex.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -18,10 +18,11 @@ import operator, warnings import re from cgi import escape +import logging +logger = logging.getLogger('TextIndex') from types import * from Globals import Persistent, DTMLFile -from zLOG import LOG, ERROR from Acquisition import Implicit from OFS.SimpleItem import SimpleItem from BTrees.IOBTree import IOBTree @@ -376,9 +377,8 @@ for wid in wids: widScores = get(wid, None) if widScores is None: - LOG('TextIndex', ERROR, - 'unindex_object tried to unindex nonexistent' - ' document, wid %s, %s' % (i,wid)) + logger.error('unindex_object tried to unindex nonexistent' + ' document, wid %s, %s' % (i,wid)) continue if type(widScores) is TupleType: del index[wid] @@ -394,9 +394,8 @@ else: del index[wid] except (KeyError, IndexError, TypeError): - LOG('TextIndex', ERROR, - 'unindex_object tried to unindex nonexistent' - ' document %s' % str(i)) + logger.error('unindex_object tried to unindex nonexistent' + ' document %s' % str(i)) def __getitem__(self, word): """Return an InvertedIndex-style result "list" Modified: Zope/branches/2.9/lib/python/Products/Sessions/SessionDataManager.py =================================================================== --- Zope/branches/2.9/lib/python/Products/Sessions/SessionDataManager.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Products/Sessions/SessionDataManager.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -12,6 +12,9 @@ ############################################################################ import re, time, sys +import logging +logger = logging.getLogger('Session Tracking') + import Globals from OFS.SimpleItem import Item from Acquisition import Implicit, Explicit, aq_base @@ -19,7 +22,7 @@ from AccessControl.Owned import Owned from AccessControl.Role import RoleManager from App.Management import Tabs -from zLOG import LOG, WARNING, BLATHER +from zLOG.EventLogger import CUSTOM_BLATHER from AccessControl import ClassSecurityInfo import SessionInterfaces from SessionPermissions import * @@ -205,7 +208,7 @@ transactions for mounted storages. """ if self.obpath is None: err = 'Session data container is unspecified in %s' % self.getId() - LOG('Session Tracking', WARNING, err) + logger.warn(err) raise SessionIdManagerErr, err try: # This should arguably use restrictedTraverse, but it @@ -215,7 +218,7 @@ # hasattr hides conflicts if DEBUG and not getattr(self, '_v_wrote_dc_type', None): args = '/'.join(self.obpath) - LOG('Session Tracking', BLATHER, + logger.log(CUSTOM_BLATHER, 'External data container at %s in use' % args) self._v_wrote_dc_type = 1 return self.unrestrictedTraverse(self.obpath) @@ -281,7 +284,7 @@ getSessionData = sdm.getSessionData except: msg = 'Session automatic traversal failed to get session data' - LOG('Session Tracking', WARNING, msg, error=sys.exc_info()) + logger.warn(msg, error=sys.exc_info()) return # set the getSessionData method in the "lazy" namespace Modified: Zope/branches/2.9/lib/python/Products/Sessions/stresstests/stresstestMultiThread.py =================================================================== --- Zope/branches/2.9/lib/python/Products/Sessions/stresstests/stresstestMultiThread.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Products/Sessions/stresstests/stresstestMultiThread.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -41,7 +41,7 @@ from OFS.Folder import Folder import sys import traceback -from zLOG import log_time +from zLOG.EventLogger import log_time sys.setcheckinterval(200) from Products.Transience.tests import fauxtime Modified: Zope/branches/2.9/lib/python/Products/ZCatalog/ProgressHandler.py =================================================================== --- Zope/branches/2.9/lib/python/Products/ZCatalog/ProgressHandler.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Products/ZCatalog/ProgressHandler.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -15,7 +15,7 @@ """ import time, sys -from zLOG import LOG, INFO +import logging from DateTime.DateTime import DateTime @@ -92,7 +92,8 @@ __implements__ = IProgressHandler def output(self, text): - LOG(self._ident, INFO, text) + logger = logging.getLogger(self._ident) + logger.info(text) class FilelogHandler(StdoutHandler): Modified: Zope/branches/2.9/lib/python/Products/ZCatalog/ZCatalog.py =================================================================== --- Zope/branches/2.9/lib/python/Products/ZCatalog/ZCatalog.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Products/ZCatalog/ZCatalog.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -37,7 +37,6 @@ import PluggableIndexInterface from Products.PluginIndexes.interfaces import IPluggableIndex from Products.PluginIndexes.TextIndex import Splitter -from zLOG import LOG from zope.interface import implements from Catalog import Catalog, CatalogError @@ -47,7 +46,7 @@ from ZCatalogIndexes import ZCatalogIndexes -LOG = logging.getLogger('Zope.ZCatalog') +logger = logging.getLogger('Zope.ZCatalog') manage_addZCatalogForm=DTMLFile('dtml/addZCatalog',globals()) @@ -304,8 +303,8 @@ except ConflictError: raise except: - LOG.error('Recataloging object at %s failed' % p, - exc_info=sys.exc_info()) + logger.error('Recataloging object at %s failed' % p, + exc_info=sys.exc_info()) if pghandler: pghandler.finish() @@ -494,8 +493,8 @@ if obj is None: obj = self.resolve_url(p, REQUEST) if obj is None: - LOG.error('reindexIndex could not resolve ' - 'an object from the uid %r.' % p) + logger.error('reindexIndex could not resolve ' + 'an object from the uid %r.' % p) else: # don't update metadata when only reindexing a single # index via the UI @@ -924,7 +923,7 @@ classes. """ - LOG.info('Start migration of indexes for %s' % self.absolute_url(1)) + logger.info('Start migration of indexes for %s' % self.absolute_url(1)) for idx in self.Indexes.objectValues(): bases = [str(name) for name in idx.__class__.__bases__] @@ -941,7 +940,7 @@ if found: idx_type = idx.meta_type idx_id = idx.getId() - LOG.info('processing index %s' % idx_id) + logger.info('processing index %s' % idx_id) indexed_attrs = getattr(idx, 'indexed_attrs', None) @@ -963,7 +962,8 @@ self.manage_reindexIndex(idx_id, REQUEST) self._migrated_280 = True - LOG.info('Finished migration of indexes for %s' % self.absolute_url(1)) + logger.info('Finished migration of indexes for %s' + % self.absolute_url(1)) if RESPONSE: RESPONSE.redirect( URL1 + Modified: Zope/branches/2.9/lib/python/Shared/DC/ZRDB/Connection.py =================================================================== --- Zope/branches/2.9/lib/python/Shared/DC/ZRDB/Connection.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/Shared/DC/ZRDB/Connection.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -15,6 +15,9 @@ $Id$''' __version__='$Revision: 1.39 $'[11:-2] +import logging +logger = logging.getLogger('Shared.DC.ZRDB.Connection') + import Globals, OFS.SimpleItem, AccessControl.Role, Acquisition, sys from DateTime import DateTime from App.Dialogs import MessageDialog @@ -24,7 +27,6 @@ from cStringIO import StringIO from Results import Results from sys import exc_info -from zLOG import LOG, ERROR from cgi import escape import DocumentTemplate, RDB from zExceptions import BadRequest @@ -68,10 +70,8 @@ if self.connection_string: try: self.connect(self.connection_string) except: - LOG('Shared.DC.ZRDB.Connection', - ERROR, - 'Error connecting to relational database.', - error=exc_info()) + logger.error('Error connecting to relational database.', + error=exc_info()) def title_and_id(self): s=Connection.inheritedAttribute('title_and_id')(self) @@ -150,10 +150,8 @@ if hasattr(self,'_v_database_connection'): self._v_database_connection.close() except: - LOG('Shared.DC.ZRDB.Connection', - ERROR, - 'Error closing relational database connection.', - error=exc_info()) + logger.error('Error closing relational database connection.', + error=exc_info()) self._v_connected='' if REQUEST is not None: return self.manage_main(self, REQUEST) Modified: Zope/branches/2.9/lib/python/TAL/XMLParser.py =================================================================== --- Zope/branches/2.9/lib/python/TAL/XMLParser.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/TAL/XMLParser.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -16,10 +16,9 @@ """ import xml.parsers.expat +import logging +logger = logging.getLogger('TAL.XMLParser') -import zLOG - - XMLParseError = xml.parsers.expat.ExpatError @@ -56,8 +55,7 @@ try: self.parser.ordered_attributes = self.ordered_attributes except AttributeError: - zLOG.LOG("TAL.XMLParser", zLOG.INFO, - "Can't set ordered_attributes") + logger.info("Can't set ordered_attributes") self.ordered_attributes = 0 for name in self.handler_names: method = getattr(self, name, None) @@ -65,8 +63,7 @@ try: setattr(p, name, method) except AttributeError: - zLOG.LOG("TAL.XMLParser", zLOG.PROBLEM, - "Can't set expat handler %s" % name) + logger.warn("Can't set expat handler %s" % name) def createParser(self, encoding=None): return xml.parsers.expat.ParserCreate(encoding, ' ') Modified: Zope/branches/2.9/lib/python/ZPublisher/BeforeTraverse.py =================================================================== --- Zope/branches/2.9/lib/python/ZPublisher/BeforeTraverse.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/ZPublisher/BeforeTraverse.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -13,9 +13,9 @@ __version__='$Revision: 1.12 $'[11:-2] """BeforeTraverse interface and helper classes""" +import logging from Acquisition import aq_base -from zLOG import LOG, ERROR import sys # Interface @@ -86,6 +86,7 @@ MultiHook calls the named hook from the class of the container, then the prior hook, then all the hooks in its list. """ + logger = logging.getLogger('MultiHook') def __init__(self, hookname, prior, defined_in_class): self._hookname = hookname self._prior = prior @@ -103,7 +104,7 @@ try: cob(container, request) except TypeError: - LOG('MultiHook', ERROR, '%s call %s failed.' % ( + self.logger.error('%s call %s failed.' % ( `self._hookname`, `cob`), error=sys.exc_info()) def add(self, cob): @@ -120,6 +121,7 @@ >>> registerBeforeTraverse(folder, NameCaller('preop'), 'XApp') """ + logger = logging.getLogger('BeforeTraverse') def __init__(self, name): self.name = name @@ -149,8 +151,6 @@ # Only catch exceptions that are likely to be logic errors. # We shouldn't catch Redirects, Unauthorizeds, etc. since # the programmer may want to raise them deliberately. - from zLOG import LOG, ERROR import sys - LOG('BeforeTraverse', ERROR, - 'Error while invoking hook: "%s"' % self.name, error= - sys.exc_info()) + self.logger.error('Error while invoking hook: "%s"' + % self.name, error=sys.exc_info()) Modified: Zope/branches/2.9/lib/python/tempstorage/TemporaryStorage.py =================================================================== --- Zope/branches/2.9/lib/python/tempstorage/TemporaryStorage.py 2006-06-24 21:59:59 UTC (rev 68834) +++ Zope/branches/2.9/lib/python/tempstorage/TemporaryStorage.py 2006-06-24 22:45:20 UTC (rev 68835) @@ -21,8 +21,10 @@ """ __version__ ='$Revision: 1.1.2.2 $'[11:-2] +import logging +logger = logging.getLogger('TemporaryStorage') -from zLOG import LOG, BLATHER +from zLOG.EventLogger import CUSTOM_BLATHER from ZODB.serialize import referencesf from ZODB import POSException from ZODB.BaseStorage import BaseStorage @@ -182,7 +184,7 @@ if version: # we allow a version to be in use although we don't # support versions in the storage. - LOG('TemporaryStorage', BLATHER, + logging.log(CUSTOM_BLATHER, ('versions in use with TemporaryStorage although Temporary' 'Storage doesnt support versions'), ) _______________________________________________ Zope-Checkins maillist - Zope-Checkins@zope.org http://mail.zope.org/mailman/listinfo/zope-checkins