Refactor in order to support dotted parameters
This commit is contained in:
parent
d21dfb4ae8
commit
9b9d2f3163
|
|
@ -133,15 +133,16 @@ class ServerConfiguration(models.TransientModel):
|
||||||
def _add_columns(self):
|
def _add_columns(self):
|
||||||
"""Add columns to model dynamically"""
|
"""Add columns to model dynamically"""
|
||||||
cols = chain(
|
cols = chain(
|
||||||
self._get_base_cols(),
|
self._get_base_cols().items(),
|
||||||
self._get_env_cols(),
|
self._get_env_cols().items(),
|
||||||
self._get_system_cols()
|
self._get_system_cols().items()
|
||||||
)
|
)
|
||||||
for col in cols:
|
for col, value in cols:
|
||||||
print col
|
col_name = col.replace('.', '_')
|
||||||
setattr(ServerConfiguration,
|
setattr(ServerConfiguration,
|
||||||
col,
|
col_name,
|
||||||
fields.Char(string=col, readonly=True))
|
fields.Char(string=col, readonly=True))
|
||||||
|
self._conf_defaults[col_name] = value
|
||||||
|
|
||||||
def _get_base_cols(self):
|
def _get_base_cols(self):
|
||||||
""" Compute base fields"""
|
""" Compute base fields"""
|
||||||
|
|
@ -152,7 +153,6 @@ class ServerConfiguration(models.TransientModel):
|
||||||
# fld.name = key
|
# fld.name = key
|
||||||
# fld.model_name = self._model
|
# fld.model_name = self._model
|
||||||
res[key] = item
|
res[key] = item
|
||||||
self._conf_defaults[key] = item
|
|
||||||
return res
|
return res
|
||||||
|
|
||||||
def _get_env_cols(self, sections=None):
|
def _get_env_cols(self, sections=None):
|
||||||
|
|
@ -163,7 +163,6 @@ class ServerConfiguration(models.TransientModel):
|
||||||
for col, item in serv_config.items(section):
|
for col, item in serv_config.items(section):
|
||||||
key = self._format_key(section, col)
|
key = self._format_key(section, col)
|
||||||
res[key] = item
|
res[key] = item
|
||||||
self._conf_defaults[key] = item
|
|
||||||
return res
|
return res
|
||||||
|
|
||||||
def _get_system_cols(self):
|
def _get_system_cols(self):
|
||||||
|
|
@ -172,7 +171,6 @@ class ServerConfiguration(models.TransientModel):
|
||||||
for col, item in get_server_environment():
|
for col, item in get_server_environment():
|
||||||
key = self._format_key('system', col)
|
key = self._format_key('system', col)
|
||||||
res[key] = item
|
res[key] = item
|
||||||
self._conf_defaults[key] = item
|
|
||||||
return res
|
return res
|
||||||
|
|
||||||
def _group(self, items):
|
def _group(self, items):
|
||||||
|
|
@ -180,7 +178,7 @@ class ServerConfiguration(models.TransientModel):
|
||||||
names = []
|
names = []
|
||||||
|
|
||||||
for key in sorted(items):
|
for key in sorted(items):
|
||||||
names.append(key)
|
names.append(key.replace('.', '_'))
|
||||||
return ('<group col="2" colspan="4">' +
|
return ('<group col="2" colspan="4">' +
|
||||||
''.join(['<field name="%s" readonly="1"/>' %
|
''.join(['<field name="%s" readonly="1"/>' %
|
||||||
_escape(name) for name in names]) +
|
_escape(name) for name in names]) +
|
||||||
|
|
|
||||||
|
|
@ -53,7 +53,7 @@ def get_server_environment():
|
||||||
lsbinfo = 'not lsb compliant'
|
lsbinfo = 'not lsb compliant'
|
||||||
return (
|
return (
|
||||||
('platform', platform.platform()),
|
('platform', platform.platform()),
|
||||||
('os name', os.name),
|
('os.name', os.name),
|
||||||
('lsb_release', lsbinfo),
|
('lsb_release', lsbinfo),
|
||||||
('release', platform.release()),
|
('release', platform.release()),
|
||||||
('version', platform.version()),
|
('version', platform.version()),
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue