Fixed a error, that occurred during setting a value to field of embedded type

Author: Taritsyn

src/MsieJavaScriptEngine/HostItemBase.cs

@@ -87,6 +87,7 @@ protected object InvokeStandardMember(string name, BindingFlags invokeAttr, Bind
 		{
 			BindingFlags processedInvokeAttr = invokeAttr;
 			if ((processedInvokeAttr.HasFlag(BindingFlags.GetProperty)
+				|| processedInvokeAttr.HasFlag(BindingFlags.SetProperty)
 				|| processedInvokeAttr.HasFlag(BindingFlags.PutDispProperty))
 				&& !_properties.Any(p => p.Name == name)
 				&& _fields.Any(p => p.Name == name))
@@ -96,6 +97,11 @@ protected object InvokeStandardMember(string name, BindingFlags invokeAttr, Bind
 					processedInvokeAttr &= ~BindingFlags.GetProperty;
 					processedInvokeAttr |= BindingFlags.GetField;
 				}
+				else if (processedInvokeAttr.HasFlag(BindingFlags.SetProperty))
+				{
+					processedInvokeAttr &= ~BindingFlags.SetProperty;
+					processedInvokeAttr |= BindingFlags.SetField;
+				}
 				else if (processedInvokeAttr.HasFlag(BindingFlags.PutDispProperty))
 				{
 					processedInvokeAttr &= ~BindingFlags.PutDispProperty;

src/MsieJavaScriptEngine/MsieJavaScriptEngine.csproj

@@ -20,7 +20,8 @@
 		<RepositoryUrl>https://github.com/Taritsyn/MsieJavaScriptEngine</RepositoryUrl>
 		<RepositoryType>git</RepositoryType>
 		<PackageTags>JavaScript;ECMAScript;MSIE;IE;Edge;Chakra</PackageTags>
-		<PackageReleaseNotes>Fixed a error, that occurred in the `Classic` mode during calling an embedded delegate, which does not return a result.</PackageReleaseNotes>
+		<PackageReleaseNotes>1. Fixed a error, that occurred in the `Classic` mode during calling an embedded delegate, which does not return a result;
+2. Fixed a error, that occurred during setting a value to field of embedded type.</PackageReleaseNotes>
 		<NeutralLanguage>en-US</NeutralLanguage>
 		<PackageOutputPath>../../nuget</PackageOutputPath>
 		<GeneratePackageOnBuild>true</GeneratePackageOnBuild>

src/MsieJavaScriptEngine/readme.txt

@@ -21,8 +21,10 @@
    =============
    RELEASE NOTES
    =============
-   Fixed a error, that occurred in the `Classic` mode during calling an embedded
-   delegate, which does not return a result.
+   1. Fixed a error, that occurred in the `Classic` mode during calling an embedded
+      delegate, which does not return a result;
+   2. Fixed a error, that occurred during setting a value to field of embedded
+      type.
 
    ============
    PROJECT SITE

test/MsieJavaScriptEngine.Test.Common/Interop/Logging/DefaultLogger.cs

@@ -0,0 +1,7 @@
+namespace MsieJavaScriptEngine.Test.Common.Interop.Logging
+{
+	public class DefaultLogger
+	{
+		public static ILogger Current = new NullLogger();
+	}
+}

test/MsieJavaScriptEngine.Test.Common/Interop/Logging/ILogger.cs

@@ -0,0 +1,17 @@
+namespace MsieJavaScriptEngine.Test.Common.Interop.Logging
+{
+	public interface ILogger
+	{
+		void Error(string category, string message,
+			string filePath = "", int lineNumber = 0, int columnNumber = 0,
+			string sourceFragment = "");
+
+		void Warn(string category, string message,
+			string filePath = "", int lineNumber = 0, int columnNumber = 0,
+			string sourceFragment = "");
+
+		void Debug(string category, string message, string filePath = "");
+
+		void Info(string category, string message, string filePath = "");
+	}
+}

test/MsieJavaScriptEngine.Test.Common/Interop/Logging/NullLogger.cs

@@ -0,0 +1,26 @@
+namespace MsieJavaScriptEngine.Test.Common.Interop.Logging
+{
+	public sealed class NullLogger : ILogger
+	{
+		public void Error(string category, string message,
+			string filePath = "", int lineNumber = 0, int columnNumber = 0,
+			string sourceFragment = "")
+		{ }
+
+		public void Warn(string category, string message,
+			string filePath = "", int lineNumber = 0, int columnNumber = 0,
+			string sourceFragment = "")
+		{ }
+
+		public void Debug(string category, string message, string filePath = "")
+		{ }
+
+		public void Info(string category, string message, string filePath = "")
+		{ }
+
+		public override string ToString()
+		{
+			return "[null logger]";
+		}
+	}
+}

test/MsieJavaScriptEngine.Test.Common/Interop/Logging/ThrowExceptionLogger.cs

@@ -0,0 +1,57 @@
+using System;
+using System.Text;
+
+namespace MsieJavaScriptEngine.Test.Common.Interop.Logging
+{
+	public sealed class ThrowExceptionLogger : ILogger
+	{
+		public void Error(string category, string message, string filePath = "",
+			int lineNumber = 0, int columnNumber = 0, string sourceFragment = "")
+		{
+			var errorBuilder = new StringBuilder();
+			errorBuilder.AppendLine("Category: " + category);
+			errorBuilder.AppendLine("Message: " + message);
+
+			if (!string.IsNullOrWhiteSpace(filePath))
+			{
+				errorBuilder.AppendLine("File: " + filePath);
+			}
+
+			if (lineNumber > 0)
+			{
+				errorBuilder.AppendLine("Line number: " + lineNumber);
+			}
+
+			if (columnNumber > 0)
+			{
+				errorBuilder.AppendLine("Column number: " + columnNumber);
+			}
+
+			if (!string.IsNullOrWhiteSpace(sourceFragment))
+			{
+				errorBuilder.AppendLine("Source fragment:" + Environment.NewLine + sourceFragment);
+			}
+
+			string errorMessage = errorBuilder.ToString();
+			errorBuilder.Clear();
+
+			throw new InvalidOperationException(errorMessage);
+		}
+
+		public void Warn(string category, string message,
+			string filePath = "", int lineNumber = 0, int columnNumber = 0,
+			string sourceFragment = "")
+		{ }
+
+		public void Debug(string category, string message, string filePath = "")
+		{ }
+
+		public void Info(string category, string message, string filePath = "")
+		{ }
+
+		public override string ToString()
+		{
+			return "[throw exception logger]";
+		}
+	}
+}

test/MsieJavaScriptEngine.Test.Common/Interop/SimpleSingleton.cs

@@ -14,6 +14,7 @@
 #if NETCOREAPP1_0
 using MsieJavaScriptEngine.Test.Common.Interop.Drawing;
 #endif
+using MsieJavaScriptEngine.Test.Common.Interop.Logging;
 
 namespace MsieJavaScriptEngine.Test.Common
 {
@@ -934,17 +935,22 @@ public virtual void EmbeddingOfCustomValueTypeWithFieldIsCorrect()
 		public virtual void EmbeddingOfCustomReferenceTypeWithFieldIsCorrect()
 		{
 			// Arrange
-			Type simpleSingletonType = typeof(SimpleSingleton);
+			Type defaultLoggerType = typeof(DefaultLogger);
+			Type throwExceptionLoggerType = typeof(ThrowExceptionLogger);
+			const string updateCode = "DefaultLogger.Current = new ThrowExceptionLogger();";
 
-			const string input = "SimpleSingleton.Instance.ToString()";
-			const string targetOutput = "[simple singleton]";
+			const string input = "DefaultLogger.Current.ToString()";
+			const string targetOutput = "[throw exception logger]";
 
 			// Act
 			string output;
 
 			using (var jsEngine = CreateJsEngine())
 			{
-				jsEngine.EmbedHostType("SimpleSingleton", simpleSingletonType);
+				jsEngine.EmbedHostType("DefaultLogger", defaultLoggerType);
+				jsEngine.EmbedHostType("ThrowExceptionLogger", throwExceptionLoggerType);
+				jsEngine.Execute(updateCode);
+
 				output = jsEngine.Evaluate<string>(input);
 			}