Opened 3 years ago

Closed 3 years ago

#253 closed defect (fixed)

Wrong encoding in call history

Reported by: Dmitriy.Trt Owned by: laurent
Priority: major Milestone: QuteCom 2.2-RC4
Component: GUI Version: 2.2
Keywords: Cc:

Description

History displays Name/ID in wrong encoding. See attached screenshot.

I've tried version 2.2~rc3 on Debian Linux (installed from Debian testing repo) and version 2.2-RC3 on Windows XP, bug exists on both platforms.

Attachments (3)

Выделение-18c.png (56.3 KB) - added by Dmitriy.Trt 3 years ago.
Wrong history encoding screenshot
utf8_support.diff (148.4 KB) - added by laurent 3 years ago.
utf8_support_2.diff (148.5 KB) - added by Dmitriy.Trt 3 years ago.

Download all attachments as: .zip

Change History (11)

Changed 3 years ago by Dmitriy.Trt

Wrong history encoding screenshot

Changed 3 years ago by laurent

comment:1 Changed 3 years ago by laurent

Hello Dmitry.Trt

This is an experimental patch which solves this issue. it needs to be tested ...

comment:2 Changed 3 years ago by Dmitriy.Trt

Sorry, but I can't apply your patch to the latest qutecom-2.2 source code from mercurial, there are tons of rejects. Am I doing something wrong?

comment:3 Changed 3 years ago by laurent

You should use http://hg.qutecom.org/qutecom-2.2/ reposit :

hg clone http://hg.qutecom.org/qutecom-2.2/

If you already have cloned QuteCom :

hg pull
hg update

apply the patch this way :

hg import utf8_support.diff

it works fine here :)

comment:4 Changed 3 years ago by laurent

One more things :
Just new history entries will be displayed correctly

comment:5 Changed 3 years ago by Dmitriy.Trt

Thanks for your help, now I've got patched source. I'll try to build and test it in a next few days.

P.S. Cmake process was failed with errors about regexp when path to folder was with "++". Should I create bug report about it?

comment:6 Changed 3 years ago by Dmitriy.Trt

There were few errors, so I've fixed them. I'm attaching second patch with my changes added to original patch. Now I've got patched version running. Tomorrow I'll make some calls to contact with Cyrillic name and report history results here.

Changed 3 years ago by Dmitriy.Trt

comment:7 Changed 3 years ago by Dmitriy.Trt

Patch fixes encoding problem! Thanks a lot! Even old history items are shown correct. It would be great to see this patch in upstream.

comment:8 Changed 3 years ago by laurent

  • Resolution set to fixed
  • Status changed from new to closed

bug fix : 666

Note: See TracTickets for help on using tickets.