Revert "make json parsing more robust"

This reverts commit c9443fe.

Author: Pulasthi Bandara

src/main/scala/com/github/vitalsoftware/macros/JsonFormatAnnotation.scala

@@ -48,53 +48,31 @@ class jsonMacro(useDefaults: Boolean) {
       }
     }
 
-    /**
-     * Generates implicit json.Formats that fallback to the defaults is the initial parsing fails.
-     */
     def jsonFormatter(className: TypeName, fields: List[ValDef]) = {
       fields.length match {
         case 0 => c.abort(c.enclosingPosition, "Cannot create json formatter for case class with no fields")
         case _ =>
-          val lowPriorityFormats = if (useDefaults) {
-            q"private val lowPriorityFormats = play.api.libs.json.Json.using[play.api.libs.json.Json.WithDefaultValues].format[$className]"
+          if (useDefaults) {
+            q"implicit val jsonAnnotationFormat = play.api.libs.json.Json.using[play.api.libs.json.Json.WithDefaultValues].format[$className]"
           } else {
-            q"private val lowPriorityFormats = play.api.libs.json.Json.format[$className]"
+            q"implicit val jsonAnnotationFormat = play.api.libs.json.Json.format[$className]"
           }
-
-          Seq(
-            lowPriorityFormats,
-            q"""
-                private val robustReads = new play.api.libs.json.Reads[$className] {
-                  override def reads(json: play.api.libs.json.JsValue) = lowPriorityFormats.reads(json) match {
-                    case s: play.api.libs.json.JsSuccess[_] => s
-                    case e: play.api.libs.json.JsError =>
-                      val transforms = e.errors.map(_._1)
-                        .filter(_.path.size == 1)
-                        .map(_.json.prune)
-                        .reduce(_ andThen _)
-
-                      json.transform(transforms).flatMap(lowPriorityFormats.reads _).orElse(e)
-                  }
-                }
-              """,
-            q"implicit val jsonAnnotationFormat = play.api.libs.json.Format(robustReads, lowPriorityFormats)"
-          )
       }
     }
 
-    def modifiedCompanion(compDeclOpt: Option[ModuleDef], format: Seq[Tree], className: TypeName) = {
+    def modifiedCompanion(compDeclOpt: Option[ModuleDef], format: ValDef, className: TypeName) = {
       compDeclOpt map { compDecl =>
         // Add the formatter to the existing companion object
         val q"object $obj extends ..$bases { ..$body }" = compDecl
         q"""
           object $obj extends ..$bases {
             ..$body
-            ..$format
+            $format
           }
         """
       } getOrElse {
         // Create a companion object with the formatter
-        q"object ${className.toTermName} { ..$format }"
+        q"object ${className.toTermName} { $format }"
       }
     }
 

src/test/scala/com/github/vitalsoftware/macros/JsonFormatAnnotationTest.scala

@@ -3,21 +3,20 @@ package com.github.vitalsoftware.macros
 import org.specs2.mutable.Specification
 import play.api.libs.json._
 
-@json case class Person(name: String, age: Int, gender: Option[String])
-@jsonDefaults case class Person2(name: String, age: Int = 7, gender: Option[String] = None)
+@json case class Person(name: String, age: Int)
+@jsonDefaults case class Person2(name: String, age: Int = 7)
 
 class JsonFormatAnnotationTest extends Specification {
 
   "@json annotation" should {
 
     "create correct formatter for case class with >= 2 fields" in {
 
-      val person = Person("Victor Hugo", 46, Some("Male"))
+      val person = Person("Victor Hugo", 46)
       val json = Json.toJson(person)
       json === Json.obj(
         "name" -> "Victor Hugo",
-        "age" -> 46,
-        "gender" -> "Male"
+        "age" -> 46
       )
       Json.fromJson[Person](json).asOpt must beSome(person)
     }
@@ -36,38 +35,4 @@ class JsonFormatAnnotationTest extends Specification {
       Json.fromJson[Person2](Json.obj("name" -> "Victor Hugo")).asOpt must beSome(person)
     }
   }
-
-  "robustParsing" should {
-    "make invalid option values None" in {
-      val json = Json.obj(
-        "name" -> "Victor Hugo",
-        "age" -> 46,
-        "gender" -> true
-      )
-
-      Json.fromJson[Person](json).asOpt must beSome(Person("Victor Hugo", 46, None))
-      Json.fromJson[Person2](json).asOpt must beSome(Person2("Victor Hugo", 46))
-    }
-
-    "make invalid values with defaults fallback to the default" in {
-      val json = Json.obj(
-        "name" -> "Victor Hugo",
-        "age" -> "non age"
-      )
-
-      Json.fromJson[Person2](json).asOpt must beSome(Person2("Victor Hugo", 7))
-    }
-
-    "throw on invalid values which are not optional or default" in {
-      val json = Json.obj(
-        "name" -> "Victor Hugo",
-        "age" -> "non age"
-      )
-
-      val result = Json.fromJson[Person](json)
-      result must beAnInstanceOf[JsError]
-      result.asInstanceOf[JsError].errors.head._1.path.head.asInstanceOf[KeyPathNode].key mustEqual "age"
-
-    }
-  }
 }