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
F18831004: D8467.id.diff
Sat, Oct 25, 10:07 AM
F18828009: D8467.diff
Fri, Oct 24, 1:39 PM
F18245504: D8467.id.diff
Aug 21 2025, 9:59 AM
F18231157: D8467.diff
Aug 20 2025, 1:43 PM
Unknown Object (File)
Jun 28 2025, 12:05 PM
Unknown Object (File)
May 10 2025, 5:29 AM
Unknown Object (File)
Apr 27 2025, 12:27 AM
Unknown Object (File)
Apr 21 2025, 8:17 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).