webdav-jaxrs
  1. webdav-jaxrs
  2. WEBDAV_JAXRS-3

Replace @XmlElement by @XmlElementRef to get rid of redundant name attributes

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 1.2
    • Fix Version/s: 1.2
    • Component/s: www
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      3

      Description

      Currently most "@XmlElement"s are containing an explicit "name=" attribute to
      specifiy the name of the element used at each particular place. In fact, in
      virtually all places this is exactly the same name provided to the "name="
      attribute of the referenced JAXB class's "@XmlRootElement" annotation.

      Not only that this leads to cluttered and unnecessarily complex code, it also
      implies a potential risk of induced errors resulting from this redundancy (e.
      g. by typos in one or more places).

      To improve the readability of the code and to reduce the potential risk of
      errors, wherever possible the rule "Convention over configuration" shall get
      applied, i. e. @XmlElement(name = "") should get replaced by @XmlElementRef.

        Issue Links

          Activity

          Hide
          mkarg added a comment -

          Added to agenda for Release 2.

          Show
          mkarg added a comment - Added to agenda for Release 2.
          Hide
          mkarg added a comment -

          Currently in progress by mkarg.

          Show
          mkarg added a comment - Currently in progress by mkarg.
          Hide
          mkarg added a comment -

          No time to fix all locations now.

          Show
          mkarg added a comment - No time to fix all locations now.

            People

            • Assignee:
              webdav-jaxrs-issues
              Reporter:
              mkarg
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated: