Automatic JSON binding to struct in parameters, also changed validation to not store validation state via a cookie, since with a json request/response no redirect is required and therefor no persistent data is required.

notzippy · notzippy · commit 031ea051609b · 2017-05-19T15:52:29.000-07:00
Also renamed local variable error to err so no package conflict error will occur
diff --git a/binder.go b/binder.go
@@ -14,6 +14,7 @@ import (
 	"strconv"
 	"strings"
 	"time"
+	"encoding/json"
 )
 
 // A Binder translates between string parameters and Go data structures.
@@ -290,7 +291,15 @@ func unbindSlice(output map[string]string, name string, val interface{}) {
 }
 
 func bindStruct(params *Params, name string, typ reflect.Type) reflect.Value {
-	result := reflect.New(typ).Elem()
+	resultPointer := reflect.New(typ)
+	result := resultPointer.Elem()
+	if params.JsonRequest {
+		// Try to inject the response as a json into the created result
+		if err := json.Unmarshal(params.Json, resultPointer.Interface()); err != nil {
+			WARN.Println("W: bindStruct: Unable to unmarshal request:", name, err)
+		}
+		return result
+	}
 	fieldValues := make(map[string]reflect.Value)
 	for key := range params.Values {
 		if !strings.HasPrefix(key, name+".") {
diff --git a/binder_test.go b/binder_test.go
@@ -5,6 +5,7 @@
 package revel
 
 import (
+	"encoding/json"
 	"fmt"
 	"io"
 	"io/ioutil"
@@ -165,6 +166,16 @@ var fileBindings = []struct{ val, arrval, f interface{} }{
 	{(*io.ReadSeeker)(nil), []io.ReadSeeker{}, ioutil.ReadAll},
 }
 
+func TestJsonBinder(t *testing.T) {
+	// create a structure to be populated
+	foo := struct {A string}{}
+	d, _ := json.Marshal(map[string]string{"a": "b"})
+	params := &Params{JsonRequest: true, Json: d}
+	ParseParams(params, NewRequest(getMultipartRequest()))
+	actual := Bind(params, "test", reflect.TypeOf(foo))
+	valEq(t, "TestJsonBinder", reflect.ValueOf(actual.Interface().(struct {A string}).A), reflect.ValueOf("b"))
+}
+
 func TestBinder(t *testing.T) {
 	// Reuse the mvc_test.go multipart request to test the binder.
 	params := &Params{}
diff --git a/params.go b/params.go
@@ -9,6 +9,7 @@ import (
 	"net/url"
 	"os"
 	"reflect"
+	"io/ioutil"
 )
 
 // Params provides a unified view of the request params.
@@ -31,6 +32,8 @@ type Params struct {
 
 	Files    map[string][]*multipart.FileHeader // Files uploaded in a multipart form
 	tmpFiles []*os.File                         // Temp files used during the request.
+	Json        []byte                          // JSON data from request body
+	JsonRequest bool                            // True if request was JSON
 }
 
 // ParseParams parses the `http.Request` params into `revel.Controller.Params`
@@ -56,6 +59,20 @@ func ParseParams(params *Params, req *Request) {
 			params.Form = req.MultipartForm.Value
 			params.Files = req.MultipartForm.File
 		}
+	case "application/json":
+		fallthrough
+	case "text/json":
+		if req.Body!=nil {
+			if content, err := ioutil.ReadAll(req.Body);err==nil{
+				// We wont bind it until we determine what we are binding too
+				params.Json = content
+				params.JsonRequest = true
+			} else {
+				ERROR.Println("Failed to ready request body bytes",err)
+			}
+		} else {
+			INFO.Println("Json post received with empty body")
+		}
 	}
 
 	params.Values = params.calcValues()
diff --git a/validation.go b/validation.go
@@ -185,49 +185,56 @@ func (v *Validation) Check(obj interface{}, checks ...Validator) *ValidationResu
 
 // ValidationFilter revel Filter function to be hooked into the filter chain.
 func ValidationFilter(c *Controller, fc []Filter) {
-	errors, err := restoreValidationErrors(c.Request.Request)
-	c.Validation = &Validation{
-		Errors: errors,
-		keep:   false,
-	}
-	hasCookie := (err != http.ErrNoCookie)
+	// If json request, we shall assume json response is intended,
+	// as such no validation cookies should be tied response
+	if c.Params != nil && c.Params.JsonRequest {
+		c.Validation = &Validation{}
+		fc[0](c, fc[1:])
+	} else {
+		errors, err := restoreValidationErrors(c.Request.Request)
+		c.Validation = &Validation{
+			Errors: errors,
+			keep:   false,
+		}
+		hasCookie := (err != http.ErrNoCookie)
 
-	fc[0](c, fc[1:])
+		fc[0](c, fc[1:])
 
-	// Add Validation errors to ViewArgs.
-	c.ViewArgs["errors"] = c.Validation.ErrorMap()
+		// Add Validation errors to ViewArgs.
+		c.ViewArgs["errors"] = c.Validation.ErrorMap()
 
-	// Store the Validation errors
-	var errorsValue string
-	if c.Validation.keep {
-		for _, error := range c.Validation.Errors {
-			if error.Message != "" {
-				errorsValue += "\x00" + error.Key + ":" + error.Message + "\x00"
+		// Store the Validation errors
+		var errorsValue string
+		if c.Validation.keep {
+			for _, err := range c.Validation.Errors {
+				if err.Message != "" {
+					errorsValue += "\x00" + err.Key + ":" + err.Message + "\x00"
+				}
 			}
 		}
-	}
 
-	// When there are errors from Validation and Keep() has been called, store the
-	// values in a cookie. If there previously was a cookie but no errors, remove
-	// the cookie.
-	if errorsValue != "" {
-		c.SetCookie(&http.Cookie{
-			Name:     CookiePrefix + "_ERRORS",
-			Value:    url.QueryEscape(errorsValue),
-			Domain:   CookieDomain,
-			Path:     "/",
-			HttpOnly: true,
-			Secure:   CookieSecure,
-		})
-	} else if hasCookie {
-		c.SetCookie(&http.Cookie{
-			Name:     CookiePrefix + "_ERRORS",
-			MaxAge:   -1,
-			Domain:   CookieDomain,
-			Path:     "/",
-			HttpOnly: true,
-			Secure:   CookieSecure,
-		})
+		// When there are errors from Validation and Keep() has been called, store the
+		// values in a cookie. If there previously was a cookie but no errors, remove
+		// the cookie.
+		if errorsValue != "" {
+			c.SetCookie(&http.Cookie{
+				Name:     CookiePrefix + "_ERRORS",
+				Value:    url.QueryEscape(errorsValue),
+				Domain:   CookieDomain,
+				Path:     "/",
+				HttpOnly: true,
+				Secure:   CookieSecure,
+			})
+		} else if hasCookie {
+			c.SetCookie(&http.Cookie{
+				Name:     CookiePrefix + "_ERRORS",
+				MaxAge:   -1,
+				Domain:   CookieDomain,
+				Path:     "/",
+				HttpOnly: true,
+				Secure:   CookieSecure,
+			})
+		}
 	}
 }
 
