openerp-community team mailing list archive

[Niels Huylebroeck <nh@xxxxxxxxxx>]

2013/9/3 Olivier Dony <odo@xxxxxxxxxxx>

>
> There is currently no special provision in OpenERP Apps for handling
> multi-valued manifest entries. Module manifests are simply parsed by the
> standard ir.module.module.get_values_**from_terp() which expects plain
> strings for `author` and `maintainer`, and a list of strings for
> `contributors`.
>
> For modules with multiple authors, it makes sense to set the author field
> to a string containing a comma-separated list of authors. Search queries
> use full-text criterions including the author field, so that  will work as
> expected: a search for "foo" will match modules authored by "foo, bar".
> There is no "group by author" anymore.
>
>
Worth mentioning if the search will also match  ['foo', 'bar'] when given
"foo" i suppose?

I would still like to suggest to maintain the different authors in a python
list with strings because it will allow easy parsing for any tool.
Also the OpenERP Apps website at this time does not do any grouping but
should there ever be a desire for such functionality it would be immensly
easier to have the authors in a proper list instead of a single string with
"," separated values.


Perhaps a quick fix on the website part should be implemented that properly
shows the list items aggregated:

if isinstance(author_value, basestring):
    # Regular string
    authors = author_value
else:
    # Assume list
    authors = ", ".join(author_value)

This fix could obviously also be applied to 'author', 'maintainer',
'contributors' fields when they are shown.

-- 
Niels Huylebroeck
Lead Architect   --   Agaplan
Tel. : +32 (0) 93 95 98 90
Web : http://www.agaplan.eu