Securities with comma in name gives wrong data

flaviovs - October 25, 2007 - 17:04
Project:Stock API
Version:5.x-1.0
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

The CSV parsing code doesn't handle fields with embedded commas (e.g. "S&P 500 INDEX,RTH") and try to split the field, which results in bad data being returned and saved on the DB.

Try it for yourself with the S&P 500 index, code ^GSPC

The attached patch fix this issue.

AttachmentSize
stockapi-csvfix.patch832 bytes

#1

kbahey - October 26, 2007 - 04:03
Status:reviewed & tested by the community» fixed

Committed to 5.x and HEAD.

Should be available in the 5.x-dev branch soon.

#2

Anonymous - November 12, 2007 - 22:42
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.