diff --git a/src/allris/oparl.rs b/src/allris/oparl.rs index 004b411..ca70f77 100644 --- a/src/allris/oparl.rs +++ b/src/allris/oparl.rs @@ -79,7 +79,7 @@ fn endpoint_url( since: DateTime, until: Option>, ) -> Url { - let mut url = url.url.join("oparl/papers").unwrap(); + let mut url = url.url.join("oparl/bodies/1/papers").unwrap(); { let mut query_pairs = url.query_pairs_mut(); diff --git a/src/database.rs b/src/database.rs index a99d309..b7fbc23 100644 --- a/src/database.rs +++ b/src/database.rs @@ -155,6 +155,7 @@ impl DatabaseConnection { async fn get_connection(&mut self) -> Result<&mut MultiplexedConnection> { if self.connection.is_some() { + #[allow(clippy::unnecessary_unwrap)] Ok(self.connection.as_mut().unwrap()) } else { let connection = self.client.get_multiplexed_async_connection().await?; @@ -181,7 +182,7 @@ impl DatabaseConnection { RetryMethod::RetryImmediately if self.retry_counter == 1 => return Ok(()), RetryMethod::WaitAndRetry | RetryMethod::RetryImmediately => { // reconnect once in a while if it doesn't work - if self.retry_counter % 3 == 0 { + if self.retry_counter.is_multiple_of(3) { self.connection = None; } } diff --git a/src/lru_cache.rs b/src/lru_cache.rs index 5458cad..c33d761 100644 --- a/src/lru_cache.rs +++ b/src/lru_cache.rs @@ -90,11 +90,11 @@ impl> CacheInner { match self.cache.entry(key) { Entry::Occupied(mut entry) => { let cell = entry.get(); - if let Some(val) = cell.get() { - if !is_valid(val) { - entry.insert(Default::default()); - return entry.get().clone(); - } + if let Some(val) = cell.get() + && !is_valid(val) + { + entry.insert(Default::default()); + return entry.get().clone(); } cell.clone()