I think we need to do one more important addition. It would be more relevant to have an ability for using spaces before Ante meridiem or Post meridiem in the acceptable popup formats (scheduler.admin.inc).

CommentFileSizeAuthor
#3 2825987-3.additional_formats_with_spaces.patch1.75 KBjonathan1055
#2 scheduler-additional_formats_with_spaces-2825987-D7.patch1.24 KBvitalius2009
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vitalius2009 created an issue. See original summary.

vitalius2009’s picture

vitalius2009 CreditAttribution: vitalius2009 as a volunteer commented
FileSize
scheduler-additional_formats_with_spaces-2825987-D7.patch1.24 KB
jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 as a volunteer commented
FileSize
2825987-3.additional_formats_with_spaces.patch1.75 KB

Yes this seems reasonable.
Fixed coding standards and included the new formats in test.

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 as a volunteer commented
Status: Needs review » Fixed

Committed.

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 as a volunteer commented
Title: Additional popup formats with spaces » Additional datetime popup formats with space before a.m/p.m
Assigned: vitalius2009 » Unassigned

Unassigning now this is fixed.

Status: Fixed » Closed (fixed)

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