toString really is evil. Fix puzzle best game PGN API

This commit is contained in:
Thibault Duplessis 2017-01-25 09:36:09 +01:00
parent 7a152ebe32
commit ecd33d8874
2 changed files with 9 additions and 5 deletions

View file

@ -1,8 +1,8 @@
package controllers
import play.api.i18n.Messages.Implicits._
import play.api.libs.json._
import play.api.mvc._
import play.api.i18n.Messages.Implicits._
import play.api.Play.current
import lila.api.Context
@ -173,13 +173,17 @@ object Puzzle extends LilaController {
Env.game.recentGoodGameActor ? true mapTo manifest[Option[String]] flatMap {
_ ?? lila.game.GameRepo.gameWithInitialFen flatMap {
case Some((game, initialFen)) =>
Ok(Env.api.pgnDump(game, initialFen.map(_.value)).toString).fuccess
Env.api.pgnDump(game, initialFen.map(_.value)) map { pgn =>
Ok(pgn.render)
}
case _ =>
lila.log("puzzle import").info("No recent good game, serving a random one :-/")
lila.game.GameRepo.findRandomFinished(1000) flatMap {
_ ?? { game =>
lila.game.GameRepo.initialFen(game) map { fen =>
Ok(Env.api.pgnDump(game, fen).toString)
lila.game.GameRepo.initialFen(game) flatMap { fen =>
Env.api.pgnDump(game, fen) map { pgn =>
Ok(pgn.render)
}
}
}
}

@ -1 +1 @@
Subproject commit 260877eced4182c61c6f07f886f5221171fe0fd9
Subproject commit 5091eba4fbfaf7232d216a730bba1063010320b9