After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 55142 - [PATCH] libole2 don't convert charset in the "Document Summary" fields
[PATCH] libole2 don't convert charset in the "Document Summary" fields
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: import/export MS Excel (tm)
0.x
Other Linux
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2001-05-24 09:22 UTC by frob
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
test sheet with "TECT"-like text in Doc-summary fields (13.50 KB, application/octet-stream)
2001-05-24 09:23 UTC, frob
  Details
screenshot. every field must look like "TECT" (cyrillic 'TEST') instead of "peRp" (26.95 KB, image/jpeg)
2001-11-09 08:43 UTC, frob
  Details
sample table (13.50 KB, application/octet-stream)
2001-11-09 08:44 UTC, frob
  Details
patch from Sultanov Albert <sar ufk62 ru> (3.75 KB, patch)
2001-11-13 08:18 UTC, frob
none Details | Review

Description frob 2001-05-24 09:22:03 UTC
If you open non-latin1 XL sheet in the gnumeric, then "Document Summary"
fields still don't converted to suitable locale.
You can use attached test-summary.xls for testing.
It was saved from XL2K and includes cyrillic strings in every "Document
Summary"
fields.
If you have cyrillic fonts and start gnumeric with LANG=ru_RU.KOI8-R,
then this string must include cyrillic word that looks like "TECT".
If you haven't cyr-fonts this possibly must looks like "o with ^ above" +
"a with 0 above" + "o with ` above" + "o with ^ above".
Now (if you haven't cyr-fonts) this possibly will looks like
"O with backtick above"+"A with 0 above"+"N with tilda above"+"O with
backtick above".
So, I hope that some parts of code that used for the cells content
conversion can be reused here.
Comment 1 frob 2001-05-24 09:23:23 UTC
Created attachment 560 [details]
test sheet with "TECT"-like text in Doc-summary fields
Comment 2 frob 2001-11-09 08:43:24 UTC
Created attachment 5994 [details]
screenshot. every field must look like "TECT" (cyrillic 'TEST') instead of "peRp"
Comment 3 frob 2001-11-09 08:44:32 UTC
Created attachment 5995 [details]
sample table
Comment 4 frob 2001-11-13 08:18:32 UTC
Created attachment 6015 [details] [review]
patch from Sultanov Albert <sar ufk62 ru>
Comment 5 Jody Goldberg 2001-12-10 15:22:11 UTC
Patch applied.  Many thanks.