Files: 
CommentFileSizeAuthor
#9 drupal_remove_theme_datetime-2008986-9.patch897 bytesmarkie
PASSED: [[SimpleTest]]: [MySQL] 56,007 pass(es). View
#2 drupal_remove_theme_datetime-2008986-2.patch867 bytesmarkie
PASSED: [[SimpleTest]]: [MySQL] 57,898 pass(es). View

Comments

thedavidmeister’s picture

Issue tags: +Novice
markie’s picture

Status: Active » Needs review
FileSize
867 bytes
PASSED: [[SimpleTest]]: [MySQL] 57,898 pass(es). View

only found one instance in datetime.

Status: Needs review » Needs work
Issue tags: -Novice

The last submitted patch, drupal_remove_theme_datetime-2008986-2.patch, failed testing.

thedavidmeister’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work
Issue tags: +Novice

The last submitted patch, drupal_remove_theme_datetime-2008986-2.patch, failed testing.

thedavidmeister’s picture

Could we call the array $form_required_marker instead of $array? no idea why the testbots hate this :/

thedavidmeister’s picture

Status: Needs work » Needs review
markie’s picture

I'll convert it this afternoon.

markie’s picture

FileSize
897 bytes
PASSED: [[SimpleTest]]: [MySQL] 56,007 pass(es). View

updated.

azinoman’s picture

Status: Needs review » Reviewed & tested by the community

This patch works well. I tested the date time features on drupal.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed f6f0478 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.