Page MenuHomePhabricator

Fix three minor edge case behaviors in Conpherence
ClosedPublic

Authored by epriestley on Mar 9 2014, 3:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 4:14 PM
Unknown Object (File)
Fri, Dec 20, 10:51 AM
Unknown Object (File)
Thu, Dec 12, 10:06 AM
Unknown Object (File)
Fri, Nov 29, 10:19 PM
Unknown Object (File)
Fri, Nov 29, 10:19 PM
Unknown Object (File)
Wed, Nov 27, 10:52 PM
Unknown Object (File)
Wed, Nov 27, 1:33 PM
Unknown Object (File)
Sun, Nov 24, 11:49 PM
Subscribers
Tokens
"Piece of Eight" token, awarded by btrahan.

Details

Summary

Couple of tweaks:

  • If a conpherence has no participants, we fail to attachParticipants(). This can happen if you leave a Conpherence as the last participant, then visit the URI again explicitly.
  • If you can't load any transactions (usually, because you don't have permission to view a thread's transactions), we try to attach null instead of array(). This can happen if you attempt to view a thread you don't have permission to see. A more general fix would be to tweak the load/filtering order, but I'm leaving that for another time since it's more involved and only gives us a small performance gain in unusual sitautions.
  • initializeNewThread() should be declared static.
Test Plan
  • Viewed a thread with no participants, got proper policy error.
  • Viewed a thread I couldn't see, got proper policy error.
  • Grepped for initializeNewThread().

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 10 2014, 5:04 PM
epriestley updated this revision to Diff 20104.

Closed by commit rP042ab0ad9d03 (authored by @epriestley).