Uploaded image for project: 'Axis'
  1. Axis
  2. AXIS-2622

Java Property Naming Convention

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 1.4
    • None
    • None
    • Linux, jdk1.5

    Description

      In class org.apache.axis.wsdl.toJava.JavaBeanHelperWriter, there is method that try to conform javabeans spec about java properties naming convention.

      The method :
      private String getAsFieldName(String fieldName)

      I have made some comparaison and build a tab which summarize that :
      XML Name Axis Generated JavaBean ElementDesc elemField.setFieldName
      al al al
      aL aL AL
      Al al al
      AL AL AL

      You can see that there is a problem on the second line... ElementDescriptor will use AL and aL is defined in the bean...

      SPECIFICATION JavaBeans 1.01 : 8.8 Capitalization of inferred names.

      When we use design patterns to infer a property or event name, we need to decide what rules
      to follow for capitalizing the inferred name. If we extract the name from the middle of a normal
      mixedCase style Java name then the name will, by default, begin with a capital letter.
      Java programmers are accustomed to having normal identifiers start with lower case letters.
      Vigorous reviewer input has convinced us that we should follow this same conventional rule
      for property and event names.

      Thus when we extract a property or event name from the middle of an existing Java name, we
      normally convert the first character to lower case. However to support the occasional use of all
      upper-case names, we check if the first two characters of the name are both upper case and if
      so leave it alone. So for example,
      "FooBah" becomes "fooBah"
      "Z" becomes "z"
      "URL" becomes "URL"
      We provide a method Introspector.decapitalize which implements this conversion rule.

      Attachments

        Activity

          People

            Unassigned Unassigned
            souillac charles SOUILLARD
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: