bug fix for issue #148

Author: Mihail Slavchev

src/jni/FieldAccessor.cpp

@@ -371,7 +371,7 @@ void FieldAccessor::SetJavaField(const Handle<Object>& target, const Handle<Valu
 			}
 			case 'J': //long
 			{
-				jlong longValue = (jlong) ArgConverter::ConvertToJavaLong(value);
+				jlong longValue = static_cast<jlong>(ArgConverter::ConvertToJavaLong(value));
 				if (isStatic)
 				{
 					env.SetStaticLongField(clazz, fieldId, longValue);
@@ -386,23 +386,23 @@ void FieldAccessor::SetJavaField(const Handle<Object>& target, const Handle<Valu
 			{
 				if (isStatic)
 				{
-					env.SetStaticFloatField(clazz, fieldId, (jfloat) value->NumberValue());
+					env.SetStaticFloatField(clazz, fieldId, static_cast<jfloat>(value->NumberValue()));
 				}
 				else
 				{
-					env.SetLongField(targetJavaObject, fieldId, (jfloat) value->NumberValue());
+					env.SetFloatField(targetJavaObject, fieldId, static_cast<jfloat>(value->NumberValue()));
 				}
 				break;
 			}
 			case 'D': //double
 			{
 				if (isStatic)
 				{
-					env.SetStaticFloatField(clazz, fieldId, value->NumberValue());
+					env.SetStaticDoubleField(clazz, fieldId, value->NumberValue());
 				}
 				else
 				{
-					env.SetLongField(targetJavaObject, fieldId, value->NumberValue());
+					env.SetDoubleField(targetJavaObject, fieldId, value->NumberValue());
 				}
 				break;
 			}

test-app/assets/app/tests/tests.js

@@ -905,6 +905,18 @@ describe("Tests ", function () {
 		expect(name).toEqual("dummy");
 	});
 
+	it("TestSetFloatInstanceField", function () {
+		
+		__log("TEST: TestSetFloatInstanceField");
+		
+		var lParams = new android.widget.LinearLayout.LayoutParams(android.widget.LinearLayout.LayoutParams.MATCH_PARENT, android.widget.LinearLayout.LayoutParams.MATCH_PARENT);
+		lParams.weight = 1;
+		
+		var value = lParams.weight;
+		
+		expect(value).toEqual(1);
+	});
+	
 	it("TestCallMethodOnAnObjectPassedAsParameterInOverriddenMethodAsAnObjectWithoutMetadata", function () {
 
 		__log("TEST: TestCallMethodOnAnObjectPassedAsParameterInOverriddenMethodAsAnObjectWithoutMetadata");