[6.0RC1][hr_timesheet]Problems printing timesheet reports

Bug #695960 reported by Alberto Luengo Cabanillas (Pexego)
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
OpenERP R&D Addons Team 3

Bug Description

Hi community,
Using following revisions of 6.0RC1 branch:

server: 3163 (12/28/2010)
gtk: 1738 (12/28/2010)
addons: 4033 (12/28/2010)

I got following error trying to print an employee timesheet:

[2010-12-31 12:33:46,443][XXX] ERROR:web-services:[07]: File "/opt/openerp/6.0/trunk/server/bin/addons/hr_timesheet/report/users_timesheet.py", line 114, in create_xml
[2010-12-31 12:33:46,443][XXX] ERROR:web-services:[08]: ''' % (header_xml,date_xml, emp_xml)
[2010-12-31 12:33:46,444][XXX] ERROR:web-services:[09]: UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 41: ordinal not in range(128)

This is because the employee I selected have accents in his name and/or surname. Same happening in 'Employees Timesheet' report.
I attach proposal patch.
Regards and best wishes for 2011!

Related branches

Revision history for this message
Alberto Luengo Cabanillas (Pexego) (alberto-pexego) wrote :
Revision history for this message
Amit Parik (amit-parik) wrote :

This could also be fixed from framework with the toxml() method inside report/interface.py
Thanks for reporting.

Changed in openobject-addons:
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) wrote :

It seems problem in toxml of interface.py so it will occur all reports (xsl).
i moved it to framework team.

Changed in openobject-addons:
assignee: OpenERP R&D Addons Team 3 (openerp-dev-addons3) → OpenERP's Framework R&D (openerp-dev-framework)
Changed in openobject-addons:
status: In Progress → Confirmed
affects: openobject-addons → openobject-server
Changed in openobject-server:
status: Confirmed → In Progress
Revision history for this message
Yogesh (SerpentCS) (yogesh-serpentcs) wrote :

Hello,

It has been fixed into https://code.launchpad.net/~openerp-dev/openobject-server/ysa-trunk-server/ branch by:
Revision ID: <email address hidden>
Revno: 3206

Thanks.

Changed in openobject-server:
status: In Progress → Fix Released
Revision history for this message
Laurent Urbain (Realdolmen) (laurent-urbain) wrote :

Hi,
With your modification, it works.

You can't merge in trunk server?

Changed in openobject-server:
milestone: none → 6.0
Changed in openobject-server:
status: Fix Released → In Progress
Revision history for this message
Yogesh (SerpentCS) (yogesh-serpentcs) wrote :

Shifting the project to addons it has to be change in addons.

affects: openobject-server → openobject-addons
Changed in openobject-addons:
milestone: 6.0 → none
status: In Progress → Confirmed
assignee: OpenERP's Framework R&D (openerp-dev-framework) → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
Changed in openobject-addons:
milestone: none → 6.0
Changed in openobject-addons:
status: Confirmed → In Progress
Changed in openobject-addons:
status: In Progress → Confirmed
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Ashvin Rathod (OpenERP) (ara-tinyerp) wrote :

Hello Alberto Luengo Cabanillas,

I have fixed the problem in lp:~openerp-commiter/openobject-addons/trunk-dev-addons3-ara branch, It will be merged soon with trunk-dev-addons3.

Revision No: <email address hidden>
Revision ID: 4660

Thank you,
ara

Changed in openobject-addons:
status: In Progress → Fix Committed
Revision history for this message
Ashvin Rathod (OpenERP) (ara-tinyerp) wrote :

Hello Alberto Luengo Cabanillas,

I have check your patch, but in this patch when ascent character the string is ignored and give blank space.

Thanks,
ara

Changed in openobject-addons:
status: Fix Committed → Fix Released
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Hello Alberto,

The situation was improved in the server with revision 3286 <email address hidden>, with a patch similar to what Yogesh originally did, but more consistent for all cases. Upgrading to this revision should solve this for you.

In addition, all addons that used to handle string with accents in an incorrect manner when building reports will also be fixed, and those patches will be merged tonight (some modifications will be done in above-proposed branches to make them correct as well).

Thank for you patience and for taking the time to report this bug and propose a patch!

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Correction, the actual revision number in the server is now: 3287 <email address hidden>.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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