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
F19061804: D8467.id.diff
Sat, Nov 29, 1:33 PM
F19053965: D8467.id.diff
Fri, Nov 28, 10:17 AM
F19048401: D8467.diff
Thu, Nov 27, 2:47 PM
F19047899: D8467.diff
Thu, Nov 27, 1:42 PM
F19043816: D8467.diff
Thu, Nov 27, 1:00 AM
F19037551: D8467.id20104.diff
Tue, Nov 25, 11:56 PM
F18871402: D8467.diff
Nov 4 2025, 10:33 PM
F18848878: D8467.diff
Oct 30 2025, 6:42 AM
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).