Differential D14913

Fix for e-mail invites not working when auth provider has new user registration disabled
Needs ReviewPublic
Actions

Authored by syserr0r on Dec 30 2015, 4:10 PM.

Details

Reviewers

None

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T10071: New account e-mail invites do not allow registration on auth providers that do not allow registration

Required Signatures

L28 Phacility Individual Contributor License Agreement

Summary

Ref T10071. Fix for e-mail invites not working when new user registration is disabled on the auth provider.

Test Plan

Accepted an invite with registration disabled on auth provider
Verified user created OK

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 9861
Build 11880: arc lint + arc unit

Event Timeline

syserr0r updated this revision to Diff 36038.Dec 30 2015, 4:10 PM

syserr0r retitled this revision from to Fix for e-mail invites not working when auth provider has new user registration disabled.

syserr0r updated this object.

syserr0r edited the test plan for this revision. (Show Details)

syserr0r added a task: T10071: New account e-mail invites do not allow registration on auth providers that do not allow registration.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptDec 30 2015, 4:10 PM

Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript

Herald added a subscriber: epriestley. · View Herald Transcript

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

src/

applications/

auth/

controller/

PhabricatorAuthRegisterController.php

14 lines

Diff 36038

View Options

src/applications/auth/controller/PhabricatorAuthRegisterController.php

Fix for e-mail invites not working when auth provider has new user registration disabledNeeds ReviewPublicActions