oops, revert - do this on the right branch...
This commit is contained in:
parent
f70eec62d1
commit
f66dd69710
4 changed files with 13 additions and 23 deletions
|
@ -1033,6 +1033,7 @@ module.exports = React.createClass({
|
||||||
idSid={this.state.register_id_sid}
|
idSid={this.state.register_id_sid}
|
||||||
email={this.props.startingQueryParams.email}
|
email={this.props.startingQueryParams.email}
|
||||||
username={this.state.upgradeUsername}
|
username={this.state.upgradeUsername}
|
||||||
|
disableUsernameChanges={Boolean(this.state.upgradeUsername)}
|
||||||
guestAccessToken={this.state.guestAccessToken}
|
guestAccessToken={this.state.guestAccessToken}
|
||||||
defaultHsUrl={this.props.config.default_hs_url}
|
defaultHsUrl={this.props.config.default_hs_url}
|
||||||
defaultIsUrl={this.props.config.default_is_url}
|
defaultIsUrl={this.props.config.default_is_url}
|
||||||
|
|
|
@ -19,7 +19,7 @@ var ReactDOM = require("react-dom");
|
||||||
var GeminiScrollbar = require('react-gemini-scrollbar');
|
var GeminiScrollbar = require('react-gemini-scrollbar');
|
||||||
var q = require("q");
|
var q = require("q");
|
||||||
|
|
||||||
var DEBUG_SCROLL = false;
|
var DEBUG_SCROLL = true;
|
||||||
|
|
||||||
if (DEBUG_SCROLL) {
|
if (DEBUG_SCROLL) {
|
||||||
// using bind means that we get to keep useful line numbers in the console
|
// using bind means that we get to keep useful line numbers in the console
|
||||||
|
|
|
@ -43,6 +43,7 @@ module.exports = React.createClass({
|
||||||
email: React.PropTypes.string,
|
email: React.PropTypes.string,
|
||||||
username: React.PropTypes.string,
|
username: React.PropTypes.string,
|
||||||
guestAccessToken: React.PropTypes.string,
|
guestAccessToken: React.PropTypes.string,
|
||||||
|
disableUsernameChanges: React.PropTypes.bool,
|
||||||
// registration shouldn't know or care how login is done.
|
// registration shouldn't know or care how login is done.
|
||||||
onLoginClick: React.PropTypes.func.isRequired
|
onLoginClick: React.PropTypes.func.isRequired
|
||||||
},
|
},
|
||||||
|
@ -107,12 +108,6 @@ module.exports = React.createClass({
|
||||||
errorText: "",
|
errorText: "",
|
||||||
busy: true
|
busy: true
|
||||||
});
|
});
|
||||||
|
|
||||||
if (formVals.username !== this.props.username) {
|
|
||||||
// don't try to upgrade if we changed our username
|
|
||||||
this.registerLogic.setGuestAccessToken(null);
|
|
||||||
}
|
|
||||||
|
|
||||||
this.onProcessingRegistration(this.registerLogic.register(formVals));
|
this.onProcessingRegistration(this.registerLogic.register(formVals));
|
||||||
},
|
},
|
||||||
|
|
||||||
|
@ -210,6 +205,7 @@ module.exports = React.createClass({
|
||||||
showEmail={true}
|
showEmail={true}
|
||||||
defaultUsername={this.props.username}
|
defaultUsername={this.props.username}
|
||||||
defaultEmail={this.props.email}
|
defaultEmail={this.props.email}
|
||||||
|
disableUsernameChanges={this.props.disableUsernameChanges}
|
||||||
minPasswordLength={MIN_PASSWORD_LENGTH}
|
minPasswordLength={MIN_PASSWORD_LENGTH}
|
||||||
onError={this.onFormValidationFailed}
|
onError={this.onFormValidationFailed}
|
||||||
onRegisterClick={this.onFormSubmit} />
|
onRegisterClick={this.onFormSubmit} />
|
||||||
|
|
|
@ -38,6 +38,7 @@ module.exports = React.createClass({
|
||||||
defaultUsername: React.PropTypes.string,
|
defaultUsername: React.PropTypes.string,
|
||||||
showEmail: React.PropTypes.bool,
|
showEmail: React.PropTypes.bool,
|
||||||
minPasswordLength: React.PropTypes.number,
|
minPasswordLength: React.PropTypes.number,
|
||||||
|
disableUsernameChanges: React.PropTypes.bool,
|
||||||
onError: React.PropTypes.func,
|
onError: React.PropTypes.func,
|
||||||
onRegisterClick: React.PropTypes.func // onRegisterClick(Object) => ?Promise
|
onRegisterClick: React.PropTypes.func // onRegisterClick(Object) => ?Promise
|
||||||
},
|
},
|
||||||
|
@ -55,7 +56,7 @@ module.exports = React.createClass({
|
||||||
getInitialState: function() {
|
getInitialState: function() {
|
||||||
return {
|
return {
|
||||||
email: this.props.defaultEmail,
|
email: this.props.defaultEmail,
|
||||||
username: null,
|
username: this.props.defaultUsername,
|
||||||
password: null,
|
password: null,
|
||||||
passwordConfirm: null,
|
passwordConfirm: null,
|
||||||
fieldValid: {}
|
fieldValid: {}
|
||||||
|
@ -77,7 +78,7 @@ module.exports = React.createClass({
|
||||||
|
|
||||||
if (this.allFieldsValid()) {
|
if (this.allFieldsValid()) {
|
||||||
var promise = this.props.onRegisterClick({
|
var promise = this.props.onRegisterClick({
|
||||||
username: this.refs.username.value.trim() || this.props.defaultUsername,
|
username: this.refs.username.value.trim(),
|
||||||
password: this.refs.password.value.trim(),
|
password: this.refs.password.value.trim(),
|
||||||
email: this.refs.email.value.trim()
|
email: this.refs.email.value.trim()
|
||||||
});
|
});
|
||||||
|
@ -119,14 +120,13 @@ module.exports = React.createClass({
|
||||||
break;
|
break;
|
||||||
case FIELD_USERNAME:
|
case FIELD_USERNAME:
|
||||||
// XXX: SPEC-1
|
// XXX: SPEC-1
|
||||||
var username = this.refs.username.value.trim() || this.props.defaultUsername;
|
if (encodeURIComponent(this.refs.username.value) != this.refs.username.value) {
|
||||||
if (encodeURIComponent(username) != username) {
|
|
||||||
this.markFieldValid(
|
this.markFieldValid(
|
||||||
field_id,
|
field_id,
|
||||||
false,
|
false,
|
||||||
"RegistrationForm.ERR_USERNAME_INVALID"
|
"RegistrationForm.ERR_USERNAME_INVALID"
|
||||||
);
|
);
|
||||||
} else if (username == '') {
|
} else if (this.refs.username.value == '') {
|
||||||
this.markFieldValid(
|
this.markFieldValid(
|
||||||
field_id,
|
field_id,
|
||||||
false,
|
false,
|
||||||
|
@ -199,7 +199,7 @@ module.exports = React.createClass({
|
||||||
if (this.props.showEmail) {
|
if (this.props.showEmail) {
|
||||||
emailSection = (
|
emailSection = (
|
||||||
<input className="mx_Login_field" type="text" ref="email"
|
<input className="mx_Login_field" type="text" ref="email"
|
||||||
autoFocus={true} placeholder="Email address (optional)"
|
autoFocus={true} placeholder="Email address"
|
||||||
defaultValue={this.state.email}
|
defaultValue={this.state.email}
|
||||||
style={this._styleField(FIELD_EMAIL)}
|
style={this._styleField(FIELD_EMAIL)}
|
||||||
onBlur={function() {self.validateField(FIELD_EMAIL)}} />
|
onBlur={function() {self.validateField(FIELD_EMAIL)}} />
|
||||||
|
@ -211,24 +211,17 @@ module.exports = React.createClass({
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
var placeholderUserName = "User name";
|
|
||||||
if (this.props.defaultUsername) {
|
|
||||||
placeholderUserName += " (default: " + this.props.defaultUsername + ")"
|
|
||||||
}
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div>
|
<div>
|
||||||
<form onSubmit={this.onSubmit}>
|
<form onSubmit={this.onSubmit}>
|
||||||
{emailSection}
|
{emailSection}
|
||||||
<br />
|
<br />
|
||||||
<input className="mx_Login_field" type="text" ref="username"
|
<input className="mx_Login_field" type="text" ref="username"
|
||||||
placeholder={ placeholderUserName } defaultValue={this.state.username}
|
placeholder="User name" defaultValue={this.state.username}
|
||||||
style={this._styleField(FIELD_USERNAME)}
|
style={this._styleField(FIELD_USERNAME)}
|
||||||
onBlur={function() {self.validateField(FIELD_USERNAME)}} />
|
onBlur={function() {self.validateField(FIELD_USERNAME)}}
|
||||||
|
disabled={this.props.disableUsernameChanges} />
|
||||||
<br />
|
<br />
|
||||||
{ this.props.defaultUsername ?
|
|
||||||
<div className="mx_Login_fieldLabel">Setting a user name will create a fresh account</div> : null
|
|
||||||
}
|
|
||||||
<input className="mx_Login_field" type="password" ref="password"
|
<input className="mx_Login_field" type="password" ref="password"
|
||||||
style={this._styleField(FIELD_PASSWORD)}
|
style={this._styleField(FIELD_PASSWORD)}
|
||||||
onBlur={function() {self.validateField(FIELD_PASSWORD)}}
|
onBlur={function() {self.validateField(FIELD_PASSWORD)}}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue