Bug 103492 - Arabic words not formed well in textboxes when sentence begins with English word
Summary: Arabic words not formed well in textboxes when sentence begins with English word
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
5.2.0.4 release
Hardware: All All
: medium normal
Assignee: ⁨خالد حسني⁩
URL:
Whiteboard: target:7.5.0 target:7.4.1inReleaseNot...
Keywords: bibisected, bisected, regression
: 119960 149889 150395 150544 (view as bug list)
Depends on:
Blocks: Arabic-and-Farsi RTL-Textbox
  Show dependency treegraph
 
Reported: 2016-10-25 12:32 UTC by Yousuf Philips (jay) (retired)
Modified: 2023-12-05 13:24 UTC (History)
11 users (show)

See Also:
Crash report or crash signature:


Attachments
sample (9.45 KB, application/vnd.oasis.opendocument.text)
2016-10-25 12:32 UTC, Yousuf Philips (jay) (retired)
Details
screenshot (36.55 KB, image/png)
2017-10-02 18:18 UTC, Yousuf Philips (jay) (retired)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yousuf Philips (jay) (retired) 2016-10-25 12:32:15 UTC
Created attachment 128245 [details]
sample

Steps:
1) Open attached doc
2) Notice the differences between the arabic words in the 2 sentences in the document and the first two sentences in the textbox

Regression as things were better in 4.4.

Version: 5.3.0.0.alpha1+
Build ID: 928776b734c6aa188151bbce048d5bef4486dce7
CPU Threads: 2; OS Version: Linux 3.19; UI Render: default; VCL: gtk2; 
TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2016-10-23_00:39:08
Locale: en-US (en_US.UTF-8); Calc: group
Comment 1 Xisco Faulí 2016-10-25 13:46:49 UTC
Confirmed in 

Version: 5.3.0.0.alpha0+
Build ID: 8974b0fafb18f9dd3f2c0e175a3255b80e4c249e
CPU Threads: 4; OS Version: Linux 4.2; UI Render: default; 
Locale: ca-ES (ca_ES.UTF-8); Calc: group

if it was better in 4.4, worth it to bibisect it
Comment 2 Xisco Faulí 2016-10-25 14:06:17 UTC
It was stil better in

Version: 5.0.0.0.alpha1+
Build ID: 0db96caf0fcce09b87621c11b584a6d81cc7df86
Locale: ca-ES (ca_ES.UTF-8)

so it needs to be bisected in 51max or 52max.

@raal: Do you want to bibisect it?
Comment 3 raal 2016-10-27 18:41:13 UTC
 393b5a03e8a580103cc31ca4752396032838ecdb is the first bad commit
commit 393b5a03e8a580103cc31ca4752396032838ecdb
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sun Sep 13 18:21:45 2015 -0700

    source 41007842ed9bb5d6165792a197769f72dae55a2c
author    Martin Hosken <martin_hosken@sil.org>    2015-09-10 03:14:18 (GMT)
committer    Martin Hosken <martin_hosken@sil.org>    2015-09-14 01:16:40 (GMT)
commit    41007842ed9bb5d6165792a197769f72dae55a2c (patch)
tree    3db9834122c1a6b43bd2428129629a6f66669978
parent    35fd0cf311d0ab6e647ef8a244f350d8a690e734 (diff)
Refactor graphite integration and update graphite
   
git bisect log
# bad: [05d11632892a322664fb52bac90b2598b7fb7544] source 5616d22b57a9a5e57d545e912e029162a230829b
# good: [c1efd324c6ad448ac9edb030dc9738b9e6899e4d] source ab465b90f6c6da5595393a0ba73f33a1e71a2b65
git bisect start 'origin/master' 'oldest'
# good: [97526ab777da7e58ce283c05498262ecdd4d6f7f] source 4ea70f87f7a2b61eda6e5ab1f48debf6fcfadc1f
git bisect good 97526ab777da7e58ce283c05498262ecdd4d6f7f
# bad: [86fee7ded76d9c2756ccab6aef160a2d7fab0ab6] source 1b62841b1859ae3443e2bf1ebe99ec3d6afb6cc2
git bisect bad 86fee7ded76d9c2756ccab6aef160a2d7fab0ab6
# bad: [ecd02a00b3cb479dcecd6a2539e2f4140cd8158f] source f45ac62a20b80033a7f5ccdef4a6c116b6fece24
git bisect bad ecd02a00b3cb479dcecd6a2539e2f4140cd8158f
# bad: [6629f1abc962685b1c83b088dff82517bb2f1691] source 5496f2a3ee8e76dda6d1c393308be1e9bbb90d6e
git bisect bad 6629f1abc962685b1c83b088dff82517bb2f1691
# good: [a5f968795bf60a73039ae687b366800b7929f17c] source 6917cd98ca6b6fd2d495d0257c7fe50611982d34
git bisect good a5f968795bf60a73039ae687b366800b7929f17c
# good: [368c6e35342202ca22c81feecd66f9073d1e3d8e] source a37535e3ff7be959d9a3aab3399ffbcc89688662
git bisect good 368c6e35342202ca22c81feecd66f9073d1e3d8e
# bad: [ab5f30fb13db2e0dbf317e87f8626cb1dfc8f234] source 584d55178d2e390e60355b18bbac4be16fe750dd
git bisect bad ab5f30fb13db2e0dbf317e87f8626cb1dfc8f234
# bad: [31777579f3e0cf27599cdeecd2d54140bb8aa92e] source 6532cb0e5ec3a59c248b332e868c4c03c31659f1
git bisect bad 31777579f3e0cf27599cdeecd2d54140bb8aa92e
# bad: [20f8cc90e939167b90170a9ecf1fc143975280ca] source 0e916b4143b2c46fec6df25cce6f14b595d5b023
git bisect bad 20f8cc90e939167b90170a9ecf1fc143975280ca
# good: [1db296d58ea920df1c5a23cc35720c4084b1ae63] source 47d3e82e4f2c0c06231c952a0cc2456b712da0cc
git bisect good 1db296d58ea920df1c5a23cc35720c4084b1ae63
# good: [bcae2a9e9aeb8394df93fb275b55873d3dbc2829] source 168ad25711c0bd6e3a025b083312e3ed2d237933
git bisect good bcae2a9e9aeb8394df93fb275b55873d3dbc2829
# good: [6f8dba2f7237c74c6d627cbcdec2c515b2f6cb0b] source d8160fa8343a395cff0116286dd24894b076c02b
git bisect good 6f8dba2f7237c74c6d627cbcdec2c515b2f6cb0b
# bad: [393b5a03e8a580103cc31ca4752396032838ecdb] source 41007842ed9bb5d6165792a197769f72dae55a2c
git bisect bad 393b5a03e8a580103cc31ca4752396032838ecdb
# good: [69568d9d51758db13ffd336e65561c4b72978d33] source 35fd0cf311d0ab6e647ef8a244f350d8a690e734
git bisect good 69568d9d51758db13ffd336e65561c4b72978d33
# first bad commit: [393b5a03e8a580103cc31ca4752396032838ecdb] source 41007842ed9bb5d6165792a197769f72dae55a2c
Comment 4 Xisco Faulí 2017-03-10 14:44:35 UTC
updating affected version
Comment 5 Yousuf Philips (jay) (retired) 2017-10-02 18:18:49 UTC
Created attachment 136707 [details]
screenshot
Comment 6 Yousuf Philips (jay) (retired) 2017-10-02 18:19:14 UTC
still present

Version: 6.0.0.0.alpha0+
Build ID: a2a3e06a29077d4274dc15eea28a01afe22e3658
CPU threads: 2; OS: Linux 4.4; UI render: default; VCL: gtk2; 
Locale: en-US (en_US.UTF-8); Calc: group
Comment 7 QA Administrators 2018-10-13 03:14:17 UTC Comment hidden (obsolete)
Comment 8 Buovjaga 2020-09-30 17:50:18 UTC
Still confirmed

Arch Linux 64-bit
Version: 7.1.0.0.alpha0+
Build ID: 09c24681a3414092fde50ec0f617c9f7c79e8a61
CPU threads: 8; OS: Linux 5.8; UI render: default; VCL: kf5
Locale: fi-FI (fi_FI.UTF-8); UI: en-US
Calc: threaded
Built on 30 September 2020
Comment 9 ⁨خالد حسني⁩ 2022-08-13 14:53:16 UTC
*** Bug 150395 has been marked as a duplicate of this bug. ***
Comment 10 ⁨خالد حسني⁩ 2022-08-13 14:53:51 UTC
https://gerrit.libreoffice.org/c/core/+/138217/2
Comment 11 Commit Notification 2022-08-14 19:14:10 UTC
Khaled Hosny committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/895843ad466954b63f6f99a1b6319b7813d0dbe1

tdf#103492: Messed Arabic letter spacing in text starting with LTR character(s)

It will be available in 7.5.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 12 Commit Notification 2022-08-15 09:49:12 UTC
Khaled Hosny committed a patch related to this issue.
It has been pushed to "libreoffice-7-4":

https://git.libreoffice.org/core/commit/8e2928ec42f2daf66c81b818a10efc99c1a14e89

tdf#103492: Messed Arabic letter spacing in text starting with LTR character(s)

It will be available in 7.4.1.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 13 Commit Notification 2022-08-15 22:27:35 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/c1446da82b999349e1a09fed3420bd1c38d7b38c

tdf#103492: vcl_pdfexport: Add unittest

It will be available in 7.5.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 14 Xisco Faulí 2022-08-23 09:00:26 UTC
*** Bug 150544 has been marked as a duplicate of this bug. ***
Comment 15 ⁨خالد حسني⁩ 2022-08-29 20:00:12 UTC
*** Bug 149889 has been marked as a duplicate of this bug. ***
Comment 16 ⁨خالد حسني⁩ 2022-09-11 20:00:47 UTC
*** Bug 119960 has been marked as a duplicate of this bug. ***
Comment 17 Stéphane Guillou (stragu) 2022-12-07 16:40:58 UTC
Fix verified:

Version: 7.5.0.0.alpha1+ (X86_64) / LibreOffice Community
Build ID: da3dd48eaf9086f8ab28d6a6655f9a638e51433a
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded
Comment 18 Commit Notification 2023-12-05 13:24:50 UTC
Chris Sherlock committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/9b5964cf5931d7c09e4fd624d68595891c2afb58

vcl: add unit test for cached glyphs based on tdf#103492

It will be available in 24.2.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.