diff --git a/validation/src/check_mapfix.rs b/validation/src/check_mapfix.rs index f5aec5b..de95811 100644 --- a/validation/src/check_mapfix.rs +++ b/validation/src/check_mapfix.rs @@ -21,37 +21,30 @@ impl crate::message_handler::MessageHandler{ // update the mapfix depending on the result match check_result{ - Ok(CheckReportAndVersion{status,version})=>{ - match status{ - // update the mapfix model status to submitted - Ok(map_info)=>self.api.action_mapfix_submitted( - submissions_api::types::ActionMapfixSubmittedRequest{ - MapfixID:mapfix_id, - ModelVersion:version, - DisplayName:map_info.display_name, - Creator:map_info.creator, - GameID:map_info.game_id as u32, - } - ).await.map_err(Error::ApiActionMapfixCheck)?, - // update the mapfix model status to request changes - Err(report)=>self.api.action_mapfix_request_changes( - submissions_api::types::ActionMapfixRequestChangesRequest{ - MapfixID:mapfix_id, - ErrorMessage:report, - } - ).await.map_err(Error::ApiActionMapfixCheck)?, + Ok(CheckReportAndVersion{status:Ok(map_info),version})=>self.api.action_mapfix_submitted( + submissions_api::types::ActionMapfixSubmittedRequest{ + MapfixID:mapfix_id, + ModelVersion:version, + DisplayName:map_info.display_name, + Creator:map_info.creator, + GameID:map_info.game_id as u32, } - }, - Err(e)=>{ - // TODO: report the error - // update the mapfix model status to request changes - self.api.action_mapfix_request_changes( - submissions_api::types::ActionMapfixRequestChangesRequest{ - MapfixID:mapfix_id, - ErrorMessage:e.to_string(), - } - ).await.map_err(Error::ApiActionMapfixCheck)?; - }, + ).await.map_err(Error::ApiActionMapfixCheck)?, + // update the mapfix model status to request changes + Ok(CheckReportAndVersion{status:Err(report),..})=>self.api.action_mapfix_request_changes( + submissions_api::types::ActionMapfixRequestChangesRequest{ + MapfixID:mapfix_id, + ErrorMessage:report, + } + ).await.map_err(Error::ApiActionMapfixCheck)?, + // TODO: report the error + // update the mapfix model status to request changes + Err(e)=>self.api.action_mapfix_request_changes( + submissions_api::types::ActionMapfixRequestChangesRequest{ + MapfixID:mapfix_id, + ErrorMessage:e.to_string(), + } + ).await.map_err(Error::ApiActionMapfixCheck)?, } Ok(()) diff --git a/validation/src/check_submission.rs b/validation/src/check_submission.rs index 1440a10..4140539 100644 --- a/validation/src/check_submission.rs +++ b/validation/src/check_submission.rs @@ -21,37 +21,31 @@ impl crate::message_handler::MessageHandler{ // update the submission depending on the result match check_result{ - Ok(CheckReportAndVersion{status,version})=>{ - match status{ - // update the submission model status to submitted - Ok(map_info)=>self.api.action_submission_submitted( - submissions_api::types::ActionSubmissionSubmittedRequest{ - SubmissionID:submission_id, - ModelVersion:version, - DisplayName:map_info.display_name, - Creator:map_info.creator, - GameID:map_info.game_id as u32, - } - ).await.map_err(Error::ApiActionSubmissionCheck)?, - // update the submission model status to request changes - Err(report)=>self.api.action_submission_request_changes( - submissions_api::types::ActionSubmissionRequestChangesRequest{ - SubmissionID:submission_id, - ErrorMessage:report, - } - ).await.map_err(Error::ApiActionSubmissionCheck)?, + // update the submission model status to submitted + Ok(CheckReportAndVersion{status:Ok(map_info),version})=>self.api.action_submission_submitted( + submissions_api::types::ActionSubmissionSubmittedRequest{ + SubmissionID:submission_id, + ModelVersion:version, + DisplayName:map_info.display_name, + Creator:map_info.creator, + GameID:map_info.game_id as u32, } - }, - Err(e)=>{ - // TODO: report the error - // update the submission model status to request changes - self.api.action_submission_request_changes( - submissions_api::types::ActionSubmissionRequestChangesRequest{ - SubmissionID:submission_id, - ErrorMessage:e.to_string(), - } - ).await.map_err(Error::ApiActionSubmissionCheck)?; - }, + ).await.map_err(Error::ApiActionSubmissionCheck)?, + // update the submission model status to request changes + Ok(CheckReportAndVersion{status:Err(report),..})=>self.api.action_submission_request_changes( + submissions_api::types::ActionSubmissionRequestChangesRequest{ + SubmissionID:submission_id, + ErrorMessage:report, + } + ).await.map_err(Error::ApiActionSubmissionCheck)?, + // TODO: report the error + // update the submission model status to request changes + Err(e)=>self.api.action_submission_request_changes( + submissions_api::types::ActionSubmissionRequestChangesRequest{ + SubmissionID:submission_id, + ErrorMessage:e.to_string(), + } + ).await.map_err(Error::ApiActionSubmissionCheck)?, } Ok(())