Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 5f07257

Browse files
committedJan 16, 2025·
Fix issues from rebase
1 parent 5f9aa6d commit 5f07257

File tree

2 files changed

+15
-23
lines changed

2 files changed

+15
-23
lines changed
 

‎src/http/client.rs

+13-13
Original file line numberDiff line numberDiff line change
@@ -837,7 +837,7 @@ impl Http {
837837
map.insert("guild_id".to_string(), guild_id.get().into());
838838
}
839839

840-
from_value(value)
840+
from_value(value).map_err(From::from)
841841
}
842842

843843
/// Creates a Guild Scheduled Event.
@@ -1777,7 +1777,7 @@ impl Http {
17771777
map.insert("guild_id".to_string(), guild_id.get().into());
17781778
}
17791779

1780-
from_value::<Member>(value)
1780+
from_value::<Member>(value).map_err(From::from)
17811781
}
17821782

17831783
/// Edits a message by Id.
@@ -1990,7 +1990,7 @@ impl Http {
19901990
map.insert("guild_id".to_string(), guild_id.get().into());
19911991
}
19921992

1993-
from_value(value)
1993+
from_value(value).map_err(From::from)
19941994
}
19951995

19961996
/// Changes the positions of roles in a guild.
@@ -2023,7 +2023,7 @@ impl Http {
20232023
}
20242024
}
20252025

2026-
from_value(value)
2026+
from_value(value).map_err(From::from)
20272027
}
20282028

20292029
/// Modifies a scheduled event.
@@ -2083,7 +2083,7 @@ impl Http {
20832083
map.insert("guild_id".to_string(), guild_id.get().into());
20842084
}
20852085

2086-
from_value(value)
2086+
from_value(value).map_err(From::from)
20872087
}
20882088

20892089
/// Edits a thread channel in the [`GuildChannel`] given its Id.
@@ -3427,7 +3427,7 @@ impl Http {
34273427
}
34283428
}
34293429

3430-
from_value(value)
3430+
from_value(value).map_err(From::from)
34313431
}
34323432

34333433
/// Gets the amount of users that can be pruned.
@@ -3482,7 +3482,7 @@ impl Http {
34823482
map.insert("guild_id".to_string(), guild_id.get().into());
34833483
}
34843484

3485-
from_value(value)
3485+
from_value(value).map_err(From::from)
34863486
}
34873487

34883488
/// Retrieves a list of roles in a [`Guild`].
@@ -3508,7 +3508,7 @@ impl Http {
35083508
}
35093509
}
35103510

3511-
from_value(value)
3511+
from_value(value).map_err(From::from)
35123512
}
35133513

35143514
/// Gets a scheduled event by Id.
@@ -3620,7 +3620,7 @@ impl Http {
36203620
}
36213621
}
36223622

3623-
from_value(value)
3623+
from_value(value).map_err(From::from)
36243624
}
36253625

36263626
/// Retrieves a single sticker in a [`Guild`].
@@ -3647,7 +3647,7 @@ impl Http {
36473647
map.insert("guild_id".to_string(), guild_id.get().into());
36483648
}
36493649

3650-
from_value(value)
3650+
from_value(value).map_err(From::from)
36513651
}
36523652

36533653
/// Retrieves the webhooks for the given [`Guild`]'s Id.
@@ -3721,7 +3721,7 @@ impl Http {
37213721
map.insert("guild_id".to_string(), guild_id.get().into());
37223722
}
37233723

3724-
from_value(value)
3724+
from_value(value).map_err(From::from)
37253725
}
37263726

37273727
/// Gets information about a specific invite.
@@ -3782,7 +3782,7 @@ impl Http {
37823782
map.insert("guild_id".to_string(), guild_id.get().into());
37833783
}
37843784

3785-
from_value(value)
3785+
from_value(value).map_err(From::from)
37863786
}
37873787

37883788
/// Gets a message by an Id, bots only.
@@ -4288,7 +4288,7 @@ impl Http {
42884288
}
42894289
}
42904290

4291-
from_value(value)
4291+
from_value(value).map_err(From::from)
42924292
}
42934293

42944294
/// Starts removing some members from a guild based on the last time they've been online.

‎src/utils/content_safe.rs

+2-10
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ pub struct ContentSafeOptions {
3434
///
3535
/// This option is ignored if the username is a next-gen username, and
3636
/// therefore does not have a discriminator.
37-
#[deprecated = "Discriminators are deprecated on the discord side, and this doesn't reflect message rendering behaviour"]
37+
#[cfg_attr(not(ignore_serenity_deprecated), deprecated = "Discriminators are deprecated on the discord side, and this doesn't reflect message rendering behaviour")]
3838
pub show_discriminator: bool,
3939
}
4040

@@ -177,7 +177,7 @@ fn clean_mention(
177177
Mention::User(id) => {
178178
if let Some(member) = guild.members.get(&id) {
179179
if options.get_show_discriminator() {
180-
#[allow(deprecated)]
180+
#[expect(deprecated)]
181181
let name = member.distinct();
182182
format!("@{name}").into()
183183
} else {
@@ -276,14 +276,6 @@ mod tests {
276276
content_safe(&no_member_guild, "<@!100000000000000000>", options, &[])
277277
);
278278

279-
#[allow(deprecated)]
280-
let options = options.show_discriminator(false);
281-
assert_eq!(
282-
format!("@{}", user.name),
283-
content_safe(&no_member_guild, "<@100000000000000000>", &options, &[])
284-
);
285-
286-
let options = options.display_as_member_from(guild.id);
287279
assert_eq!(
288280
format!("@{}", member.nick.as_ref().unwrap()),
289281
content_safe(&guild, "<@100000000000000000>", options, &[])

0 commit comments

Comments
 (0)
Please sign in to comment.