Copied to clipboard

Flag this post as spam?

This post will be reported to the moderators as potential spam to be looked at


  • Martin Pawollek 17 posts 148 karma points
    Jun 29, 2022 @ 11:53
    Martin Pawollek
    0

    SetProperties on OrderLine causing NullException after Vendr.Core Update 2.3.1

    Hi Matt,

    we've upgraded our whole application to Umbraco 10, including our packages. In our testing phase, I noticed an error, when setting properties to an orderline.

    After further investigation I found the source which caused the error: Order.cs -> RaiseOrderLineChangeEvents() -> line 1414 this._originalOrderLineState = null enter image description here

    We did not change our codebase and this can be solved by reverting the update.

    Sadly this is preventing us from migrating to Umbraco 10 now, so I appriciate any suggestions.

    Kind regards, Martin

  • Matt Brailsford 3688 posts 19399 karma points MVP 9x c-trib
    Jun 29, 2022 @ 12:40
    Matt Brailsford
    0

    Hi Martin,

    Can you share the code you are using to set the properties on the order line?

    Many thanks

    Matt

  • Matt Brailsford 3688 posts 19399 karma points MVP 9x c-trib
    Jun 29, 2022 @ 12:43
    Matt Brailsford
    1

    Actually, never mind, I see what the issue is. _originalOrderLineState can be null so we should be doing null propagation.

  • Matt Brailsford 3688 posts 19399 karma points MVP 9x c-trib
    Jun 29, 2022 @ 12:52
    Matt Brailsford
    100

    Ok, so there is a new 2.3.0-beta0002 build on our unstable feed at https://nuget.outfield.digital/unstable/vendr/v3/index.json If you would like to test this and see if it resolves your issue, that would be great.

  • Martin Pawollek 17 posts 148 karma points
    Jun 29, 2022 @ 13:32
    Martin Pawollek
    0

    Hi Matt,

    thank you, I will test it out later.

    Martin

  • Martin Pawollek 17 posts 148 karma points
    Jun 29, 2022 @ 13:59
    Martin Pawollek
    0

    So I didn't found a 2.3.0-beta0002 build, but I found the 2.3.2-beta0002 build, is that what you meant?

    Well I tested it with the 2.3.2-beta0002 build and it was successful :)

  • Matt Brailsford 3688 posts 19399 karma points MVP 9x c-trib
    Jun 29, 2022 @ 14:07
    Matt Brailsford
    0

    Sorry yea, that was the one 🤦‍♂️

Please Sign in or register to post replies

Write your reply to:

Draft