IReadOnlyCollection as interface param uses Length instead of Count when source is an array about mapperly

[kockicica]

Just tried and yes, i think that we can finally mark this as resolved for good. 😃 Thanks a lot.

[latonz]

Thank you for opening this issue, looks like a regression. I‘ll look into it.

[TimothyMakkison]

Probably line 208 in EnumerableMappingBuilder, uses the correct base types but uses the original source types count property.

[github-actions]

🎉 This issue has been resolved in version 3.4.0-next.4 🎉

The release is available on:

• GitHub release
• v3.4.0-next.4

Your semantic-release bot 📦🚀

[kockicica]

I dont know should I open new issue or update this, so let's try former: something similar still exists in 3.4.0/next.4.

    private partial IList<Data.PagerContact>? _MapDtoPagerContactsToPagerContacts(Model.PagerContact[]? source);

Actual generated code

    [global::System.CodeDom.Compiler.GeneratedCode("Riok.Mapperly", "3.4.0.0")]
    private partial global::System.Collections.Generic.IList<global::Tekelija.TAPaging.Data.PagerContact>? _MapDtoPagerContactsToPagerContacts(global::Tekelija.Model.Dardo.PagerContact[]? source)
    {
        if (source == null)
            return default;
        var target = new global::System.Collections.Generic.List<global::Tekelija.TAPaging.Data.PagerContact>(source.Count);
        foreach (var item in source)
        {
            target.Add(_MapPagerContactDtoToPagerContact(item));
        }
        return target;
    }

Expected generated code

    [global::System.CodeDom.Compiler.GeneratedCode("Riok.Mapperly", "3.4.0.0")]
    private partial global::System.Collections.Generic.IList<global::Tekelija.TAPaging.Data.PagerContact>? _MapDtoPagerContactsToPagerContacts(global::Tekelija.Model.Dardo.PagerContact[]? source)
    {
        if (source == null)
            return default;
        var target = new global::System.Collections.Generic.List<global::Tekelija.TAPaging.Data.PagerContact>(source.Length);
        foreach (var item in source)
        {
            target.Add(_MapPagerContactDtoToPagerContact(item));
        }
        return target;
    }

[latonz]

@kockicica I cannot reproduce it with the latest next release 😞 Sometimes when upgrading a source generator package the IDE does not correctly regenerate the code with the new source generator. Have you restarted your IDE after upgrading the Mapperly package?

[kockicica]

Oh' I've restarted everything, several times - without effect unfortunately. 😄 Here's is a simple example: https://github.com/kockicica/mapperly-test

[latonz]

Thank you for the repro, now I got it. The issue still exists when the parameter of the user defined mapping method is an array. I onl tested enumerables/arrays as object properties which is fixed. I‘ll look into it in the next days.

[github-actions]

🎉 This issue has been resolved in version 3.4.0-next.5 🎉

The release is available on:

• GitHub release
• v3.4.0-next.5

Your semantic-release bot 📦🚀

[latonz]

This is hopefully really fixed in 3.4.0-next.5. Could you (@kockicica) verify this?

[github-actions]

🎉 This issue has been resolved in version 3.4.0 🎉

The release is available on:

• GitHub release
• v3.4.0

Your semantic-release bot 📦🚀