Skip to content

Removed date filter from failed_list.html.twig - #43

Merged
tonypiper merged 1 commit into
michelsalib:masterfrom
ruudk:patch-1
Mar 27, 2013
Merged

Removed date filter from failed_list.html.twig#43
tonypiper merged 1 commit into
michelsalib:masterfrom
ruudk:patch-1

Conversation

@ruudk

@ruudk ruudk commented Mar 27, 2013

Copy link
Copy Markdown
Collaborator

Otherwise you get this error:

An exception has been thrown during the rendering of a template ("DateTime::__construct(): Failed to parse time string (di mrt 26 12:46:53 UTC 2013) at position 0 (d): The timezone could not be found in the database") in BCCResqueBundle:Default:failed_list.html.twig at line 21.

This is because PHP-Resque uses a localized timestamp:

$data->failed_at = strftime('%a %b %d %H:%M:%S %Z %Y');

Otherwise you get this error:

An exception has been thrown during the rendering of a template ("DateTime::__construct(): Failed to parse time string (di mrt 26 12:46:53 UTC 2013) at position 0 (d): The timezone could not be found in the database") in BCCResqueBundle:Default:failed_list.html.twig at line 21.
@tonypiper

Copy link
Copy Markdown
Contributor

Grrr. Works OK with 'en' as a locale. Might it be worth submitting a PR to php-resque to use ISO 8601 for failed_at? Meanwhile, I will merge and tag.

tonypiper added a commit that referenced this pull request Mar 27, 2013
Removed date filter from failed_list.html.twig
@tonypiper
tonypiper merged commit 694f148 into michelsalib:master Mar 27, 2013
@tonypiper

Copy link
Copy Markdown
Contributor

tagged v1.1.6

@ruudk

ruudk commented Mar 27, 2013

Copy link
Copy Markdown
Collaborator Author

There is already a PR for this: chrisboulton/php-resque#95

@tonypiper

Copy link
Copy Markdown
Contributor

Great. Hope it gets merged soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants