Sur Codacy : Prohibit using isDefined on optional types

[jdauphant]

La régle "Prohibit using isDefined on optional types" me semble pas pertinente, un avis ? on la supprime ?

[niladic]

Bah cette règle serait pertinente si Option fold inférait correctement les types en scala, mais c'est pas le cas :(
Typiquement ici ce serait plus clean d'écrire :

    request.session
          .get("userId")
          .orElse(request.queryString.get("token"))
          .orElse(request.queryString.get("key"))
          .fold(
             Ok(views.html.home())
           )(
             _ => TemporaryRedirect(
                s"${routes.ApplicationController.myApplications()}?${request.rawQueryString}"
             ) 

mais l'inférence est tellement mauvaise que le compilateur va t'inférer Ok et te lever une erreur comme quoi TemporaryRedirect n'est pas de type Ok... :(

Du coup, oui, supprime la règle, pas de pertinance due à la lib standard qui est pas top...

[jdauphant]

Ok