recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

Conpherence - fix send bug for users with no javascript

Summary: we should do a redirect if its not an ajax request, which solves this problem since folks with no javascript aren't sending ajax requests. Fixes T5049.

Test Plan: inverted the predicate, tried ajax requests, and got redirected

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T5049

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

Bob Trahan 45f4431a 302d5bce

+1 -1
+1 -1
src/applications/conpherence/controller/ConpherenceUpdateController.php
··· 32 32 33 33 $action = $request->getStr('action', ConpherenceUpdateActions::METADATA); 34 34 $latest_transaction_id = null; 35 - $response_mode = 'ajax'; 35 + $response_mode = $request->isAjax() ? 'ajax' : 'redirect'; 36 36 $error_view = null; 37 37 $e_file = array(); 38 38 $errors = array();