diff options
author | Andy Wortman <ixineeringeverywhere@gmail.com> | 2017-10-21 14:02:37 -0700 |
---|---|---|
committer | Andy Wortman <ixineeringeverywhere@gmail.com> | 2017-10-21 14:02:37 -0700 |
commit | 79f626c9ea8e8503bd17c8ea640d8b192f0bbee5 (patch) | |
tree | 3c7c87923a26a3d3b8482e6c1214c194670cbd3e | |
parent | faa5864206ad8ab315ae12fb925386c6b76b1a90 (diff) |
fix bug with @s not going through in conversations
split included @, meaning no handle started with @, so get_mentions() returned empty vec
-rw-r--r-- | src/commands/twete.rs | 2 | ||||
-rw-r--r-- | src/tw/tweet.rs | 8 |
2 files changed, 6 insertions, 4 deletions
diff --git a/src/commands/twete.rs b/src/commands/twete.rs index a66d5eb..c399df1 100644 --- a/src/commands/twete.rs +++ b/src/commands/twete.rs @@ -95,7 +95,7 @@ fn rep(line: String, tweeter: &mut tw::TwitterCache, queryer: &mut Queryer) { if let Some(twete) = tweeter.retrieve_tweet(&TweetId::Bare(inner_twid)) { // get handles to reply to... let author_handle = tweeter.retrieve_user(&twete.author_id).unwrap().handle.to_owned(); - let mut ats: Vec<String> = twete.get_mentions().into_iter().map(|x| x.to_owned()).collect(); //std::collections::HashSet::new(); + let mut ats: Vec<String> = twete.get_mentions(); //std::collections::HashSet::new(); /* for handle in twete.get_mentions() { ats.insert(handle); diff --git a/src/tw/tweet.rs b/src/tw/tweet.rs index 72802ca..a3fdde3 100644 --- a/src/tw/tweet.rs +++ b/src/tw/tweet.rs @@ -22,17 +22,19 @@ pub struct Tweet { } impl Tweet { - pub fn get_mentions(&self) -> Vec<&str> { + pub fn get_mentions(&self) -> Vec<String> { self.text.split(&[ ',', '.', '/', ';', '\'', '[', ']', '\\', '~', '!', - '@', '#', '$', '%', '^', + '#', '$', '%', '^', '&', '*', '(', ')', '-', '=', '{', '}', '|', ':', '"', '<', '>', '?', '`', ' ' // forgot this initially. awkward. ][..]) - .filter(|x| x.starts_with("@") && x.len() > 1) + .filter(|x| x.starts_with("@") && x.len() > 1 && x.chars().skip(1).all(|c| c != '@')) + // discard @, mentions are just the usernames. + .map(|handle| handle.chars().skip(1).collect()) .collect() } |