Stock: shipping type should be hidden or readonly

[Numérigraphe]

In 6.0 up to current trunk, the extended view has the field "shipping type" available for read/write in the "additional info" of receptions and deliveries.
This should be hidden because the context makes it quite clear if this is a reception, delivery or internal move.
If it's still displayed for a reason or another, it should at least be for read only, because changing it will have unexpected results such as not updating the average price on receptions.

[Ravish(OpenERP) (rmu-openerp)]

Hello,

If picking is in done state this field called "shipping type" should be read only . For all reception, delivery or internal move.

Thanks!!!

[Numérigraphe (numerigraphe)]

Le 21/02/2012 07:34, Ravish(OpenERP) a écrit :
> Hello,
>
> If picking is in done state this field called "shipping type" should be
> read only . For all reception, delivery or internal move.
>
> Thanks!!!
>
> ** Changed in: openobject-addons
> Status: New => Confirmed
>
> ** Changed in: openobject-addons
> Importance: Undecided => Low
>
> ** Changed in: openobject-addons
> Assignee: (unassigned) => OpenERP R&D Addons Team 2 (openerp-dev-addons2)
>
I suggest it should be read-only in all states. Each value of the field
normally corresponds to a distinct view with customized buttons.
Also, changing this value makes the picking "move" from one menu to the
other. You can't expect users to understand what's going on in this case.

[Mayur Maheshwari(OpenERP) (mma-openerp)]

Hello

Thanks for Reporting.
It has been fixed in lp:~openerp-dev/openobject-addons/trunk-bug-937077-mma With
Revision ID: <email address hidden>
Revision num: 6626

It will be available in trunk soon.

[Ravish(OpenERP) (rmu-openerp)]

Hello Lionel ,

As you suggested that all field should be read only .But when I want to create picking from "incoming shipment" that time I need all provide selection options.so this fix committed is perfect for now.

Thanks!!

[Numérigraphe (numerigraphe)]

Please let's discuss this on the merge proposal.
Lionel.

[Ravish(OpenERP) (rmu-openerp)]

Hello Lionel ,

I agree with you that ,It should be read only except draft state.

Thanks!!

[qdp (OpenERP) (qdp)]

well actually this field should be always readonly and set by the domains of menuitems.

I made my own fix that landed in trunk in revision 7132

thanks for the contribution,
Quentin