diff --git a/bin/addons/__init__.py b/bin/addons/__init__.py index 43dd256055c..e18213d466f 100644 --- a/bin/addons/__init__.py +++ b/bin/addons/__init__.py @@ -140,8 +140,6 @@ def get_module_path(module): logger.notifyChannel('init', netsvc.LOG_WARNING, 'module %s: module not found' % (module,)) return False - raise IOError, 'Module not found : %s' % module - def get_module_filetree(module, dir='.'): path = get_module_path(module) @@ -193,7 +191,11 @@ def get_modules(): if name[-4:] == '.zip': name = name[:-4] return name - return map(clean, os.listdir(dir)) + + def is_really_module(name): + name = opj(dir, name) + return os.path.isdir(name) or zipfile.is_zipfile(name) + return map(clean, filter(is_really_module, os.listdir(dir))) return list(set(listdir(ad) + listdir(_ad))) @@ -545,9 +547,7 @@ def load_modules(db, force_demo=False, status=None, update_module=False): modobj = pool.get('ir.module.module') logger.notifyChannel('init', netsvc.LOG_INFO, 'updating modules list') - cr.execute("select id from ir_module_module where state in ('to install','to upgrade') and name=%s", ('base',)) - if cr.rowcount: - modobj.update_list(cr, 1) + modobj.update_list(cr, 1) mods = [k for k in tools.config['init'] if tools.config['init'][k]] if mods: diff --git a/bin/addons/base/ir/ir.xml b/bin/addons/base/ir/ir.xml index 263f5428718..c649b12cf99 100644 --- a/bin/addons/base/ir/ir.xml +++ b/bin/addons/base/ir/ir.xml @@ -1142,13 +1142,7 @@ - + @@ -1197,7 +1191,8 @@ - @@ -1223,6 +1218,7 @@ ir.actions.act_window ir.actions.server form + tree,form {'key':'server_action'} diff --git a/bin/addons/base/ir/ir_actions.py b/bin/addons/base/ir/ir_actions.py index d31a13211a5..c270557f6c5 100644 --- a/bin/addons/base/ir/ir_actions.py +++ b/bin/addons/base/ir/ir_actions.py @@ -365,21 +365,22 @@ server_object_lines() # class actions_server(osv.osv): + def fields_view_get(self, cr, user, view_id=None, view_type='form', context=None, toolbar=False): + res = super(actions_server, self).fields_view_get(cr, user, view_id, view_type, context, toolbar) + #print 'RES : ',res + return res + def _select_signals(self, cr, uid, context={}): cr.execute("select distinct t.signal as key, t.signal || ' - [ ' || w.osv || ' ] ' as val from wkf w, wkf_activity a, wkf_transition t "\ " where w.id = a.wkf_id " \ " and t.act_from = a.wkf_id " \ - " or t.act_to = a.wkf_id ") - return cr.fetchall() - - def on_trigger_obj_id(self, cr, uid, ids, context={}): - cr.execute("select distinct t.signal as key, t.signal as val from wkf w, wkf_activity a, wkf_transition t "\ - " where w.id = a.wkf_id " \ - " and t.act_from = a.wkf_id " \ - " or t.act_to = a.wkf_id " \ - " and w.osv = %s ", ('account.invoice')) - data = cr.fetchall() - return {"values":{'trigger_name':data}} + " or t.act_to = a.wkf_id and t.signal not in (null, NULL)") + result = cr.fetchall() or [] + res = [] + for rs in result: + if not rs[0] == None and not rs[1] == None: + res.append(rs) + return res _name = 'ir.actions.server' _table = 'ir_act_server' @@ -396,7 +397,6 @@ class actions_server(osv.osv): ('object_write','Write Object'), ('other','Multi Actions'), ], 'Action State', required=True, size=32), - 'code': fields.text('Python Code'), 'sequence': fields.integer('Sequence'), 'model_id': fields.many2one('ir.model', 'Object', required=True), 'action_id': fields.many2one('ir.actions.actions', 'Client Action'), @@ -417,16 +417,7 @@ class actions_server(osv.osv): _defaults = { 'state': lambda *a: 'dummy', 'type': lambda *a: 'ir.actions.server', - 'sequence': lambda *a: 0, - 'code': lambda *a: """# You can use the following variables -# - object -# - object2 -# - time -# - cr -# - uid -# - ids -# If you plan to return an action, assign: action = {...} -""", + 'sequence': lambda *a: 5, } @@ -568,7 +559,7 @@ class actions_server(osv.osv): if exp.type == 'equation': obj_pool = self.pool.get(action.model_id.model) obj = obj_pool.browse(cr, uid, context['active_id'], context=context) - expr = eval(euq, {'context':context, 'object': obj}) + expr = eval(euq, {'context':context, 'object': obj, 'time':time}) else: expr = exp.value res[exp.col1.name] = expr @@ -592,7 +583,7 @@ class actions_server(osv.osv): if exp.type == 'equation': obj_pool = self.pool.get(action.model_id.model) obj = obj_pool.browse(cr, uid, context['active_id'], context=context) - expr = eval(euq, {'context':context, 'object': obj}) + expr = eval(euq, {'context':context, 'object': obj, 'time':time}) else: expr = exp.value res[exp.col1.name] = expr diff --git a/bin/addons/base/module/module.py b/bin/addons/base/module/module.py index 439cf5b57a6..376e80450f9 100644 --- a/bin/addons/base/module/module.py +++ b/bin/addons/base/module/module.py @@ -292,10 +292,7 @@ class module(osv.osv): res = [0, 0] # [update, add] # iterate through installed modules and mark them as being so - for name in addons.get_modules(): - mod_name = name - if name[-4:]=='.zip': - mod_name=name[:-4] + for mod_name in addons.get_modules(): ids = self.search(cr, uid, [('name','=',mod_name)]) if ids: id = ids[0] @@ -304,8 +301,7 @@ class module(osv.osv): if terp.get('installable', True) and mod.state == 'uninstallable': self.write(cr, uid, id, {'state': 'uninstalled'}) if parse_version(terp.get('version', '')) > parse_version(mod.latest_version or ''): - self.write(cr, uid, id, { - 'url': ''}) + self.write(cr, uid, id, { 'url': ''}) res[0] += 1 self.write(cr, uid, id, { 'description': terp.get('description', ''), @@ -314,28 +310,16 @@ class module(osv.osv): 'website': terp.get('website', ''), 'license': terp.get('license', 'GPL-2'), }) - cr.execute('DELETE FROM ir_module_module_dependency\ - WHERE module_id = %s', (id,)) - self._update_dependencies(cr, uid, ids[0], terp.get('depends', - [])) - self._update_category(cr, uid, ids[0], terp.get('category', - 'Uncategorized')) + cr.execute('DELETE FROM ir_module_module_dependency WHERE module_id = %s', (id,)) + self._update_dependencies(cr, uid, ids[0], terp.get('depends', [])) + self._update_category(cr, uid, ids[0], terp.get('category', 'Uncategorized')) continue - terp_file = addons.get_module_resource(name, '__terp__.py') - mod_path = addons.get_module_path(name) - if mod_path and (os.path.isdir(mod_path) or os.path.islink(mod_path) or zipfile.is_zipfile(mod_path)): + mod_path = addons.get_module_path(mod_name) + if mod_path: terp = self.get_module_info(mod_name) if not terp or not terp.get('installable', True): continue - #if not os.path.isfile( mod_path ): - # import imp - # path = imp.find_module(mod_name, [addons.ad, addons._ad]) - # imp.load_module(name, *path) - #else: - # import zipimport - # zimp = zipimport.zipimporter(mod_path) - # zimp.load_module(mod_name) id = self.create(cr, uid, { 'name': mod_name, 'state': 'uninstalled', @@ -349,8 +333,6 @@ class module(osv.osv): self._update_dependencies(cr, uid, id, terp.get('depends', [])) self._update_category(cr, uid, id, terp.get('category', 'Uncategorized')) - #import socket - #socket.setdefaulttimeout(10) for repository in robj.browse(cr, uid, robj.search(cr, uid, [])): try: index_page = urllib.urlopen(repository.url).read() @@ -363,9 +345,7 @@ class module(osv.osv): modules = re.findall(repository.filter, index_page, re.I+re.M) mod_sort = {} for m in modules: - name = m[0] - version = m[1] - extension = m[-1] + name, version, extension = m[0], m[1], m[-1] if version == 'x': # 'x' version was a mistake version = '0' if name in mod_sort: @@ -390,16 +370,13 @@ class module(osv.osv): if installed_version == 'x': # 'x' version was a mistake installed_version = '0' if parse_version(version) > parse_version(installed_version): - self.write(cr, uid, id, { - 'url': url - }) + self.write(cr, uid, id, { 'url': url }) res[0] += 1 published_version = self.read(cr, uid, id, ['published_version'])['published_version'] if published_version == 'x' or not published_version: published_version = '0' if parse_version(version) > parse_version(published_version): - self.write(cr, uid, id, - {'published_version': version}) + self.write(cr, uid, id, {'published_version': version}) return res def download(self, cr, uid, ids, download=True, context=None): diff --git a/bin/netsvc.py b/bin/netsvc.py index b969ee2ff9f..73cbdfceeff 100644 --- a/bin/netsvc.py +++ b/bin/netsvc.py @@ -263,6 +263,7 @@ class xmlrpc(object): class GenericXMLRPCRequestHandler: def _dispatch(self, method, params): +# print 'TERP-CALL : ',method, params import traceback try: n = self.path.split("/")[-1] diff --git a/bin/osv/fields.py b/bin/osv/fields.py index b3ac6164b84..544bd9b3fee 100644 --- a/bin/osv/fields.py +++ b/bin/osv/fields.py @@ -658,7 +658,6 @@ class related(function): return [(self._arg[0], 'in', sarg)] def _fnct_write(self,obj,cr, uid, ids, field_name, values, args, context=None): - print 'Related Write', obj._name if values and field_name: self._field_get2(cr, uid, obj, context) relation = obj._name diff --git a/bin/osv/orm.py b/bin/osv/orm.py index 25d2f310b8f..1375794aaf3 100644 --- a/bin/osv/orm.py +++ b/bin/osv/orm.py @@ -709,8 +709,7 @@ class orm_template(object): translation_obj = self.pool.get('ir.translation') model_access_obj = self.pool.get('ir.model.access') for parent in self._inherits: - res.update(self.pool.get(parent).fields_get(cr, user, fields, - context)) + res.update(self.pool.get(parent).fields_get(cr, user, fields, context)) for f in self._columns.keys(): if fields and f not in fields: continue @@ -727,12 +726,10 @@ class orm_template(object): and getattr(self._columns[f], arg): res[f][arg] = getattr(self._columns[f], arg) - res_trans = translation_obj._get_source(cr, user, - self._name + ',' + f, 'field', context.get('lang', False) or 'en_US') + res_trans = translation_obj._get_source(cr, user, self._name + ',' + f, 'field', context.get('lang', False) or 'en_US') if res_trans: res[f]['string'] = res_trans - help_trans = translation_obj._get_source(cr, user, - self._name + ',' + f, 'help', context.get('lang', False) or 'en_US') + help_trans = translation_obj._get_source(cr, user, self._name + ',' + f, 'help', context.get('lang', False) or 'en_US') if help_trans: res[f]['help'] = help_trans @@ -744,9 +741,7 @@ class orm_template(object): for (key, val) in sel: val2 = None if val: - val2 = translation_obj._get_source(cr, user, - self._name + ',' + f, 'selection', - context.get('lang', False) or 'en_US', val) + val2 = translation_obj._get_source(cr, user, self._name + ',' + f, 'selection', context.get('lang', False) or 'en_US', val) sel2.append((key, val2 or val)) sel = sel2 res[f]['selection'] = sel diff --git a/bin/sql_db.py b/bin/sql_db.py index 08b6a25da79..3f4e23cc392 100644 --- a/bin/sql_db.py +++ b/bin/sql_db.py @@ -117,7 +117,7 @@ class Cursor(object): if self.sql_log: now = mdt.now() - + res = self._obj.execute(query, p or None) if self.sql_log: @@ -213,15 +213,15 @@ class PoolManager(object): logger = netsvc.Logger() try: logger.notifyChannel('dbpool', netsvc.LOG_INFO, 'Connecting to %s' % (db_name,)) - PoolManager._pools[db_name] = ConnectionPool(ThreadedConnectionPool(0, PoolManager.maxconn, PoolManager.dsn(db_name)), db_name) + PoolManager._pools[db_name] = ConnectionPool(ThreadedConnectionPool(1, PoolManager.maxconn, PoolManager.dsn(db_name)), db_name) except Exception, e: - logger.notifyChannel('dbpool', netsvc.LOG_CRITICAL, 'Unable to connect to %s: %r' % (db_name, e)) + logger.notifyChannel('dbpool', netsvc.LOG_CRITICAL, 'Unable to connect to %s: %s' % (db_name, e.message)) raise return PoolManager._pools[db_name] get = staticmethod(get) def close(db_name): - if db_name is PoolManager._pools: + if db_name in PoolManager._pools: logger.notifyChannel('dbpool', netsvc.LOG_INFO, 'Closing all connections to %s' % (db_name,)) PoolManager._pools[db_name].closeall() del PoolManager._pools[db_name] diff --git a/bin/tools/convert.py b/bin/tools/convert.py index aa24ea03cc2..d76e6ac7a43 100644 --- a/bin/tools/convert.py +++ b/bin/tools/convert.py @@ -284,11 +284,13 @@ form: module.record_id""" % (xml_id,) id = self.pool.get('ir.model.data')._update(cr, self.uid, "ir.actions.report.xml", self.module, res, xml_id, noupdate=self.isnoupdate(data_node), mode=self.mode) self.idref[xml_id] = int(id) + + if not rec.hasAttribute('menu') or eval(rec.getAttribute('menu')): keyword = str(rec.getAttribute('keyword') or 'client_print_multi') keys = [('action',keyword),('res_model',res['model'])] value = 'ir.actions.report.xml,'+str(id) - replace = rec.hasAttribute('replace') and rec.getAttribute("replace") + replace = rec.hasAttribute('replace') and rec.getAttribute("replace") or True self.pool.get('ir.model.data').ir_set(cr, self.uid, 'action', keyword, res['name'], [res['model']], value, replace=replace, isobject=True, xml_id=xml_id) return False @@ -421,7 +423,7 @@ form: module.record_id""" % (xml_id,) keyword = 'client_action_relate' keys = [('action', keyword), ('res_model', res_model)] value = 'ir.actions.act_window,'+str(id) - replace = rec.hasAttribute('replace') and rec.getAttribute('replace') + replace = rec.hasAttribute('replace') and rec.getAttribute('replace') or True self.pool.get('ir.model.data').ir_set(cr, self.uid, 'action', keyword, xml_id, [src_model], value, replace=replace, isobject=True, xml_id=xml_id) # TODO add remove ir.model.data return False diff --git a/bin/tools/misc.py b/bin/tools/misc.py index 8d2a821f5d7..fbdaf44c55b 100644 --- a/bin/tools/misc.py +++ b/bin/tools/misc.py @@ -531,12 +531,14 @@ def is_hashable(h): except TypeError: return False -# -# Use it as a decorator of the function you plan to cache -# Timeout: 0 = no timeout, otherwise in seconds -# class cache(object): + """ + Use it as a decorator of the function you plan to cache + Timeout: 0 = no timeout, otherwise in seconds + """ + def __init__(self, timeout=10000, skiparg=2, multi=None): + assert skiparg >= 2 # at least self and cr self.timeout = timeout self.skiparg = skiparg self.multi = multi @@ -553,9 +555,9 @@ class cache(object): del self.cache[kwargs['clear_keys']] return True - # Update named arguments with positional argument values + # Update named arguments with positional argument values (without self and cr) kwargs2 = kwargs.copy() - kwargs2.update(dict(zip(arg_names, args))) + kwargs2.update(dict(zip(arg_names, args[self.skiparg-2:]))) for k in kwargs2: if isinstance(kwargs2[k], (list, dict, set)): kwargs2[k] = tuple(kwargs2[k]) @@ -577,7 +579,7 @@ class cache(object): # Work out new value, cache it and return it # FIXME Should copy() this value to avoid futur modifications of the cache ? # FIXME What about exceptions ? - result = fn(self2,cr,*args, **kwargs) + result = fn(self2, cr, *args, **kwargs) self.cache[key] = (result, time.time()) return result