From 00cf06e4f333171d69e6f38cbec6987bdc6c6212 Mon Sep 17 00:00:00 2001 From: Andrey Breslav Date: Thu, 31 May 2012 11:55:00 +0400 Subject: [PATCH] DI: Don't call getters on non-public fields --- .../src/org/jetbrains/jet/di/ConstructorCall.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/injector-generator/src/org/jetbrains/jet/di/ConstructorCall.java b/injector-generator/src/org/jetbrains/jet/di/ConstructorCall.java index 88486c9864d..a424938170b 100644 --- a/injector-generator/src/org/jetbrains/jet/di/ConstructorCall.java +++ b/injector-generator/src/org/jetbrains/jet/di/ConstructorCall.java @@ -42,7 +42,12 @@ class ConstructorCall implements Expression { StringBuilder builder = new StringBuilder("new " + constructor.getDeclaringClass().getSimpleName() + "("); for (Iterator iterator = constructorArguments.iterator(); iterator.hasNext(); ) { Field argument = iterator.next(); - builder.append(argument.getGetterName() + "()"); + if (argument.isPublic()) { + builder.append(argument.getGetterName()).append("()"); + } + else { + builder.append(argument.getName()); + } if (iterator.hasNext()) { builder.append(", "); } -- GitLab