On 27/06/12 21:07 +0200, Cédric Krier wrote:
> On 27/06/12 20:21 +0200, Udo Spallek wrote:
> > Wed, 27 Jun 2012 20:04:55 +0200
> > Cédric Krier <[email protected]>:
> > > On 27/06/12 19:36 +0200, Udo Spallek wrote:
> > > > Mon, 25 Jun 2012 13:10:16 +0200
> > > > Cédric Krier <[email protected]>:
> > > > > On 25/06/12 11:16 +0200, Udo Spallek wrote:
> > > > > Also we don't want to have to add every new location created to
> > > > > the configuration.
> > > > What is the problem with this?
> > > It doesn't scale if you have thousand of locations.
> > For me there is no need to put all existing or new locations to the
> > configuration.
> > 
> > > > For me stock.lot.type model looks very restrictive and hard to
> > > > extend.
> > > I don't see why. For now it follow the same concept as the location.
> > > After that it is still possible to extend it by overriding
> > > Product.lot_is_required
> > It would be much more flexible when stock.lot.type is a table,
> > which stores lines of lot types to apply:
> > 
> > class LotType(ModelSQL, ModelView):
> >     "Stock Lot Type"
> >     _name = 'stock.lot.type'
> >     _description = __doc__
> > 
> >     location_type = fields.Many2One('stock.location.type', 
> >         'Location Type', states={
> >             'required': True,
> >             })
> > 
> > LotType()
> > 
> > class StockLocationType(ModelSQL, ModelView):
> >     "Stock Location Type"
> >     _name = 'stock.location.type'
> >     _description = __doc__
> > 
> >     name = fields.Char('Name', required=True, translate=True)
> >     code = fields.Char('Code', required=True)
> > 
> > StockLocationType()
> > 
> > This kind of lot type has only a selection of a stock location type.
> > This would make the model much more flexible, because it is possible to
> > extend it with other criteria then the location type. E.g. one can add
> > another many2one to stock location.
> 
> 
> It is the same except that your design is more complicated.
> Nothing prevent you to create LotType that are not linked to location.

By the way, the current LotType.code is just like your
LotType.location_type

-- 
Cédric Krier

B2CK SPRL
Rue de Rotterdam, 4
4000 Liège
Belgium
Tel: +32 472 54 46 59
Email/Jabber: [email protected]
Website: http://www.b2ck.com/

Attachment: pgpwLHtHWPUY2.pgp
Description: PGP signature

Reply via email to