Issue #2152205 by joelpittet, mark.labrecque, Manuel Garcia, pakmanlh, steveoliver, hussainweb, shanethehat, jenlampton, kpa, AnythonyR, EVIIILJ, kgoel, Cottser, dsdeiz, hanpersand: Convert theme_date() to #theme input__date
Task
Convert theme_date() to #theme input__date.
Remaining tasks
PatchPatch reviewManual testingProfilingNot needed because it's not a template conversion - JP
Steps to test
@todo
| Comment | File | Size | Author |
|---|---|---|---|
| #21 | drupal-2152205-21.patch | 2.95 KB | mark.labrecque |
| #18 | drupal-2152205-18.patch | 2.92 KB | manuel garcia |
| #15 | 2152205-theme_date-15.patch | 2.92 KB | pakmanlh |
| #11 | 2152205-theme_date-11.patch | 2.91 KB | joelpittet |
| #11 | interdiff.txt | 835 bytes | joelpittet |
Comments
Comment #1
star-szrAdding a commit message to the issue summary so the folks who already worked on #1898480: [meta] form.inc - Convert theme_ functions to Twig and in the Twig sandbox get credit.
Comment #2
joelpittetSplit from form.inc twig conversion.
Comment #3
joelpittetRe-rolled.
Comment #5
joelpittetThis issue was a typo from core.
Comment #6
joelpittetComment #8
joelpittetAnd another typo... geez
Comment #9
joelpittetright interdiff...
Comment #10
joelpittetFYI, this is a typo bug fix, that likely wouldn't have broken anything but could have since there is no 'attribute' key and it would always get set.
Comment #11
joelpittetFixed an issue with the patch. Sorry @rfay, got excited with my new assignment abilities.
Also, here's a markup review, note the fix.

Comment #12
joelpittetUnassigned and likely doesn't need profiling as there is no twig template in this. Could use a review and RTBC;)
Comment #13
star-szrTagging for reroll.
Comment #14
pakmanlhComment #15
pakmanlhRerolled.
Comment #16
joelpittetPrefect re-roll, thank you @pakmanlh!
Now just need someone to do a final review/RTBC
Comment #17
manuel garcia commentedRe-rolling...
Comment #18
manuel garcia commentedComment #19
star-szrThanks @Manuel Garcia! Looks like this needs another reroll now that theme_checkboxes has been converted. Tagging for reroll.
Comment #20
mark.labrecqueComment #21
mark.labrecqueComment #22
mark.labrecqueComment #23
mark.labrecqueComment #24
joelpittetReviewed #11, Don't need profiling because it's not a template.
Fixes a bug as demoed in screenshot in #11.
Removes a theme function.
Comment #25
star-szrPatch looks good to me, updating suggested commit message.
Comment #26
catchCommitted/pushed to 8.x, thanks!