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
F19523514: D8467.diff
Sat, Jan 17, 5:05 PM
F19506356: D8467.id20104.diff
Fri, Jan 9, 4:45 PM
F19061804: D8467.id.diff
Nov 29 2025, 1:33 PM
F19053965: D8467.id.diff
Nov 28 2025, 10:17 AM
F19048401: D8467.diff
Nov 27 2025, 2:47 PM
F19047899: D8467.diff
Nov 27 2025, 1:42 PM
F19043816: D8467.diff
Nov 27 2025, 1:00 AM
F19037551: D8467.id20104.diff
Nov 25 2025, 11:56 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).