Add xsd support for Boolean attributes

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0-alpha-20
  • Fix Version/s: 1.0-alpha-22
  • Component/s: modello-plugin-xsd
  • Labels:
    None
  • Complexity:
    Intermediate

Description

boolean attributes are detected as xs:boolean schema primitive type
Boolean objects are not recognized as primitive type and cause an exception: java.lang.IllegalStateException: Non-association field of a non-primitive type 'Boolean' for 'xxx'

Issue Links

depends upon

Improvement - An improvement or enhancement to an existing feature or task. MODELLO-123 Add support for Integer, Long, Float and Double fields

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.
is depended upon by

Bug - A problem which impairs or prevents the functions of the product. MNG-1999 Reporting inheritance does not work properly

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Bug - A problem which impairs or prevents the functions of the product. MNG-3898 can't generate Maven site: modello fails at xsd generation step due to a Boolean field in model

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Herve Boutemy added a comment -

should it simply be represented like boolean as xs:boolean?
Or is there something more subtle to do to handle null value?
Any thought?

Show
Herve Boutemy added a comment - should it simply be represented like boolean as xs:boolean? Or is there something more subtle to do to handle null value? Any thought?
Hide
Permalink
Benjamin Bentmann added a comment - - edited

should it simply be represented like boolean as xs:boolean?

+1, the semantic still matches the definition of xs:boolean, i.e. the field's value space is {true, false}. Null is only used internally (on the Java side) to detect whether the field was set at all.

BTW, are the other primitive types and their wrappers properly handled, e.g. java.lang.Integer?

Show
Benjamin Bentmann added a comment - - edited
should it simply be represented like boolean as xs:boolean?
+1, the semantic still matches the definition of xs:boolean, i.e. the field's value space is {true, false}. Null is only used internally (on the Java side) to detect whether the field was set at all. BTW, are the other primitive types and their wrappers properly handled, e.g. java.lang.Integer?
Hide
Permalink
Herve Boutemy added a comment -

done in [r936|]

added support for every Object types equivalent to primitives: Integer, Boolean, Long, Float and Double
I'll update the issue title

Show
Herve Boutemy added a comment - done in [r936|] added support for every Object types equivalent to primitives: Integer, Boolean, Long, Float and Double I'll update the issue title
Hide
Permalink
Herve Boutemy added a comment -

after more testing, I found that Boolean was supported by java generator, but not Integer, Long, Float or Double
=> created a specific issue to add these types to java generator (and any other plugins needing changes)

Show
Herve Boutemy added a comment - after more testing, I found that Boolean was supported by java generator, but not Integer, Long, Float or Double => created a specific issue to add these types to java generator (and any other plugins needing changes)

People

  • Assignee:
    Herve Boutemy
    Reporter:
    Herve Boutemy
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: