Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thedavidmeister’s picture

Issue tags: +Novice
markie’s picture

Status: Active » Needs review
FileSize
867 bytes

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

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.