better naming for oauth version variable

This commit is contained in:
Mike Bannister
2012-08-02 01:04:56 -04:00
committed by Nick Martin
parent 63eeec4708
commit ca40bf2875

View File

@@ -3,7 +3,7 @@
//
// @param state {String} The OAuth state generated by the client
// @param url {String} url to page
Meteor.accounts.oauth.initiateLogin = function(state, url, version) {
Meteor.accounts.oauth.initiateLogin = function(state, url, oauthVersion) {
// XXX these dimensions worked well for facebook and google, but
// it's sort of weird to have these here. Maybe an optional
// argument instead?
@@ -12,7 +12,7 @@
var checkPopupOpen = setInterval(function() {
if (popup.closed) {
clearInterval(checkPopupOpen);
tryLoginAfterPopupClosed(state, version);
tryLoginAfterPopupClosed(state, oauthVersion);
}
}, 100);
};
@@ -20,9 +20,9 @@
// Send an OAuth login method to the server. If the user authorized
// access in the popup this should log the user in, otherwise
// nothing should happen.
var tryLoginAfterPopupClosed = function(state, version) {
var tryLoginAfterPopupClosed = function(state, oauthVersion) {
Meteor.apply('login', [
{oauth: {version: version, state: state}}
{oauth: {oauthVersion: oauthVersion, state: state}}
], {wait: true}, function(error, result) {
if (error)
throw error;