Martin Trigaux
f138aa2608
- display_name uses name_get and not the other way around:
name_get should not call _compute_display_name, _compute_display_name should call name_get.
The previous behaviour was not backward-compatible with the old api.
All the models redefining name_get would have 2 different behaviors between name_get and display_name.
- Do not set an inverse function to display_name:
In most cases, writing on display_name writes on _rec_name (if any, not mandatory).
If the display_name computation is redefined, we need to redefine as well the inverse method to avoid unexpected behaviour
This required to also modify tests in base_import as readonly fields are avoided.
- Remove search method on display_name:
For the same reason as for the first point, it could be good that searching on display_name use name_search (and not the other way around).
However doing this would be very inefficiant (need to do the search, without limit, extract the ids of the name_get result just to generate
a subdomain ('id', 'in', [...]). As in most cases it would anyway mean to search on the _rec_name it's better to directly do so.
- Changing label to avoid mismatch:
In view displaying the list of fields or when a match is made on the label of a field (e.g. when importing csv file,
matching is made on both label and technical name), the fact that display_name field has '
Calling it 'Display Name' will avoid most errors.
- remove display_name definition from website_forum_doc,ir_model:
These fields are doing the same thing as the display_name of the new api, we can remove them.
We need to keep the one for res.partner as it's a stored field.
76 lines
2.9 KiB
Python
76 lines
2.9 KiB
Python
# -*- coding: utf-8 -*-
|
|
|
|
import openerp
|
|
from openerp.osv import osv, fields
|
|
|
|
class Documentation(osv.Model):
|
|
_name = 'forum.documentation.toc'
|
|
_description = 'Documentation ToC'
|
|
_inherit = ['website.seo.metadata']
|
|
_order = "parent_left"
|
|
_parent_order = "sequence, name"
|
|
_parent_store = True
|
|
def name_get(self, cr, uid, ids, context=None):
|
|
if isinstance(ids, (list, tuple)) and not len(ids):
|
|
return []
|
|
if isinstance(ids, (long, int)):
|
|
ids = [ids]
|
|
reads = self.read(cr, uid, ids, ['name','parent_id'], context=context)
|
|
res = []
|
|
for record in reads:
|
|
name = record['name']
|
|
if record['parent_id']:
|
|
name = record['parent_id'][1]+' / '+name
|
|
res.append((record['id'], name))
|
|
return res
|
|
|
|
# TODO master remove me
|
|
def _name_get_fnc(self, cr, uid, ids, prop, unknow_none, context=None):
|
|
res = self.name_get(cr, uid, ids, context=context)
|
|
return dict(res)
|
|
|
|
_columns = {
|
|
'sequence': fields.integer('Sequence'),
|
|
'name': fields.char('Name', required=True, translate=True),
|
|
'introduction': fields.html('Introduction', translate=True),
|
|
'parent_id': fields.many2one('forum.documentation.toc', 'Parent Table Of Content', ondelete='cascade'),
|
|
'child_ids': fields.one2many('forum.documentation.toc', 'parent_id', 'Children Table Of Content'),
|
|
'parent_left': fields.integer('Left Parent', select=True),
|
|
'parent_right': fields.integer('Right Parent', select=True),
|
|
'post_ids': fields.one2many('forum.post', 'documentation_toc_id', 'Posts'),
|
|
'forum_id': fields.many2one('forum.forum', 'Forum', required=True),
|
|
}
|
|
|
|
_constraints = [
|
|
(osv.osv._check_recursion, 'Error ! You cannot create recursive categories.', ['parent_id'])
|
|
]
|
|
|
|
|
|
class DocumentationStage(osv.Model):
|
|
_name = 'forum.documentation.stage'
|
|
_description = 'Post Stage'
|
|
_order = 'sequence'
|
|
_columns = {
|
|
'sequence': fields.integer('Sequence'),
|
|
'name': fields.char('Stage Name', required=True, translate=True),
|
|
}
|
|
|
|
|
|
class Post(osv.Model):
|
|
_inherit = 'forum.post'
|
|
_columns = {
|
|
'documentation_toc_id': fields.many2one('forum.documentation.toc', 'Documentation ToC', ondelete='set null'),
|
|
'documentation_stage_id': fields.many2one('forum.documentation.stage', 'Documentation Stage'),
|
|
'color': fields.integer('Color Index')
|
|
}
|
|
def _read_group_stage_ids(self, cr, uid, ids, domain, read_group_order=None, access_rights_uid=None, context=None):
|
|
stage_obj = self.pool.get('forum.documentation.stage')
|
|
stage_ids = stage_obj.search(cr, uid, [], context=context)
|
|
result = stage_obj.name_get(cr, uid, stage_ids, context=context)
|
|
return result, {}
|
|
|
|
_group_by_full = {
|
|
'documentation_stage_id': _read_group_stage_ids,
|
|
}
|
|
|