hatter/dingtalk
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

update follow clippy

Browse Source
This commit is contained in:
Hatter Jiang
2020-01-08 03:51:17 +08:00
parent 1316f9cc7b
commit a0a95e0f8f
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/lib.rs
View File

@@ -183,7 +183,7 @@ impl <'a> DingTalkMessage<'a> {
/// New DingTalk message /// New DingTalk message
pub fn new(message_type: DingTalkMessageType) -> Self { pub fn new(message_type: DingTalkMessageType) -> Self {
DingTalkMessage { DingTalkMessage {
message_type: message_type, message_type,
..Default::default() ..Default::default()
} }
} }
@@ -268,7 +268,7 @@ impl <'a> DingTalkMessage<'a> {
} }
/// At mobiles /// At mobiles
pub fn at_mobiles(mut self, mobiles: &Vec<String>) -> Self { pub fn at_mobiles(mut self, mobiles: &[String]) -> Self {
for m in mobiles { for m in mobiles {
self.at_mobiles.push(m.clone()); self.at_mobiles.push(m.clone());
} }
@@ -313,9 +313,9 @@ impl <'a> DingTalk<'a> {
let sec_token = Self::string_to_a_str(json_value["sec_token"].as_str().unwrap_or_default()); let sec_token = Self::string_to_a_str(json_value["sec_token"].as_str().unwrap_or_default());
Ok(DingTalk { Ok(DingTalk {
default_webhook_url: default_webhook_url, default_webhook_url,
access_token: access_token, access_token,
sec_token: sec_token, sec_token,
..Default::default() ..Default::default()
}) })
} }
@@ -333,8 +333,8 @@ impl <'a> DingTalk<'a> {
pub fn new(access_token: &'a str, sec_token: &'a str) -> Self { pub fn new(access_token: &'a str, sec_token: &'a str) -> Self {
DingTalk { DingTalk {
default_webhook_url: DEFAULT_DINGTALK_ROBOT_URL, default_webhook_url: DEFAULT_DINGTALK_ROBOT_URL,
access_token: access_token, access_token,
sec_token: sec_token, sec_token,
..Default::default() ..Default::default()
} }
} }
@@ -416,7 +416,7 @@ impl <'a> DingTalk<'a> {
"links" => json::JsonValue::Array(links), "links" => json::JsonValue::Array(links),
}; };
} }
if dingtalk_message.at_all || dingtalk_message.at_mobiles.len() > 0 { if dingtalk_message.at_all || !dingtalk_message.at_mobiles.is_empty() {
let mut at_mobiles = json::JsonValue::new_object(); let mut at_mobiles = json::JsonValue::new_object();
for m in &dingtalk_message.at_mobiles { for m in &dingtalk_message.at_mobiles {
at_mobiles.push(m.clone()).ok(); at_mobiles.push(m.clone()).ok();