@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
Reduce some code duplication in PhortuneLandingController
Summary: Ref T12451. This code is the same as the other code.
Test Plan: Went through the default-account case with this code, worked the same as the other code.
Reviewers: chad
Reviewed By: chad
Maniphest Tasks: T12451
Differential Revision: https://secure.phabricator.com/D17693
+3
-11
+3
-11
src/applications/phortune/controller/PhortuneLandingController.php
+3
-11
src/applications/phortune/controller/PhortuneLandingController.php
···
5
5
public function handleRequest(AphrontRequest $request) {
6
6
$viewer = $request->getViewer();
7
7
8
-
$accounts = id(new PhortuneAccountQuery())
9
-
->setViewer($viewer)
10
-
->withMemberPHIDs(array($viewer->getPHID()))
11
-
->execute();
12
-
13
-
if (!$accounts) {
14
-
$account = PhortuneAccount::createNewAccount(
15
-
$viewer,
16
-
PhabricatorContentSource::newFromRequest($request));
17
-
$accounts = array($account);
18
-
}
8
+
$accounts = PhortuneAccountQuery::loadAccountsForUser(
9
+
$viewer,
10
+
PhabricatorContentSource::newFromRequest($request));
19
11
20
12
if (count($accounts) == 1) {
21
13
$account = head($accounts);