• Resolved glenstewart

    (@glenstewart)


    In postie-tags.php (latest release) there seems to be a bug. One solution is shown, but another is proposed at the URL provided.

    Nice program – I hope this report helps it shine even more. Thanks!

    Starting at line 222:

    if (empty($message_date)) {
                $dateInSeconds = new DateTime(current_time('mysql'), new DateTimeZone($tzs));
                DebugEcho("tag_Delay: using current date: " . $dateInSeconds->format(DATE_RFC2822));
            } else {
                DebugEcho("tag_Delay: using message date(1): $message_date");
    			
    			// Glen trying to fix "double timezone specification" error on dates like:
    			// Mon, 06 Sep 2004 08:15:56 +0000 (America/Los_Angeles)
    			$message_date = strstr($message_date, " (", true); // Mon, 06 Sep 2004 08:15:56 +0000
    			// as documented at https://geeksqa.com/failed-to-parse-time-string-at-position-41-i-double-timezone-specification
    			
                $dateInSeconds = new DateTime($message_date, new DateTimeZone($tzs));
                DebugEcho("tag_Delay: using message date(2): " . $dateInSeconds->format(DATE_RFC2822));
            }
Viewing 2 replies - 1 through 2 (of 2 total)
  • Thread Starter glenstewart

    (@glenstewart)

    Correction – the earlier post core fix should be:

    // Mon, 06 Sep 2004 08:15:56 +0000 (America/Los_Angeles)
    			if (strstr($message_date, " (", true)) {
    			  $message_date = strstr($message_date, " (", true); // Mon, 06 Sep 2004 08:15:56 +0000
    			}
    			// as documented at https://geeksqa.com/failed-to-parse-time-string-at-position-41-i-double-timezone-specification
    Plugin Author Wayne Allen

    (@wayneallen-1)

    Excellent work! I’ve included this change in the next version of Postie.

Viewing 2 replies - 1 through 2 (of 2 total)

The topic ‘double timezone error – fix proposals provided’ is closed to new replies.