Merge pull request #2978 from BorisBorshevsky/fix_reflection_bug

fix bug #2972

Merge pull request #2978 from BorisBorshevsky/fix_reflection_bug
fix bug #2972
f1668881 · astaxie · GitHub · bf5c5626 · 0e369e6d · f1668881
Unverified Commit f1668881 authored Dec 18, 2017 by astaxie Committed by GitHub Dec 18, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

router.go router.go +5 -2

No files found.
--- a/router.go
+++ b/router.go
@@ -201,9 +201,12 @@ func (p *ControllerRegister) addWithMethodParams(pattern string, c ControllerInt

 		numOfFields := elemVal.NumField()
 		for i := 0; i < numOfFields; i++ {
-			fieldVal := elemVal.Field(i)
 			fieldType := elemType.Field(i)
-			execElem.FieldByName(fieldType.Name).Set(fieldVal)
+
+			if execElem.FieldByName(fieldType.Name).CanSet() {
+				fieldVal := elemVal.Field(i)
+				execElem.FieldByName(fieldType.Name).Set(fieldVal)
+			}
 		}

 		return execController