lp:~camptocamp/openerp-connector/7.0-unicode-pickled-1288187

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~openerp-connector-core-editors/openerp-connector/7.0 at revision 627

Leonardo Pistone (community): Approve (code review) on 2014-03-06

Stéphane Bidoul (Acsone) (community): Approve (code review and test) on 2014-03-06

OpenERP Connector Core Editors: Pending requested 2014-03-05

Diff: 112 lines (+51/-4)

5 files modified

connector/__openerp__.py (+1/-1)
connector/migrations/2.2.0/pre-migration.py (+9/-0)
connector/queue/job.py (+2/-2)
connector/queue/model.py (+1/-1)
connector/tests/test_job.py (+38/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1288187: Error storing jobs with unicode arguments

Undecided

Fix Released

Link a bug report

Related blueprints

628. By Guewen Baconnier @ Camptocamp on 2014-03-19

func::bytea can fail to convert the string to bytea because it interprets the backshlashes. Use convert_to to LATIN1 instead

627. By Guewen Baconnier @ Camptocamp on 2014-03-06

confusing typo

626. By Guewen Baconnier @ Camptocamp on 2014-03-06

more interesting to read the values after the write/read roundtrip than the values set in the Job instance...

625. By Guewen Baconnier @ Camptocamp on 2014-03-06

add a test with accented bytestring

624. By Guewen Baconnier @ Camptocamp on 2014-03-06

conversion no longer necessary

623. By Guewen Baconnier @ Camptocamp on 2014-03-06

compare the job with the values to ensure the encoding is correct

622. By Guewen Baconnier @ Camptocamp on 2014-03-05

migrate the text field containing the pickled function to a bytea field

621. By Guewen Baconnier @ Camptocamp on 2014-03-05

[FIX] Jobs with unicode arguments cannot be stored.

The error here is that pickled data are binary data. Psycopg2 tries to decode
data to UTF-8 which is not possible because this is not UTF-8 data but binary
data.

Use a binary field rather than a text field.

620. By Launchpad Translations on behalf of openerp-connector-core-editors on 2014-02-22

Launchpad automatic translations update.

619. By Launchpad Translations on behalf of openerp-connector-core-editors on 2014-02-14

Launchpad automatic translations update.