refactor: move CheckoutOptions change logic into next/previous methods

Author: Fatpandac

src/popups/checkout_option.rs

@@ -131,25 +131,9 @@ impl CheckoutOptionPopup {
 
 	fn change_kind(&mut self, incr: bool) {
 		self.option = if incr {
-			match self.option {
-				CheckoutOptions::KeepLocalChanges => {
-					CheckoutOptions::Unchange
-				}
-				CheckoutOptions::Unchange => CheckoutOptions::Discard,
-				CheckoutOptions::Discard => {
-					CheckoutOptions::KeepLocalChanges
-				}
-			}
+			self.option.next()
 		} else {
-			match self.option {
-				CheckoutOptions::KeepLocalChanges => {
-					CheckoutOptions::Discard
-				}
-				CheckoutOptions::Unchange => {
-					CheckoutOptions::KeepLocalChanges
-				}
-				CheckoutOptions::Discard => CheckoutOptions::Unchange,
-			}
+			self.option.previous()
 		};
 	}
 }

src/strings.rs

@@ -446,6 +446,28 @@ pub enum CheckoutOptions {
 }
 
 impl CheckoutOptions {
+	pub fn previous(&self) -> Self {
+		match self {
+			CheckoutOptions::KeepLocalChanges => {
+				CheckoutOptions::Discard
+			}
+			CheckoutOptions::Unchange => {
+				CheckoutOptions::KeepLocalChanges
+			}
+			CheckoutOptions::Discard => CheckoutOptions::Unchange,
+		}
+	}
+
+	pub fn next(&self) -> Self {
+		match self {
+			CheckoutOptions::KeepLocalChanges => CheckoutOptions::Unchange,
+			CheckoutOptions::Unchange => CheckoutOptions::Discard,
+			CheckoutOptions::Discard => {
+				CheckoutOptions::KeepLocalChanges
+			}
+		}
+	}
+
 	pub fn to_string_pair(&self) -> (&'static str, &'static str) {
 		const CHECKOUT_OPTION_STASH_AND_REAPPLY: &str =
 			" 🟢 Stash and reapply changes";