Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1
    • Component/s: Aegis Module
    • Labels:
      None
    • Number of attachments :
      2

      Description

      For subclasses without a mapping file, mappings for superclasses are ignored. A workaround is to provide mapping files for such subclasses.

      Mappings files for keyType and componentType of maps are also not read.
      Investigation shows that XMLTypeCreator.createMapType delegates to nextCreator, which is a DefaultTypeCreator, since there are no mapping file for java.util.Map. DefaultTypeCreator then creates the keyType and componentType without bothering to check for mapping files.

      A patch is included for the latter case.

      1. XMLTypeCreator.java.patch
        1 kB
        Řyvind Matheson Wergeland
      2. XMLTypeCreator.java.patch
        1 kB
        Řyvind Matheson Wergeland

        Activity

        Hide
        Mika Goeckel added a comment -

        Patch applied.

        Show
        Mika Goeckel added a comment - Patch applied.
        Hide
        Mika Goeckel added a comment -

        Oops.. only the second issue is in fact resolved. I look into the first one now.

        Show
        Mika Goeckel added a comment - Oops.. only the second issue is in fact resolved. I look into the first one now.
        Hide
        Mika Goeckel added a comment -

        Fixed parent searching for mapping files.

        Show
        Mika Goeckel added a comment - Fixed parent searching for mapping files.
        Hide
        Řyvind Matheson Wergeland added a comment -

        Mapping files for implemented/extended interfaces should also be read.
        I am attatching a patch for this.

        Show
        Řyvind Matheson Wergeland added a comment - Mapping files for implemented/extended interfaces should also be read. I am attatching a patch for this.
        Hide
        Řyvind Matheson Wergeland added a comment -

        Patch to read mapping files for implemented/extended interfaces.
        Also contains a minor update.
        Based on the 1.0 release.

        Show
        Řyvind Matheson Wergeland added a comment - Patch to read mapping files for implemented/extended interfaces. Also contains a minor update. Based on the 1.0 release.
        Hide
        Dan Diephouse added a comment -

        Was this ever integrated?

        Show
        Dan Diephouse added a comment - Was this ever integrated?
        Hide
        Mika Goeckel added a comment -

        Not fully. I tried to do it quickly but actually need a closer look.

        Show
        Mika Goeckel added a comment - Not fully. I tried to do it quickly but actually need a closer look.
        Hide
        Mika Goeckel added a comment -

        Added testcases and the patch.

        Show
        Mika Goeckel added a comment - Added testcases and the patch.

          People

          • Assignee:
            Mika Goeckel
            Reporter:
            Řyvind Matheson Wergeland
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: