[ http://issues.apache.org/jira/browse/BEEHIVE-892?page=all ]

Eddie O'Neil updated BEEHIVE-892:
---------------------------------

    Fix Version: TBD
      Assign To:     (was: Mike Foster)

Fixing some fo the fields -- Zach, unfortunately, (3) isn't going to be an 
option once Beehive 1.0 ships.  So, if we want to do that, we need to get a 
discussion going on the beehive-dev@ list pretty quick.  :)

> AnnotationMemberTypes.Decimal.places() should have new default value
> --------------------------------------------------------------------
>
>          Key: BEEHIVE-892
>          URL: http://issues.apache.org/jira/browse/BEEHIVE-892
>      Project: Beehive
>         Type: Improvement
>   Components: Controls
>     Versions: V1
>     Reporter: Zach Smith
>     Priority: Minor
>      Fix For: TBD

>
> Currently AnnotationMemberTypes.Decimal.places() has a default value of '0' 
> which in my opinion causes some unecessary work on the part of the Controls 
> developer.  If a developer sets an AnnotationMemberTypes.Decimal and neglects 
> to set places() greater than '0' no decimal places can be used (fails checks 
> at compile time).  This does not seem intuitive to me given that a Decimal by 
> very nature has a decimal place assumed.  I propose that one of three things 
> be done with places()
> 1. Set it in a way such that it represents unlimited decimal places (limited 
> by the system, anyway)
> 2. If the main use case for this is currency consider setting places() 
> default value to '2' (however this could be confusing as well and perhaps a 
> BasicCurrency type should be created with this value)
> 3.  Do away with 'places()' all together and leave it to the developer to 
> programatically restrict decimal values.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to