Cache time registration on 'set'

Author: mawkler

src/infrastructure/repositories/maconomy_http_client.rs

@@ -195,7 +195,7 @@ impl MaconomyHttpClient<'_> {
         days: &HashSet<u8>,
         row: u8,
         container_instance: &ContainerInstance,
-    ) -> Result<ConcurrencyControl> {
+    ) -> Result<(TimeRegistration, ConcurrencyControl)> {
         let concurrency_control = &container_instance.concurrency_control.0;
         let instance_url = self.get_container_instance_url(&container_instance.id.0);
         let url = format!("{instance_url}/data/panes/table/{row}");
@@ -218,7 +218,12 @@ impl MaconomyHttpClient<'_> {
             bail!("Server responded with {status}");
         }
 
-        Ok(concurrency_control.into())
+        let time_registration = response
+            .json()
+            .await
+            .context("Failed to parse response to time registration")?;
+
+        Ok((time_registration, concurrency_control.into()))
     }
 
     pub async fn get_job_number_from_name(&self, job_name: &str) -> Result<Option<String>> {

src/infrastructure/repositories/time_sheet_repository.rs

@@ -158,13 +158,13 @@ impl TimeSheetRepository<'_> {
         let days: HashSet<_> = days.iter().map(|&d| d as u8).collect();
 
         info!("Setting time");
-        let concurrency_control = self
+        let (time_registration, concurrency_control) = self
             .client
             .set_time(hours, &days, line_number, &container_instance)
             .await
             .with_context(|| format!("Failed to set {hours} hours on row {line_number}"))?;
 
-        // TODO: also update self.time_registration
+        self.time_registration = Some(time_registration);
         self.update_concurrency_control(concurrency_control);
         Ok(())
     }