Active
Project:
Drupal core
Version:
main
Component:
token system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
15 Oct 2010 at 21:35 UTC
Updated:
15 Jul 2025 at 21:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dave reidComment #2
dave reidSmall bug in the patch.
Comment #4
dave reidTry that again, fixing it for real.
Comment #5
dave reidComment #6
dave reidForgot 'type' => 'date' in the info declaration which only affects things like the token UI tree in token for contrib. This one should be good to go.
Comment #7
dave reidThis is now too late for D7 since it would be changing and making people's existing tokens not work.
Comment #8
dave reid#6: 943028-date-current-date-namespace-D7.patch queued for re-testing.
Comment #9
dave reidRe-rolled for D8.
Comment #10
dave reidMissed 'type' => 'date' on the definition of the current-date token.
Comment #11
jenlamptonCan a token type have a type?
when the current date tokens are generated, do we have to provide data:
or could we just allow them to grab the request time:
Comment #12
avpaderno#10: 943028-date-current-date-namespace.patch queued for re-testing.
Comment #14
osopolarThere should be a fallback to REQUEST_TIME if no data was provided, otherwise this would break current behavior.
[Edit] Sorry, I overlooked that this is about drupal 8. But anyway the
[date]token is currently the only way to pass a date value and have a fallback, if no value was provided. I use this to add a date in file paths. In migration I use the node create date but for new content (by node form) I want to use the REQUEST_TIME.Comment #15
mgiffordUnassigning stale issue. Hopefully someone else will pursue this.
Comment #29
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
Comment #30
berdirThis is still technically an issue, although token has provided a current-date token for ages.
I'm not 100% sure it's worth fixing, especially the part about requiring a date for date tokens as that's a BC break and even with BC layers would be quite hard to deal with as tokens tend to be used all over the place in config and content.