[TRUNK] Rev 2144 broken a field property type char

Bug #562356 reported by Christophe CHAUVET
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Server (MOVED TO GITHUB)
Status tracked in Trunk
Trunk
Fix Released
High
Anup(SerpentCS)

Bug Description

Hi

In rev 2144, the field property type char (same thing with float, date, etc...) is broken, on multi company environnement,

try to add

'test_field':fields.property(None, method=True, type='char', string='test_field', size = 32),

and try to test it with normal user (not with admin)

Regards,

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

It has been fixed by revision 2194 <email address hidden>.
Thanks.

Revision history for this message
Christophe CHAUVET (christophe-chauvet) wrote :

Hi Jay

I recheck now with the last revision and it doesn't work.

Regards,

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Make sure you have the change in code.
Thanks.

Revision history for this message
Christophe CHAUVET (christophe-chauvet) wrote :

OpenObject: TRUNK
Server revno: 2188
Addons revno: 3313
Client revno: 1247
Web revno: 2955

I have test it with these revisions, i publish the module [1] to check the regression, this module was develop when i integrate the possibility to use fields.property not only in many2one, and Fabien Pinckaers (22 december 2009) merge it in revision 1939 (trunk)

This module add a new tab on product form and add one char field, one float field, one date field

Regards,

[1] bzr branch lp:~syleam/+junk/property_check

Distribution: Ubuntu
Version: 9.10 (karmic)
Python 2.6.4

Locale:
  LANG=fr_FR.UTF-8

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

I have checked with latest trunk 2200, it works!

Revision history for this message
Claude Brulé (claude-brule-syleam) wrote :

This module does not work properly with revno 2207 !

The aim of fields.property is to have differents values according to the company (in witch you are connected).

Actually, it is write that there is no traceback at this point. But the behaviours is NOT the one we expected.

Let's take an example :
1) install multi_company module with demon datas
2) install property_check module (it would have dependances between them)
3) login (event as admin)
4) choose company "shop 1" (user/preferences)
5) open a product form (Eg PC1) and have a look at "Property check" tab
6) change values of all fields. For example :
    test prop = "shop 1"
    A float = 1,11
    Date = 11/11/2011 (according the format of your country, this one is French)
7) save
8) change company, choose "shop 2"

At this point values are still the same than the one you entered. It is wrong. As it is another compagny, values would have changed.
For this example NULL or default values were expected.

9) change values of all fields. (all this fields are fields.property)
    test prop = "shop 2"
    A float = 2,22
    Date = 2022-02-22
10) save this recod (product)

8) change company, come back to "shop 1"
Values of all fields still didn't change.
We would retreive the first we entered "shop 1", 1.11 and so on.

For fields.property the value entered in one company is NOT overwrite in another company (only standard fields working like that).

This bug is ralated to a maintenance contract.
Please, remember that the developpement made by Christophe Chauve - Syleam was working in december.
It has been merged in trunk, and actually we are in trouble because it work diffrently.

Sincerly
Claude Brulé

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Hello Claude Brulé,

The problem pointed by the title is already solved.

The problem you point out, is not a problem at all. I don't think that Property will work according to company.

It only has a relation with The Product.
Kindly make sure.
Thanks.

Revision history for this message
Stephane Wirtel (OpenERP) (stephane-openerp) wrote :

Fixed in
revno: 2236
revision-id: <email address hidden>
parent: <email address hidden>
fixes bug(s): https://launchpad.net/bugs/562356
author: Quality Team
committer: Stephane Wirtel <email address hidden>
branch nick: server
timestamp: Wed 2010-05-05 12:02:41 +0200
message:
  [IMP] Add the capabilities to store a datetime, date, integer, float and char in a field.property
------------------------------------------------------------
revno: 2235
revision-id: <email address hidden>
parent: <email address hidden>
author: Quality Team
committer: Stephane Wirtel <email address hidden>
branch nick: server
timestamp: Tue 2010-05-04 16:46:42 +0200
message:
  [FIX] Rewrite the field.property engine to work as expected (value per
          company)

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.