Fop
  1. Fop
  2. FOP-1914

[PATCH] Accessibilty - Incorrect parent tree for links

    Details

    • Type: Bug Bug
    • Status: Closed
    • Resolution: Fixed
    • Affects Version/s: trunk
    • Fix Version/s: None
    • Component/s: unqualified
    • Labels:
      None
    • Environment:
      Operating System: All
      Platform: All
    • External issue ID:
      50987

      Description

      FOP puts in the parent tree for links the link itself instead of the related structure element.

      diff --git a/src/java/org/apache/fop/render/pdf/PDFLogicalStructureHandler.java b/src/java/org/apache/fop/render/pdf/PDFLogicalStructureHandler.java
      — a/src/java/org/apache/fop/render/pdf/PDFLogicalStructureHandler.java
      +++ b/src/java/org/apache/fop/render/pdf/PDFLogicalStructureHandler.java
      @@ -314,7 +314,6 @@ class PDFLogicalStructureHandler {
      void addLinkContentItem(PDFLink link, String structurePointer) {
      int structParent = getNextParentTreeKey();
      link.setStructParent(structParent);

      • parentTree.getNums().put(structParent, link);
        PDFDictionary contentItem = new PDFDictionary();
        contentItem.put("Type", OBJR);
        contentItem.put("Pg", this.currentPage);
        @@ -322,5 +321,6 @@ class PDFLogicalStructureHandler { PDFStructElem parent = (PDFStructElem) structTreeMap.get(structurePointer); + parentTree.getNums().put(structParent, parent); parent.addKid(contentItem); }

        Activity

        Martin K created issue -
        Hide
        Andreas L. Delmelle added a comment -

        Fixed in r1088234.

        Thanks for the patch!

        Show
        Andreas L. Delmelle added a comment - Fixed in r1088234. Thanks for the patch!
        Hide
        Glenn Adams added a comment -

        batch transition to closed; if someone wishes to restore one of these to resolved in order to perform a verification step, then feel free to do so

        Show
        Glenn Adams added a comment - batch transition to closed; if someone wishes to restore one of these to resolved in order to perform a verification step, then feel free to do so

          People

          • Assignee:
            fop-dev
            Reporter:
            Martin K
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development