[open-ils-commits] r7794 - trunk/Open-ILS/src/python/oils/utils

svn at svn.open-ils.org svn at svn.open-ils.org
Sun Sep 16 13:53:47 EDT 2007


Author: erickson
Date: 2007-09-16 13:45:22 -0400 (Sun, 16 Sep 2007)
New Revision: 7794

Modified:
   trunk/Open-ILS/src/python/oils/utils/csedit.py
Log:
rolling back unnecessary use of custom replace method

Modified: trunk/Open-ILS/src/python/oils/utils/csedit.py
===================================================================
--- trunk/Open-ILS/src/python/oils/utils/csedit.py	2007-09-16 17:44:20 UTC (rev 7793)
+++ trunk/Open-ILS/src/python/oils/utils/csedit.py	2007-09-16 17:45:22 UTC (rev 7794)
@@ -15,7 +15,6 @@
 
 from osrf.log import *
 from osrf.json import *
-from oils.utils.utils import replace
 from oils.utils.idl import oilsGetIDLParser
 from osrf.ses import osrfClientSession
 from oils.const import *
@@ -138,11 +137,11 @@
       method = "%s.direct.%s.%s" % (self.app, type, action)
 
       if options.get('idlist'):
-         method = replace(method, 'search', 'id_list')
+         method = method.replace('search', 'id_list')
          del options['idlist']
 
       if action == 'search':
-         method = replace(method, '$', '.atomic')
+         method = method.replace('$', '.atomic')
 
       params = [arg];
       if len(options.keys()):
@@ -163,8 +162,8 @@
    for k, fm in obj.iteritems():
       for action in ACTIONS:
 
-         fmname = replace(fm['fieldmapper'], '::', '_')
-         type = replace(fm['fieldmapper'], '::', '.')
+         fmname = fm['fieldmapper'].replace('::', '_')
+         type = fm['fieldmapper'].replace('::', '.')
          name = "%s_%s" % (action, fmname)
 
          str = 'def %s(self, arg, **options):\n' % name



More information about the open-ils-commits mailing list