Ticket #3579 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

Use comma as decimal seperator

Reported by: pirsig@quisma.de Assigned to: bobbysmith007
Priority: normal Component: TimingAndEstimationPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description

Hi,

if I use a "," as decimal seperator which is the standard decimal seperator in germany i get this error:

hours changed from NaNh to NaNh.

Is it possible to implement local specific seperators for float values?

Thx,

Alex

Attachments

Change History

08/20/08 15:39:15 changed by bobbysmith007

um... I am just using python's builtin float() function which converts strings to floats.

I can probably preprocess the string though. I will try to get to this soon.

Russ

08/20/08 15:43:44 changed by bobbysmith007

Unfortunatly all of the numbers are still going to print with '.' characters (at least right now). Thats alot of work to fix that one right now.

08/20/08 15:50:20 changed by bobbysmith007

(In [4157]) re #3579 T&E 0.6.8 made the ticket_daemon handle numbers with , as the decimal separator

08/20/08 15:50:39 changed by bobbysmith007

(In [4158]) re #3579 T&E 0.6.8 made the ticket_daemon handle numbers with , as the decimal separator

08/21/08 10:29:43 changed by pirsig@quisma.com

Hi bobbysmith007,

thx for the quick response and the fix!! I've upgraded to 0.6.8 and it works if i put values in "Add Hours to Ticket"-field but if i do the same with "Estimated Number of Hours"-field. I get: "Estimated Number of Hours: NaNh?"

Thx,

Alex

08/21/08 17:01:22 changed by bobbysmith007

doh... Sorry I didnt even think of that because I dont ever interact with the estimated hours field from python.

I will try to have a similar patch up for you soon that handles estimated hours.

08/21/08 18:12:31 changed by bobbysmith007

(In [4164]) re #3579 this should hopefully successfully convert all incoming hour strings into a form where decimals are separated by . rather than ,

08/21/08 18:13:17 changed by bobbysmith007

  • status changed from new to closed.
  • resolution set to fixed.

(In [4165]) closes #3579 updated version to 0.6.9

08/21/08 18:14:03 changed by bobbysmith007

damn you are using trac 10. I will upload a new trac 10 version after lunch

08/21/08 19:29:51 changed by bobbysmith007

(In [4166]) re #3579

I found that this was creating extra changelog crap that it shouldnt have. This fixes that by not inserting changes that were not changes.

08/21/08 19:31:11 changed by bobbysmith007

(In [4168]) re #3579

Updating this to better support entry of floats that use comma as the decimal separator

version 0.6.9

08/21/08 19:51:51 changed by bobbysmith007

(In [4170]) closes #3588 re #3579 thanks for catching that, this patch fixes a bug where i would occasionaly try to replace on a value I wasnt sure was a string

08/21/08 19:52:15 changed by bobbysmith007

(In [4171]) closes #3588 re #3579 thanks for catching that, this patch fixes a bug where i would occasionaly try to replace on a value I wasnt sure was a string


Add/Change #3579 (Use comma as decimal seperator)




Change Properties
Action