Browse Source

!44 optimize:field set
Merge pull request !44 from fuhai/master

JFinal 6 years ago
parent
commit
30d1b16c94

+ 10 - 3
src/main/java/com/jfinal/aop/AopFactory.java

@@ -162,9 +162,16 @@ public class AopFactory {
 					fieldInjectedClass = field.getType();
 				}
 				
-				Object fieldInjectedObject = doGet(fieldInjectedClass);
-				field.setAccessible(true);
-				field.set(targetObject, fieldInjectedObject);
+				// Object fieldInjectedObject = doGet(fieldInjectedClass);
+				// field.setAccessible(true);
+				// field.set(targetObject, fieldInjectedObject);
+
+				if (field.isAccessible()) {
+					field.set(targetObject, fieldInjectedObject);
+				} else {
+					field.setAccessible(true);
+					field.set(targetObject, fieldInjectedObject);
+				}
 			}
 		}
 		

+ 2 - 1
src/main/java/com/jfinal/core/Controller.java

@@ -164,7 +164,8 @@ public abstract class Controller {
 	 * @return a String representing the single value of the parameter
 	 */
 	public String getPara(String name) {
-		return request.getParameter(name);
+		//return request.getParameter(name);
+		return getPara(name, null);
 	}
 	
 	/**