starting to worry about open source code

Jan 26, 2012 17:18

sure, if you don't have data on a conversion rate for a particular currency, it's probably fine to silently assume it trades with the euro at 1:1, instead of throwing an error. it's just money, right? people don't get very picky about money.

Leave a comment

Comments 9

chienne_folle January 26 2012, 23:40:34 UTC
Wow. What a mistake to make!

What was the currency in question?

Reply

dilletante January 27 2012, 00:07:21 UTC
the currency? bitcoin. but from looking at the code i was complaining about (the default currency conversion module that ships with virtuemart, a reasonably successful open-source shopping cart), the behavior i described is the deliberately-programmed default for any case of a currency that isn't in the data provided by the source (an xml file published by the european central bank).

i don't think this is the right default.

Reply

moominmolly January 27 2012, 16:08:08 UTC
Simply amazing.

Reply


ceo January 27 2012, 00:27:25 UTC

That's going to have kind of unfortunate results in the case of, say, yen.

Reply

dilletante January 28 2012, 16:19:52 UTC
yen is unfortunate for the seller, bitcoin is unfortunate for the buyer. in neither case is it a good idea!

though i suppose with btc the sale just won't go through when the buyer says "that's not right." whereas with yen you'll have lots of sales as every prospective buyer says "hey, i'll take 100!"

yen is of course covered by the database. maybe zimbabwean dollars...

Reply


twoeleven January 27 2012, 03:27:39 UTC
you're just starting to worry?

Reply


justinjs January 27 2012, 16:01:50 UTC
The difference here between open source code and closed code is of course that with the open source code it is possible for a concerned observer like yourself to discover such problems.

Reply

dilletante January 28 2012, 16:21:24 UTC
well, it's easy to discover the problems without looking at the code. the difference with open source code is the urge to write "you lie like a rug!" after every comment block.

Reply


darxus January 30 2012, 19:24:40 UTC
Ack. You reported the bug, right?

Reply


Leave a comment

Up