Skip to content

PDFBOX-6281: Wrong order of characters for Arabic text with ActualText with PDFTextStripper, Add ActualText to awt/GlyphLayoutBidiTest.java, awt/GlyphLayoutSMPTest.java, awt/GlyphLayoutDin91379FormTest.java - #536

Open
vk-github18 wants to merge 6 commits into
apache:trunkfrom
vk-github18:27-09-2026-with-current-trunk-test-arabic

Conversation

@vk-github18

@vk-github18 vk-github18 commented Sep 27, 2026 •

Copy link
Copy Markdown

Add a test for ActualText in GlyphLayoutBidiTest
See https://issues.apache.org/jira/browse/PDFBOX-6281

The text is written correctly into ActualText in the PDF file.
The order of the text extracted from PDFTextStripper is wrong.
I assume the text would be correct if PDFTextStripper took the unchanged text from ActualText and does not reorder the text.

image

2026-10-11: Added a fix for PDFTextStripper, the result is now correct

@vk-github18 vk-github18 changed the title Add ActualText to GlyphLayoutBidiTest PDFBOX-6256: Add ActualText to GlyphLayoutBidiTest Sep 29, 2026
@vk-github18 vk-github18 changed the title PDFBOX-6256: Add ActualText to GlyphLayoutBidiTest PDFBOX-6256: Add ActualText to awt/GlyphLayoutSMPTest.java, awt/GlyphLayoutDin91379FormTest.java, and awt/GlyphLayoutBidiTest.java Oct 1, 2026
@vk-github18 vk-github18 changed the title PDFBOX-6256: Add ActualText to awt/GlyphLayoutSMPTest.java, awt/GlyphLayoutDin91379FormTest.java, and awt/GlyphLayoutBidiTest.java PDFBOX-6256: Add ActualText to awt/GlyphLayoutBidiTest.java, awt/GlyphLayoutSMPTest.java, awt/GlyphLayoutDin91379FormTest.java Oct 1, 2026
Fixes the extraction of ActualText for Arabic.
@vk-github18 vk-github18 changed the title PDFBOX-6256: Add ActualText to awt/GlyphLayoutBidiTest.java, awt/GlyphLayoutSMPTest.java, awt/GlyphLayoutDin91379FormTest.java PDFBOX-6281: Wrong order of characters for Arabic text with ActualText with PDFTextStripper, Add ActualText to awt/GlyphLayoutBidiTest.java, awt/GlyphLayoutSMPTest.java, awt/GlyphLayoutDin91379FormTest.java Oct 11, 2026
asf-gitbox-commits pushed a commit that referenced this pull request Oct 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant