Fixing non-existent user profile viewing.

- Fixes #381
This commit is contained in:
Dessalines 2019-12-28 20:58:01 -05:00
parent a4b9e635ba
commit d57b2d4865
3 changed files with 8 additions and 4 deletions

1
.travis.yml vendored
View file

@ -17,7 +17,6 @@ before_install:
- cd server - cd server
script: script:
- cargo build - cargo build
- cargo install diesel_cli --no-default-features --features postgres
- diesel migration run - diesel migration run
- cargo test - cargo test
env: env:

View file

@ -367,11 +367,13 @@ impl Perform<GetUserDetailsResponse> for Oper<GetUserDetails> {
let user_details_id = match data.user_id { let user_details_id = match data.user_id {
Some(id) => id, Some(id) => id,
None => { None => {
User_::read_from_name( match User_::read_from_name(
&conn, &conn,
data.username.to_owned().unwrap_or("admin".to_string()), data.username.to_owned().unwrap_or("admin".to_string()),
)? ) {
.id Ok(user) => user.id,
Err(_e) => return Err(APIError::err(&self.op, "couldnt_find_that_username_or_email"))?
}
} }
}; };

View file

@ -777,6 +777,9 @@ export class User extends Component<any, UserState> {
if (msg.error) { if (msg.error) {
alert(i18n.t(msg.error)); alert(i18n.t(msg.error));
this.state.deleteAccountLoading = false; this.state.deleteAccountLoading = false;
if (msg.error == 'couldnt_find_that_username_or_email') {
this.context.router.history.push('/');
}
this.setState(this.state); this.setState(this.state);
return; return;
} else if (op == UserOperation.GetUserDetails) { } else if (op == UserOperation.GetUserDetails) {