Page MenuHomePhabricator

Fix issue reported from github
ClosedPublic

Authored by btrahan on Oct 10 2013, 10:05 PM.
Tags
None
Referenced Files
F13280257: D7284.id16399.diff
Sun, Jun 2, 4:07 AM
F13279204: D7284.diff
Sat, Jun 1, 12:32 PM
F13265657: D7284.diff
Tue, May 28, 5:53 AM
F13246099: D7284.diff
Thu, May 23, 7:43 AM
F13189738: D7284.diff
Sat, May 11, 6:20 AM
Unknown Object (File)
May 4 2024, 5:32 AM
Unknown Object (File)
May 4 2024, 4:59 AM
Unknown Object (File)
May 1 2024, 12:36 AM

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPdb71bf6128c8: Fix issue reported from github
Summary

we filter the $actors above such that its possible to have no $actor anymore (if $actor is not a deliverable email address). ergo, make sure we have actor before we start calling methods.

Fixes github issue 403

Test Plan

logic on this one - not 100% sure how to easily reproduce

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Ah, thanks. I took a peek at this and imagined writing a similar patch, but then forgot about it instead. Pretty sure this is the way to go, though.