Send date time is listed as 12:xx a.m. instead of 12:xx p.m. in Message Status when using 12h Time notation pref in MM sttgs

sdemps3 - September 8, 2005 - 01:49
Project:Mass Mailer
Version:HEAD
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

When configuring "Time notation preference:" as 12h in massmailer settings (accessible at administer >> settings >> massmailer) then composing a message for a mailing list and setting its send date time to any time during the hour of 12:00 p.m., the message will be queued with a send date time of 12:00 a.m. instead. If the auto-send feature was working (But it is not -- See defect http://drupal.org/node/30549) the message would not be sent until the next morning.

Initial Conditions
1. Massmailer module must be enabled in: http://civicrm.civicspacelabs.com/home/admin/modules

Steps
1. Configure massmailer settings: http://civicrm.civicspacelabs.com/home/admin/settings/massmailer
2. In General Options, set "Time notation preference:" to 12h
3. Save Configuration.
4. Create a Mailing List: http://civicrm.civicspacelabs.com/home/admin/massmailer
5. Manage the Mailing List: http://civicrm.civicspacelabs.com/home/admin/massmailer/list/manage/#
6. Assign Current Subscribers, Add them manually by entering e-mail addresses or Subscribe a CiviCRM Group.
7. Add Subscribers.
8. Compose a message for the list: http://civicrm.civicspacelabs.com/home/admin/massmailer/message/send
9. For List Assignments, check the list created in Step #4.
10.Set the Send date to: Current Month, Today's Date, Current Year, 12, Current Min, pm (Note: If it is already past 12:00 p.m. when you
regress this defect or run this test, you are queueing a message to be sent in the past)
11.Queue Message
12.View message status: http://civicrm.civicspacelabs.com/home/admin/massmailer/message
13.Select the message from the list and click view.

Results
14. The "Send Date:" for the message will read, "yyyy-mm-dd 00:xx:00" where
dd = today's date + 1
xx = current min

Expected Results
14. The "Send Date:" should read "yyyy-mm-dd 12:xx:00" with dd being the today's date

Notes
1. If you use am instead of pm, the defect's logic is reversed. Message status will show the time stamp as "12:xx:00" for military time.

#1

sdemps3 - September 8, 2005 - 21:16
Status:active» closed

This issue occurred for me because I was using inputting a send date for the message as PST instead of GMT Greenwhich time which the host server is configured to use. So whenever I input a time of 12:00 p.m., the time is reset to 00:xx a.m. the next day since GMT time flips the 24 hour clock over after 11:59 p.m.

I will file a feature request on using Server's Time Zone vs. Admin's Client Time Zone later..

#2

sdemps3 - September 9, 2005 - 20:39
Priority:normal» critical
Status:closed» active

I'm reopening this defect becuase Gerhard is fixing the Time Zone setting issue that causes it.. I can regress the time zone setting issue by using this defect as one of the side effects.. The spec he is fixing is to configure mass mailer to send messages from the host server based on the admin's client Time Zone settings specified in admininster >> settings >> Date settings

#3

killes@www.drop.org - September 9, 2005 - 23:45
Status:active» fixed

The tz issue has been fixed.

the ampm issue has been fixed.

#4

sdemps3 - September 10, 2005 - 01:21
Status:fixed» closed

Fix verified...

#5

sdemps3 - September 10, 2005 - 01:39

This is truly fixed, but the time zone issues are not.. I'll file a new defect for massmailer about this.. I created a message to be sent to me 7 hours from now (1:00 a.m. PST) and it was e-mailed to me immediately by massmailer auto-send... Most likely because it was going off of the host's server's time..

#6

sdemps3 - September 10, 2005 - 02:18

Sorry.. I made a huge mistake.. Time Zone is fixed and not broken.. Everything is good to go.. When I created the message and set send date and time 7 hours into the future, I forgot to flip the date up by 1..

 
 

Drupal is a registered trademark of Dries Buytaert.