Great module. Well, only using the currency_api.module, but it was almost spot on to what I was looking for.
One slight problem, the date and time returned should be combined into a single value based on the format_date function ( http://api.drupal.org/api/function/format_date/5 ).
Right now it's a date/time value totally separated from everything else related to time in Drupal which is a no-no.

CommentFileSizeAuthor
#2 338560-also_return_a_timestamp-1.patch6.15 KBamateescu

Comments

amateescu’s picture

amateescu commented
Version: 5.x-1.3 » 6.x-1.x-dev
Status: Active » Postponed

The 5.x branch is not maintained anymore. Maybe we can fix this in 6.x.

amateescu’s picture

amateescu commented
Status: Postponed » Needs review
StatusFileSize
new 338560-also_return_a_timestamp-1.patch6.15 KB

Here's a fix for this issue. It changes the currency_api_convert() function to also return a timestamp.

amateescu’s picture

amateescu commented
Status: Needs review » Fixed

Commited with 778f392.

Status: Fixed » Closed (fixed)

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