sale/stock_view.xml - conceptual error

Bug #601485 reported by Ferdinand
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Won't Fix
Wishlist
OpenERP R&D Addons Team 3

Bug Description

in this file sale_id is added to the view after "active" - but it never gets displayed (V5 and V6) (because of no field active in the view?)

if it is displayed - which is important to be able to navigate back from picking to SO - IMHO the field "Origin" can be eliminated

Revision history for this message
Ferdinand (office-chricar) wrote :

please check also for procurements

Revision history for this message
Nhomar - Vauxoo (nhomar) wrote : Re: [Bug 601485] [NEW] sale/stock_view.xml - conceptual error

Im Agree with you, but, the field "Origin" is useful in some cases, we use
this field in a lot of functions,

Regards

2010/7/3 Ferdinand @ ChriCar <email address hidden>

> Public bug reported:
>
> in this file sale_id is added to the view after "active" - but it never
> gets displayed (V5 and V6) (because of no field active in the view?)
>
> if it is displayed - which is important to be able to navigate back from
> picking to SO - IMHO the field "Origin" can be eliminated
>
> ** Affects: openobject-addons
> Importance: Undecided
> Status: New
>
> --
> sale/stock_view.xml - conceptual error
> https://bugs.launchpad.net/bugs/601485
> You received this bug notification because you are a member of OpenERP
> Drivers, which is subscribed to OpenObject Addons.
>
> Status in OpenObject Addons Modules: New
>
> Bug description:
> in this file sale_id is added to the view after "active" - but it never
> gets displayed (V5 and V6) (because of no field active in the view?)
>
> if it is displayed - which is important to be able to navigate back from
> picking to SO - IMHO the field "Origin" can be eliminated
>
>
>

--
Saludos Cordiales

Nhomar G. Hernandez M.
+58-414-4110269
+58-212-6615932
+58-212-9536734 ext 124
+58-212-9512643
Skype: nhomar00
Web-Blog: http://geronimo.com.ve
Servicios IT: http://openerp.netquatro.com
Linux-Counter: 467724
Correos:
<email address hidden>
<email address hidden>

Revision history for this message
Ferdinand (office-chricar) wrote :

@ Nhomar:
OK - but shouldn't it then be called differently ?

Revision history for this message
Nhomar - Vauxoo (nhomar) wrote : Re: [Bug 601485] Re: sale/stock_view.xml - conceptual error

Well. With this option i'm agree (for version 6), but IMHO the function
should be the same "__fill automatcally tracing all process about a
document__" the "Link" as ferdinand says is the correct way.

2010/7/3 Ferdinand @ ChriCar <email address hidden>

> @ Nhomar:
> OK - but shouldn't it then be called differently ?
>
> --
> sale/stock_view.xml - conceptual error
> https://bugs.launchpad.net/bugs/601485
> You received this bug notification because you are a member of OpenERP
> Drivers, which is subscribed to OpenObject Addons.
>
> Status in OpenObject Addons Modules: New
>
> Bug description:
> in this file sale_id is added to the view after "active" - but it never
> gets displayed (V5 and V6) (because of no field active in the view?)
>
> if it is displayed - which is important to be able to navigate back from
> picking to SO - IMHO the field "Origin" can be eliminated
>
>
>

--
Saludos Cordiales

Nhomar G. Hernandez M.
+58-414-4110269
+58-212-6615932
+58-212-9536734 ext 124
+58-212-9512643
Skype: nhomar00
Web-Blog: http://geronimo.com.ve
Servicios IT: http://openerp.netquatro.com
Linux-Counter: 467724
Correos:
<email address hidden>
<email address hidden>

Changed in openobject-addons:
status: New → Triaged
Revision history for this message
Amit Parik (amit-parik) wrote :

Hello DR Ferdinand,

In version5.0 "active" field in "Extended View". So you can add group "Extended View" in related user then you can see the "sale_id" and "active" both field in "Others info" tab.

In version6.0 "active" field use as a "auto_picking" name and this field is in "Multi Company" and "Extended" View. So for that you have to install the "multi_company" module and add "Multi Company" and "Extended" both groups in the related users. Now You can see "sale_id" and "active" both field in "Additional info" tab.

Thanks.

Changed in openobject-addons:
status: Triaged → Invalid
Revision history for this message
Ferdinand (office-chricar) wrote :

well problem described in the initial report is "solved" by

                <field name="auto_picking" position="after">
                    <field name="sale_id"/>

instead of
                <field name="acitve" position="after">
                    <field name="sale_id"/>

nevertheless I am not sure if the rule described in #5 is the correct way to do
Example:
* field A is only visible for "Extended"
* field B should be placed after A, but visible for all

What AMP is saying is that B is only visible for group "Extended" which IMHO is not correct.
at least it might create unwanted side effects as it is very difficult to find the inheriting views and it shouldn't be the job of the maintainer of field A to tell the maintainer of field B that he has added an attribute "Extended" and hence B must be placed elsewhere to be displayed.

IMHO inheritance and display logic must be completely separated.

Changed in openobject-addons:
status: Invalid → New
Amit Parik (amit-parik)
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
status: New → Triaged
Revision history for this message
qdp (OpenERP) (qdp) wrote :

Hello,
Thank you for your suggestion! However this is unfortunately out of the scope of the current OpenERP release, so we cannot implement it.
Let's close this bug for now, for the sake of clarity in Launchpad, it can always be reopened later when we consider new features for future roadmaps.
Thank you for your understanding!

Changed in openobject-addons:
importance: Undecided → Wishlist
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.