Bug 42307 - [PATCH] Java2d renderers render arabic text incorrectly.
Summary: [PATCH] Java2d renderers render arabic text incorrectly.
Status: CLOSED REMIND
Alias: None
Product: Fop - Now in Jira
Classification: Unclassified
Component: awt renderer (show other bugs)
Version: trunk
Hardware: Other All
: P2 critical
Target Milestone: ---
Assignee: fop-dev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-01 01:46 UTC by Richard Wheeldon
Modified: 2012-04-01 13:49 UTC (History)
0 users


Attachments
Patch to fix arabic text rendering with example (5.56 KB, patch)
2007-05-01 01:47 UTC, Richard Wheeldon 		Details | Diff
Fixed to work with latest trunk code and fixed some checkstyle problems (3.33 KB, patch)
2008-06-26 12:50 UTC, Adrian Cumiskey 		Details | Diff
Incorrect direction latin text inside arabic text (12.87 KB, image/gif)
2010-10-06 10:26 UTC, Dmitrij Golubev 		Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Richard Wheeldon 2007-05-01 01:46:27 UTC 
The characters are in the wrong order (e.g "apache fop" -> "pof ehcapa"), and
are not joined correctly.
Comment 1 Richard Wheeldon 2007-05-01 01:47:02 UTC 
Created attachment 20083 [details]
Patch to fix arabic text rendering with example
Comment 2 Richard Wheeldon 2007-05-08 01:43:48 UTC 
Don't use this patch. It's broken. It's not as broken as the current code, but
it still doesn't fix the word ordering problems so should be left until a full
fix is available.
Comment 3 Ravindra Adiga 2008-06-19 09:53:04 UTC 
Is there a patch fix available for this bug please. We have a customer in Middle East who is expected to go live with our product in less than 3 weeks and we are stuck with this arabic character ordering issue. Kindly help in fixing this issue at the earliest. Thanks a lot.
Comment 4 Adrian Cumiskey 2008-06-26 12:50:21 UTC 
Created attachment 22181 [details]
Fixed to work with latest trunk code and fixed some checkstyle problems

Applied Richard Wheeldon's patch to the latest trunk code and fixed some checkstyle issues.  Can someone who is able to read Arabic verify this fix please? :)
Comment 5 Pascal Sancho 2008-06-27 01:57:21 UTC 
In reply to comment #4:
I've applied the patch on rev indicated in, then tried to generate both AWT and PDF, using DejaVu font:

My colleague (arabic read/whrite) says that character order is right now, but character shaping is wrong (inner-word ligatures) : Cf http://en.wikipedia.org/wiki/Complex_Text_Layout_languages for further information.

HTH,
Pascal
Comment 6 Tarek Khattab 2009-03-17 08:37:20 UTC 
(In reply to comment #4)
> Created an attachment (id=22181) [details]
> Fixed to work with latest trunk code and fixed some checkstyle problems
> 
> Applied Richard Wheeldon's patch to the latest trunk code and fixed some
> checkstyle issues.  Can someone who is able to read Arabic verify this fix
> please? :)

But how to apply this patch.. please help me :)
Comment 7 Jeremias Maerki 2009-03-19 06:20:20 UTC 
(In reply to comment #6)
> But how to apply this patch.. please help me :)

See http://wiki.apache.org/general/ApplyingPatches

If you're using Eclipse, you can use "Team / Apply Patch...". TortoiseSVN under Windows also has an "Apply patch..." function.
Comment 8 Dmitrij Golubev 2010-10-06 10:26:46 UTC 
Created attachment 26125 [details]
Incorrect direction latin text inside arabic text

Hello

Direction of latin text inside arabic text. 
Got last version (1.0) from svn.

I am using <fo:block-container writing-mode="rl-tb" for setting direction of text.

as result direction of arabic is ok, but latin text mirrored
lorem Ipsum -> musqI meroL
Comment 9 Glenn Adams 2012-02-27 18:15:45 UTC 
Added complex script support (bidi, shaping, etc) at revision 1293736. However, support for Java2D output format is not yet wired, so stand by for further updates.
Comment 10 Glenn Adams 2012-04-01 13:49:41 UTC 
batch transition to closed remaining pre-FOP1.0 resolved bugs