Announcement

Collapse
No announcement yet.

[CLOSED] [#3165] [16.3.2] PropertyInfo.SetValue() with null index parameter causes invalid javascript

Collapse
X
  • Filter
  • Time
  • Show
Clear All
new posts

    [CLOSED] [#3165] [16.3.2] PropertyInfo.SetValue() with null index parameter causes invalid javascript

    Hi,

    Sample is here: https://dev.deck.net/9d3b2e781fc64db...ace3262269/top

    This C#:
    propertyInfo2.SetValue(ninjaScript, val, null);
    causes the following JavaScript to be created

    Bridge.Reflection.midel(propertyInfo2.s, ninjaScript).apply(null, null.concat(Bridge.unbox(val)));
    The "null.concat" portion causes an exception at runtime.

    Calling the SetValue overload without the third parameter works correctly, as does providing a variable for the third parameter.

    #2
    Hi SBeanland – Can you confirm if this issue is still occurring with the latest Bridge release? I tried your sample in Deck, but no JavaScript error was thrown?

    Comment


      #3
      Hi geoffrey.mcgill - please try the following modified sample which will output the exception message.

      https://dev.deck.net/1ac81667b3ba5db...0683e94590/top

      Comment


        #4
        Hi geoffrey.mcgill I wanted to see if this was still being looked at or if I need to provide something different in the sample in my previous comment? This still occurs in 16.3.1.

        Comment


          #5
          Hi SBeanland,

          Thanks for the reminder on this thread. We will review your sample again.

          Comment


            #6
            Hi,

            Git issue - https://github.com/bridgedotnet/Bridge/issues/3165
            The PR is created

            Comment


              #7
              Fix for issue #3165 merged into v16.3.2 branch and will be included in the next Bridge.NET release.

              Comment

              Working...
              X