[Bug]: Reactive property from partial property with default value

[IvanJosipovic]

Describe the bug 🐞

Partial properties with default values appear to be broken. Example Class below is from the readme.

Error: Class1.cs(9,27): error CS8050: Only auto-implemented properties, or properties that use the 'field' keyword, can have initializers.

Step to reproduce

Create project as follows:

<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net9.0</TargetFramework>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
    <LangVersion>preview</LangVersion>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="ReactiveUI" Version="20.2.45" />
    <PackageReference Include="ReactiveUI.SourceGenerators" Version="2.1.27">
      <PrivateAssets>all</PrivateAssets>
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
    </PackageReference>
  </ItemGroup>

</Project>

using ReactiveUI;
using ReactiveUI.SourceGenerators;

namespace ClassLibrary1;

public partial class MyReactiveClass : ReactiveObject
{
    [Reactive]
    public partial string MyProperty { get; set; } = "Default Value";
}

Expected behavior

This should work as per readme

IDE

Visual Studio 2022

Operating system

Windows

Version

11

ReactiveUI Version

20.2.45

[ChrisPulman]

The README is relative to the current source code, there is a pending release that should resolve a few scenarios for reactive partial properties including initial values.
Thank you.

[ChrisPulman]

Version 2.2.4 should now resolve this issue.
Please let us know if there is any further issues experienced.
Thank you.

[IvanJosipovic]

@ChrisPulman, the issue exists on 2.2.4

[ChrisPulman]

Please check that you have set
<LangVersion>preview</LangVersion>
This is required for partial properties with initial values, I am using this in my current project, and I am not experiencing this issue.

[IvanJosipovic]

Please check that you have set <LangVersion>preview</LangVersion> This is required for partial properties with initial values, I am using this in my current project, and I am not experiencing this issue.

That was it, forgot that in my retest. Thank you!

[RhodeHummel]

For anyone that comes across this issue like I did you need to make sure you also are using the correct version of ReactiveUI.SourceGenerators. I was running 20.2.45. It worked after upgrading to 20.3.1. The properly generated code will make using of the field keyword instead of a backing field.

It will look like this:

[global::System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
public partial bool IsChecked { get => field; set => this.RaiseAndSetIfChanged(ref field, value); }

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.