From 16135fa549017f8c13050550c4ce4868d1a87247 Mon Sep 17 00:00:00 2001
From: rooklift <16438795+rooklift@users.noreply.github.com>
Date: Thu, 8 Feb 2024 11:21:16 +0000
Subject: [PATCH] Maybe this is best order

---
 files/src/renderer/40_position.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/files/src/renderer/40_position.js b/files/src/renderer/40_position.js
index c78adfad..26eaa044 100644
--- a/files/src/renderer/40_position.js
+++ b/files/src/renderer/40_position.js
@@ -747,6 +747,12 @@ const position_prototype = {
 
 	parse_pgn: function(s) {		// Returns a UCI move and an error message.
 
+		// Replace fruity dash characters with proper ASCII dash "-"
+
+		for (let n of [8208, 8210, 8211, 8212, 8213, 8722]) {
+			s = ReplaceAll(s, String.fromCodePoint(n), "-");
+		}
+
 		// If the string contains any dots it'll be something like "1.e4" or "...e4" or whatnot...
 
 		let lio = s.lastIndexOf(".");
@@ -754,7 +760,7 @@ const position_prototype = {
 			s = s.slice(lio + 1);
 		}
 
-		// At this point, if s was actually a UCI string (which it would never be in real PGN) we can return it.
+		// At this point, if s is actually a UCI string (which it won't be in real PGN) we can return it.
 		// This is a hack to allow pasting of stuff from non-PGN sources I guess...
 
 		if (s.length === 4 || (s.length === 5 && ["q", "r", "b", "n"].includes(s[4]))) {
@@ -770,12 +776,6 @@ const position_prototype = {
 			}
 		}
 
-		// Replace fruity dash characters with proper ASCII dash "-"
-
-		for (let n of [8208, 8210, 8211, 8212, 8213, 8722]) {
-			s = ReplaceAll(s, String.fromCodePoint(n), "-");
-		}
-
 		// Delete things we don't need...
 
 		s = ReplaceAll(s, "x", "");