Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

Commit 02c6b27

Browse files
author
jwngr
committed
Changed $changeEmail() version check to throw synchronous error
1 parent 06ff37e commit 02c6b27

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/FirebaseAuth.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -338,16 +338,16 @@
338338
* @return {Promise<>} An empty promise fulfilled once the email change is complete.
339339
*/
340340
changeEmail: function(credentials) {
341+
if (typeof this._ref.changeEmail !== 'function') {
342+
throw new Error('$firebaseAuth.$changeEmail() requires Firebase version 2.1.0 or greater.');
343+
}
344+
341345
var deferred = this._q.defer();
342346

343-
if (typeof this._ref.changeEmail !== 'function') {
344-
deferred.reject('$firebaseAuth.$changeEmail() requires Firebase version 2.1.0 or greater.');
345-
} else {
346-
try {
347-
this._ref.changeEmail(credentials, this._utils.makeNodeResolver(deferred));
348-
} catch (error) {
349-
deferred.reject(error);
350-
}
347+
try {
348+
this._ref.changeEmail(credentials, this._utils.makeNodeResolver(deferred));
349+
} catch (error) {
350+
deferred.reject(error);
351351
}
352352

353353
return deferred.promise;

0 commit comments

Comments
 (0)