[PDFBOX-3576] Add getter methods to TextPosition

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.3
• Fix Version/s: 2.0.5, 3.0.0 PDFBox
• Assignee: Tilman Hausherr
• Reporter: Kurt Devlin
• Created: 16/Nov/16 18:00
• Updated: 01/Dec/19 09:18
• Resolved: 28/Jan/17 12:57

Description

I've inherited code in my department that created a local copy of TextPosition and broke it's immutability. I'm trying to refactor this code and use the core Apache implementation and I understand the need for TextPosition to be immutable, but can getters be added to provide read-only access to the class's fields?

We have custom code that does comparison of endX/endY values to determine if there are overlapping characters and other features. In our application, we also have a swap feature where we need to create/clone new TextPosition instances with minor differences between an existing TextPosition instance. Since there are a lot of fields that are required by the constructor of the new instances, but can't be read from an existing instance, we need a way to access these fields.

There probably should also be an override for equals() and hashCode().

Issue Links

• is related to: PDFBOX-3794 Problem in TextPosition implementation
• is related to: PDFBOX-4701 TextPosition.equal() fails after getDir()