Remove unnecessary test and rewrite match

This commit is contained in:
Miroslav Prasil 2018-05-15 13:57:00 +01:00
parent f43d329e22
commit 3b537f70ac

View File

@ -352,15 +352,12 @@ fn post_cipher_share(uuid: String, data: Json<ShareCipherData>, headers: Headers
let data: ShareCipherData = data.into_inner(); let data: ShareCipherData = data.into_inner();
let mut cipher = match Cipher::find_by_uuid(&uuid, &conn) { let mut cipher = match Cipher::find_by_uuid(&uuid, &conn) {
Some(cipher) => match cipher.uuid == uuid { Some(cipher) => {
true => { if cipher.is_write_accessible_to_user(&headers.user.uuid, &conn) {
if cipher.is_write_accessible_to_user(&headers.user.uuid, &conn) { cipher
cipher } else {
} else { err!("Cipher is not write accessible")
err!("Cipher is not write accessible") }
}
},
false => err!("Wrong Cipher id provided")
}, },
None => err!("Cipher doesn't exist") None => err!("Cipher doesn't exist")
}; };