It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.
Part of #2006152: [meta] Don't call theme() directly anywhere outside drupal_render().
only found one instance in datetime.
The last submitted patch, drupal_remove_theme_datetime-2008986-2.patch, failed testing.
#2: drupal_remove_theme_datetime-2008986-2.patch queued for re-testing.
Could we call the array $form_required_marker instead of $array? no idea why the testbots hate this :/
I'll convert it this afternoon.
updated.
This patch works well. I tested the date time features on drupal.
Committed f6f0478 and pushed to 8.x. Thanks!
Automatically closed -- issue fixed for 2 weeks with no activity.
Comments
Comment #1
thedavidmeister CreditAttribution: thedavidmeister commentedComment #2
markie CreditAttribution: markie commentedonly found one instance in datetime.
Comment #4
thedavidmeister CreditAttribution: thedavidmeister commented#2: drupal_remove_theme_datetime-2008986-2.patch queued for re-testing.
Comment #6
thedavidmeister CreditAttribution: thedavidmeister commentedCould we call the array $form_required_marker instead of $array? no idea why the testbots hate this :/
Comment #7
thedavidmeister CreditAttribution: thedavidmeister commented#2: drupal_remove_theme_datetime-2008986-2.patch queued for re-testing.
Comment #8
markie CreditAttribution: markie commentedI'll convert it this afternoon.
Comment #9
markie CreditAttribution: markie commentedupdated.
Comment #10
azinoman CreditAttribution: azinoman commentedThis patch works well. I tested the date time features on drupal.
Comment #11
alexpottCommitted f6f0478 and pushed to 8.x. Thanks!