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

Chad Schoettger resolved BEEHIVE-892.
-------------------------------------

    Fix Version/s: v.next
       Resolution: Fixed
         Assignee: matt c  (was: Chad Schoettger)

Implement option 1, made the default value for places the equivalent of no 
restriction on the default number of digits following the decimal point.

svn rev 448323

> AnnotationMemberTypes.Decimal.places() should have new default value
> --------------------------------------------------------------------
>
>                 Key: BEEHIVE-892
>                 URL: http://issues.apache.org/jira/browse/BEEHIVE-892
>             Project: Beehive
>          Issue Type: Improvement
>          Components: Controls
>    Affects Versions: V1
>            Reporter: Zach Smith
>         Assigned To: matt c
>            Priority: Minor
>             Fix For: v.next
>
>
> 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