Remove useless variable

During some refactoring this seems to be overlooked.
This variable gets filled but isn't used at all afterwards.

Fixes #4105

src/api/core/ciphers.rs

@@ -849,7 +849,6 @@ async fn put_cipher_share_selected(
     nt: Notify<'_>,
 ) -> EmptyResult {
     let mut data: ShareSelectedCipherData = data.into_inner().data;
-    let mut cipher_ids: Vec<String> = Vec::new();
 
     if data.Ciphers.is_empty() {
         err!("You must select at least one cipher.")
@@ -860,10 +859,9 @@ async fn put_cipher_share_selected(
     }
 
     for cipher in data.Ciphers.iter() {
-        match cipher.Id {
+        if cipher.Id.is_none() {
-            Some(ref id) => cipher_ids.push(id.to_string()),
+            err!("Request missing ids field")
-            None => err!("Request missing ids field"),
+        }
-        };
     }
 
     while let Some(cipher) = data.Ciphers.pop() {