Merge pull request beego#2978 from BorisBorshevsky/fix_reflection_bug

fix bug beego#2972
wo642436249 · Dec 18, 2017 · f166888 · f166888
2 parents bf5c562 + 0e369e6
commit f166888
Showing 1 changed file with 5 additions and 2 deletions.
diff --git 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