Merge pull request #192 from Mind-Sports-Games/pla-1035-issue-loading-go-games-into-analysisstudy-with-passes-in

account for old style go fen format without pass info

Author: JamesHeppell

build.sbt

@@ -2,7 +2,7 @@ name := "strategygames"
 
 organization := "org.playstrategy"
 
-version := "10.2.1-pstrat158"
+version := "10.2.1-pstrat159"
 
 scalaVersion := "2.13.10"
 

src/main/scala/go/format/FEN.scala

@@ -22,10 +22,13 @@ final case class FEN(value: String) extends AnyVal {
 
   def komi: Double = intFromFen(7).getOrElse(0) / 10.0
 
+  // need to account for old style of fen without pass info due to studies and setup info
+  def oldFenSytle: Boolean = value.split(' ').length == 9
+
   // Consecutive Pass count. Capped at 2, 3 is reserved for end game (after "ss:")
-  def fenPassCount: Int = intFromFen(8).getOrElse(0)
+  def fenPassCount: Int = if (oldFenSytle) 0 else intFromFen(8).getOrElse(0)
 
-  def fullMove: Option[Int] = intFromFen(9)
+  def fullMove: Option[Int] = if (oldFenSytle) intFromFen(8) else intFromFen(9)
 
   def ply: Option[Int] =
     fullMove map { fm =>

src/main/scala/go/format/Forsyth.scala

@@ -26,10 +26,10 @@ object Forsyth {
           variant = variant,
           pocketData = apiPosition.pocketData,
           uciMoves = fen.fenPassCount match {
-            case 0 => List()
             case 1 => List("pass")
             case 2 => List("pass", "pass")
             case 3 => List("ss:")
+            case _ => List()
           },
           position = apiPosition.some
         ),