Download & Extend

Use format_date instead of date and time

Project:Currency Exchange
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

Comments

#1

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.

#2

Status:postponed» needs review

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

AttachmentSize
338560-also_return_a_timestamp-1.patch 6.15 KB

#3

Status:needs review» fixed

Commited with 778f392.

#4

Status:fixed» closed (fixed)

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

nobody click here