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)
Change History (11)
Changed 3 years ago by Dmitriy.Trt
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

Wrong history encoding screenshot