validator: flatten check matches

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(())

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(())