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

Comments

vitalius2009 created an issue. See original summary.

vitalius2009’s picture

vitalius2009 commented
StatusFileSize
new scheduler-additional_formats_with_spaces-2825987-D7.patch1.24 KB
jonathan1055’s picture

jonathan1055 commented
StatusFileSize
new 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 commented
Status: Needs review » Fixed

Committed.

jonathan1055’s picture

jonathan1055 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.