mirror of
https://gitlab.computer.surgery/matrix/grapevine.git
synced 2025-12-17 07:41:23 +01:00
enable assigning_clones lint
This commit is contained in:
parent
793d809ac6
commit
c9859a9b2d
6 changed files with 14 additions and 14 deletions
|
|
@ -72,7 +72,6 @@ wildcard_dependencies = "warn"
|
||||||
missing_errors_doc = "allow"
|
missing_errors_doc = "allow"
|
||||||
missing_panics_doc = "allow"
|
missing_panics_doc = "allow"
|
||||||
|
|
||||||
assigning_clones = "allow"
|
|
||||||
doc_markdown = "allow"
|
doc_markdown = "allow"
|
||||||
manual_is_variant_and = "allow"
|
manual_is_variant_and = "allow"
|
||||||
mixed_attributes_style = "allow"
|
mixed_attributes_style = "allow"
|
||||||
|
|
|
||||||
|
|
@ -60,7 +60,7 @@ pub(crate) async fn update_device_route(
|
||||||
.get_device_metadata(sender_user, &body.device_id)?
|
.get_device_metadata(sender_user, &body.device_id)?
|
||||||
.ok_or(Error::BadRequest(ErrorKind::NotFound, "Device not found."))?;
|
.ok_or(Error::BadRequest(ErrorKind::NotFound, "Device not found."))?;
|
||||||
|
|
||||||
device.display_name = body.display_name.clone();
|
device.display_name.clone_from(&body.display_name);
|
||||||
|
|
||||||
services().users.update_device_metadata(
|
services().users.update_device_metadata(
|
||||||
sender_user,
|
sender_user,
|
||||||
|
|
|
||||||
|
|
@ -221,7 +221,7 @@ pub(crate) async fn kick_user_route(
|
||||||
.map_err(|_| Error::bad_database("Invalid member event in database."))?;
|
.map_err(|_| Error::bad_database("Invalid member event in database."))?;
|
||||||
|
|
||||||
event.membership = MembershipState::Leave;
|
event.membership = MembershipState::Leave;
|
||||||
event.reason = body.reason.clone();
|
event.reason.clone_from(&body.reason);
|
||||||
|
|
||||||
let mutex_state = Arc::clone(
|
let mutex_state = Arc::clone(
|
||||||
services()
|
services()
|
||||||
|
|
@ -360,7 +360,7 @@ pub(crate) async fn unban_user_route(
|
||||||
.map_err(|_| Error::bad_database("Invalid member event in database."))?;
|
.map_err(|_| Error::bad_database("Invalid member event in database."))?;
|
||||||
|
|
||||||
event.membership = MembershipState::Leave;
|
event.membership = MembershipState::Leave;
|
||||||
event.reason = body.reason.clone();
|
event.reason.clone_from(&body.reason);
|
||||||
|
|
||||||
let mutex_state = Arc::clone(
|
let mutex_state = Arc::clone(
|
||||||
services()
|
services()
|
||||||
|
|
|
||||||
|
|
@ -997,8 +997,8 @@ impl KeyValueDatabase {
|
||||||
.get(content_rule_transformation[0]);
|
.get(content_rule_transformation[0]);
|
||||||
if rule.is_some() {
|
if rule.is_some() {
|
||||||
let mut rule = rule.unwrap().clone();
|
let mut rule = rule.unwrap().clone();
|
||||||
rule.rule_id =
|
content_rule_transformation[1]
|
||||||
content_rule_transformation[1].to_owned();
|
.clone_into(&mut rule.rule_id);
|
||||||
rules_list
|
rules_list
|
||||||
.content
|
.content
|
||||||
.shift_remove(content_rule_transformation[0]);
|
.shift_remove(content_rule_transformation[0]);
|
||||||
|
|
@ -1027,7 +1027,7 @@ impl KeyValueDatabase {
|
||||||
rules_list.underride.get(transformation[0]);
|
rules_list.underride.get(transformation[0]);
|
||||||
if let Some(rule) = rule {
|
if let Some(rule) = rule {
|
||||||
let mut rule = rule.clone();
|
let mut rule = rule.clone();
|
||||||
rule.rule_id = transformation[1].to_owned();
|
transformation[1].clone_into(&mut rule.rule_id);
|
||||||
rules_list
|
rules_list
|
||||||
.underride
|
.underride
|
||||||
.shift_remove(transformation[0]);
|
.shift_remove(transformation[0]);
|
||||||
|
|
|
||||||
|
|
@ -256,11 +256,11 @@ impl Service {
|
||||||
pusher.ids.pushkey.clone(),
|
pusher.ids.pushkey.clone(),
|
||||||
);
|
);
|
||||||
device.data.default_payload = http.default_payload.clone();
|
device.data.default_payload = http.default_payload.clone();
|
||||||
device.data.format = http.format.clone();
|
device.data.format.clone_from(&http.format);
|
||||||
|
|
||||||
// Tweaks are only added if the format is NOT event_id_only
|
// Tweaks are only added if the format is NOT event_id_only
|
||||||
if !event_id_only {
|
if !event_id_only {
|
||||||
device.tweaks = tweaks.clone();
|
device.tweaks.clone_from(&tweaks);
|
||||||
}
|
}
|
||||||
|
|
||||||
let d = vec![device];
|
let d = vec![device];
|
||||||
|
|
|
||||||
|
|
@ -87,11 +87,12 @@ impl Service {
|
||||||
for (list_id, list) in &mut request.lists {
|
for (list_id, list) in &mut request.lists {
|
||||||
if let Some(cached_list) = cached.lists.get(list_id) {
|
if let Some(cached_list) = cached.lists.get(list_id) {
|
||||||
if list.sort.is_empty() {
|
if list.sort.is_empty() {
|
||||||
list.sort = cached_list.sort.clone();
|
list.sort.clone_from(&cached_list.sort);
|
||||||
};
|
};
|
||||||
if list.room_details.required_state.is_empty() {
|
if list.room_details.required_state.is_empty() {
|
||||||
list.room_details.required_state =
|
list.room_details
|
||||||
cached_list.room_details.required_state.clone();
|
.required_state
|
||||||
|
.clone_from(&cached_list.room_details.required_state);
|
||||||
};
|
};
|
||||||
list.room_details.timeline_limit = list
|
list.room_details.timeline_limit = list
|
||||||
.room_details
|
.room_details
|
||||||
|
|
@ -140,8 +141,8 @@ impl Service {
|
||||||
(..) => {}
|
(..) => {}
|
||||||
}
|
}
|
||||||
if list.bump_event_types.is_empty() {
|
if list.bump_event_types.is_empty() {
|
||||||
list.bump_event_types =
|
list.bump_event_types
|
||||||
cached_list.bump_event_types.clone();
|
.clone_from(&cached_list.bump_event_types);
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
cached.lists.insert(list_id.clone(), list.clone());
|
cached.lists.insert(list_id.clone(), list.clone());
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue