Paths

Table of Contentst

Set time and date on Calendar Date Control form
ClosedPublic
Actions

Authored by chad on Apr 9 2016, 6:52 PM.

Tags

None

Referenced Files

	F14005101: D15666.id37752.diff
	Sun, Oct 27, 7:22 AM

	F14004990: D15666.diff
	Sun, Oct 27, 6:28 AM

	F13975671: D15666.diff
	Oct 18 2024, 11:09 AM

	F13973589: D15666.id37752.diff
	Oct 18 2024, 1:34 AM

	Unknown Object (File)
	Oct 8 2024, 4:49 AM

	Unknown Object (File)
	Sep 5 2024, 7:23 PM

	Unknown Object (File)
	Sep 5 2024, 7:23 PM

	Unknown Object (File)
	Sep 5 2024, 7:23 PM

View All Files

Subscribers

CodeMouse92

Korvin

Tokens

"Like" token, awarded by CodeMouse92.

Details

Reviewers

epriestley
lpriestley

Maniphest Tasks

T10766: Calendar - Unhandled Exception

Commits

rP5ddaf5289862: (stable) Set time and date on Calendar Date Control form
rP2bdf8ae5a234: Set time and date on Calendar Date Control form

Summary

Recurring events will fatal a Calendar with this not set. newDateTime requires a date and time to be called property. I think this is correct fix? Fixes T10766

Test Plan

Build a recurring event, pull up /calendar/, see recurring events as expected. Previously, fatal.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 37752.Apr 9 2016, 6:52 PM

chad retitled this revision from to Set time and date on Calendar Date Control form.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added reviewers: epriestley, lpriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptApr 9 2016, 6:52 PM

chad updated this object.Apr 9 2016, 6:53 PM

chad added a task: T10766: Calendar - Unhandled Exception.

CodeMouse92 awarded a token.Apr 9 2016, 6:54 PM

CodeMouse92 added a subscriber: CodeMouse92.

Thanks!

This revision is now accepted and ready to land.Apr 9 2016, 6:56 PM

Closed by commit rP2bdf8ae5a234: Set time and date on Calendar Date Control form (authored by chad, committed by chad). · Explain WhyApr 9 2016, 7:01 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

view/

form/

control/

AphrontFormDateControlValue.php

2 lines

Diff 37754

View Options

src/view/form/control/AphrontFormDateControlValue.php

Set time and date on Calendar Date Control formClosedPublicActions