PDFBox
  1. PDFBox
  2. PDFBOX-1344

xml namespace problem in ResourceRef

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7.0
    • Fix Version/s: 1.8.0
    • Component/s: JempBox
    • Labels:
      None

      Description

      Working on xmpbox and jempbox, I encountered a problem with ResourceRef in jempbox

      The attached example fails with current version of jempbox

      Reading dom.Element javadoc, I understand that is because le namespaceURI is not good. Changing it to the expected constants fixes the problem.

      But I am not sure, I understand all the problem... So if someone has 5 minuts to look at it and comment that issue to validate or invalidate the patch ... let do it.
      The patch is really short.

      Thanks

      1. JempboxCreateXmpMedia.java
        0.8 kB
        Guillaume Bailleul
      2. resourceref.patch
        0.7 kB
        Guillaume Bailleul

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        39d 19h 12m 1 Guillaume Bailleul 28/Jul/12 15:49
        Guillaume Bailleul made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 1.8.0 [ 12321650 ]
        Resolution Fixed [ 1 ]
        Hide
        Guillaume Bailleul added a comment -

        Fix done according to Andreas comment (with XMPSchema.NS_NAMESPACE)

        Show
        Guillaume Bailleul added a comment - Fix done according to Andreas comment (with XMPSchema.NS_NAMESPACE)
        Hide
        Andreas Lehmkühler added a comment -

        All other similar implementations are using XMPSchema.NS_NAMESPACE which is identical to XMLConstants.XMLNS_ATTRIBUTE_NS_URI. Maybe you should use it too to ensure consistency.

        Show
        Andreas Lehmkühler added a comment - All other similar implementations are using XMPSchema.NS_NAMESPACE which is identical to XMLConstants.XMLNS_ATTRIBUTE_NS_URI. Maybe you should use it too to ensure consistency.
        Guillaume Bailleul made changes -
        Description Working on xmpbox and jempbox, I encountered a problem with ResourceRef in jempbox

        The attached example fails with current version of jempbox

        Reading dom.Element javadoc, I understand that is because le namespaceURI is not good. Changing it to the expected constants fix the problem.

        But I am not sure, I understand all the problem... So if someone has 5 minuts to look at it and comment that issue to validate or invalidate the patch ... let do it.
        The patch is really short.

        Thanks



        Working on xmpbox and jempbox, I encountered a problem with ResourceRef in jempbox

        The attached example fails with current version of jempbox

        Reading dom.Element javadoc, I understand that is because le namespaceURI is not good. Changing it to the expected constants fixes the problem.

        But I am not sure, I understand all the problem... So if someone has 5 minuts to look at it and comment that issue to validate or invalidate the patch ... let do it.
        The patch is really short.

        Thanks



        Guillaume Bailleul made changes -
        Field Original Value New Value
        Attachment JempboxCreateXmpMedia.java [ 12532451 ]
        Attachment resourceref.patch [ 12532452 ]
        Guillaume Bailleul created issue -

          People

          • Assignee:
            Guillaume Bailleul
            Reporter:
            Guillaume Bailleul
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development