Wrong use of "short" in "format_date" function.

I have changed the Priority status to "critical".

I do apologise, I thought only the maintainers are allowed to implement patches and review any code concerned.

But thank you for the advice, and I will do so in the future.

Oh and I will implement your patch and test it.

Thanks
Stephen

Thanks for the excellent pointers, and apologies again.

Regards
Stephen

I have implemented the patch, but when I use the "Token" values of "[yyyy]/[mm]/[dd]" to use with
the "Filefield" CCK module I now get a directory structure of "yyyy/mm/dd" and not "2008/10/16" as I had hoped.

I will investigate further.

Thanks
Stephen

The use of [site-date] should resolve to a file structure of (in my case UK date format) "17/10/2008".
But the result is only "17/10" the year is missing

The functions only work in the "Public" upload mode and not "Private" but that is something else, I will investigate further and determine if the issue is with the "Token" module.

Thanks
Stephen

I have changed the token.module as follows =>

- $values['site-date'] = format_date(time(), 'small', '', variable_get('date_default_timezone', 0));
+ $values['site-date'] = format_date(time(), 'custom', 'Y/m/d', variable_get('date_default_timezone', 0));

and everything works a treat both in the "Public" and "Private" upload methods.
I will try and figure out the why it works when I install some debugging tools on my laptop.

Thanks
Stephen