Paths

Table of Contentst

Define payload property in Event class
ClosedPublic
Actions

Authored by dereckson on Dec 30 2016, 04:08.

Tags

None

Referenced Files

	F3211425: D781.id1975.diff
	Sat, Jun 29, 06:01

	F3211348: D781.id1976.diff
	Sat, Jun 29, 05:52

	F3208752: D781.id1975.diff
	Fri, Jun 28, 21:47

	Unknown Object (File)
	Thu, Jun 27, 13:04

	Unknown Object (File)
	Wed, Jun 26, 12:33

	Unknown Object (File)
	Wed, Jun 26, 05:19

	Unknown Object (File)
	Wed, Jun 26, 05:19

	Unknown Object (File)
	Wed, Jun 26, 03:14

View All Files

Subscribers

alken-orin

Details

Reviewers

dereckson

Commits

rNOTIF0b99e9b259b2: Define payload property in Event class

Summary

This allows to use code like:

function handleEvent (Event $event): void {
    $this->doStuff($event->payload); // This is now always valid.
}

Test Plan

Run tests, then phan | grep payload.

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson created this revision.Dec 30 2016, 04:08

Build is green https://ci.nasqueron.org/job/test-notifications-phab/245/ for more details.

dereckson accepted this revision.Dec 30 2016, 04:51

dereckson edited the summary of this revision. (Show Details)

dereckson edited edge metadata.

This revision is now accepted and ready to land.Dec 30 2016, 04:51

Closed by commit rNOTIF0b99e9b259b2: Define payload property in Event class (authored by dereckson, committed by dereckson). · Explain WhyDec 30 2016, 05:03

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app/

Events/

Event.php

11 lines

Diff 1976

View Options

Define payload property in Event classClosedPublicActions