Null Reference Exception in ABI.Microsoft.UI.Xaml.Data.ICustomProperty

[JanRajnoha]

Describe the bug
First think, I'm not sure, it should be here, but I don't know about better place.

In my app I get Null Reference exception, when I'm trying to Notify Property Change. Null exception but throw after some time, with no specific time or user flow through app, it is random. Problem is probably in function Do_Abi_GetValue_2, but i'm not sure. Here is stack trace of exception:

Exception has no other useful information (I think), no keys or values related to exception. I tried to find, what was null, but without success.

Steps to reproduce the bug
Steps to reproduce the behavior:
Repo link - This is copy of real repo, but with changed DB info

1. Start app packaged
2. Sign as User
3. Do some sells and payments
4. Tables
5. Click on T1
6. Do sell by click on some product
7. Pay by Pay button
8. Do payment (you could split table, transfer items, ...)
9. After a few steps between tables view, table and payment you will get Null Reference Exception

Expected behavior
No exception, property changed notification should work okay.

Version Info
Windows 10 Insider, tested on multiple versions including 21364.1

NuGet package version:
[WinUI 3 - Project Reunion 0.5: 0.5.6]

Windows 10 version	Saw the problem?
Insider Build (21364.1)	Yes
May 2020 Update (19041)	Not tested
November 2019 Update (18363)	Not tested
May 2019 Update (18362)	Not tested
October 2018 Update (17763)	Not tested

Additional context

[Marv51]

Hi @angelazhangmsft and @Scottj1s

Can you give an update on this issue? For me with WinUI 3 this is a blocker. The app crashes randomly when PropertyChanged events occur. We can not get through a 10 minute demo of our app without this crash occurring.

Any guidance or tips how to find a workaround?

[Marv51]

BTW this is tracked at WinUI as microsoft/microsoft-ui-xaml#4926

[Scottj1s]

Thanks @JanRajnoha and @Marv51 - we can repro this issue and are currently investigating

[Scottj1s]

@JanRajnoha can you provide more repro steps? I tried both slns in your repo, with multiple combinations of build config and startup project. The farthest I got with the CloudCash.Package was a crash at launch:
'Windows.Graphics.Display: GetForCurrentView must be called on a thread that is associated with a CoreWindow.'.

[Scottj1s]

@JanRajnoha - never mind, it looks like @manodasanW has found workable repro steps:

1. Adding the Windows Community toolkit preview feed
2. Changing the configuration to debug
3. Set the main project as packaging project

[JanRajnoha]

Hi @angelazhangmsft, this bug makes development process of our app for client much slower, and it is becoming critical to fulfill release plan. Could you please provide any release date for this bug fix?

[manodasanW]

@JanRajnoha I am actively investigating this issue. Since I didn't have a root cause before the deadline for this milestone / release, it had to be moved to the next. This issue is being a bit tricky to root cause given the number of custom properties being created and I hadn't been able to create a much smaller isolated repro that hits the issue so that I can build with the test projections from our repo. But I have been able to reduce the pages involved in your repo app while still hitting the issue and I am hoping I can use that to root cause it.

[JanRajnoha]

@manodasanW thank you for your answer. It is weird feel, when you just see your bug report moving from milestone to milestone without any proper explaining, what is happening. I believe, it is hard to understand core of the bug and given repo by me is not simple app :) So thank you for your explaining, what happened.