HomePhabricator

Fix Diffusion blame/highlight for logged-out users

Description

Fix Diffusion blame/highlight for logged-out users

Summary:
Fixes T5199. We try to save these options in user preferences, but logged-out users don't have preferences.

Instead, just use GET links for logged-out users.

Test Plan:

  • As a logged-out user, toggled blame and highlight on and off.
  • As a logged-in user, toggled blame and highlight on and off.

Reviewers: btrahan, vrana

Reviewed By: vrana

Subscribers: epriestley

Maniphest Tasks: T5199

Differential Revision: https://secure.phabricator.com/D9310

Details

Provenance
epriestleyAuthored on
epriestleyPushed on May 28 2014, 12:37 AM
Reviewer
vrana
Differential Revision
D9310: Fix Diffusion blame/highlight for logged-out users
Parents
rP71d2b0868b73: Fix typo: behvaior->behavior
Branches
Unknown
Tags
Unknown
Tasks
T5199: Anonymous users couldn't display blame

Event Timeline