From afbe04139561761295b0d037436ae13fe9e9b9f8 Mon Sep 17 00:00:00 2001
From: Thibault Duplessis <thibault.duplessis@gmail.com>
Date: Wed, 5 Jul 2017 10:11:25 +0200
Subject: [PATCH] only show current lag when online

---
 app/controllers/User.scala      | 2 +-
 app/templating/UserHelper.scala | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/controllers/User.scala b/app/controllers/User.scala
index 09957a2a75..2a243bf76a 100644
--- a/app/controllers/User.scala
+++ b/app/controllers/User.scala
@@ -55,7 +55,7 @@ object User extends LilaController {
         crosstable <- ctx.userId ?? { Env.game.crosstableApi(user.id, _) }
         followable <- ctx.isAuth ?? { Env.pref.api.followable(user.id) }
         relation <- ctx.userId ?? { relationApi.fetchRelation(_, user.id) }
-        ping = UserLagCache.getLagRating(user.id)
+        ping = env.isOnline(user.id) ?? UserLagCache.getLagRating(user.id)
         res <- negotiate(
           html = !ctx.is(user) ?? GameRepo.lastPlayedPlaying(user) map { pov =>
           Ok(html.user.mini(user, pov, blocked, followable, relation, ping, crosstable))
diff --git a/app/templating/UserHelper.scala b/app/templating/UserHelper.scala
index 11a875c0a9..58a65b8285 100644
--- a/app/templating/UserHelper.scala
+++ b/app/templating/UserHelper.scala
@@ -274,8 +274,8 @@ trait UserHelper { self: I18nHelper with StringHelper with NumberHelper =>
   val patronIconChar = ""
   val lineIconChar = ""
 
-  val lineIcon: String = s"""<i class="line"></i>"""
-  val patronIcon: String = s"""<i class="line patron" title="lichess Patron"></i>"""
+  val lineIcon: String = """<i class="line"></i>"""
+  val patronIcon: String = """<i class="line patron" title="lichess Patron"></i>"""
   private def lineIcon(patron: Boolean): String = if (patron) patronIcon else lineIcon
   private def lineIcon(user: Option[LightUser]): String = lineIcon(user.??(_.isPatron))
   def lineIcon(user: User): String = lineIcon(user.isPatron)