Skip to content
This repository has been archived by the owner on Jun 16, 2024. It is now read-only.

Alias and name checks on aggregates do not raise exception #485

Open
filcole opened this issue Mar 6, 2020 · 0 comments · May be fixed by #486
Open

Alias and name checks on aggregates do not raise exception #485

filcole opened this issue Mar 6, 2020 · 0 comments · May be fixed by #486

Comments

@filcole
Copy link

filcole commented Mar 6, 2020

The checking in XrmFakedContext.Aggregations.cs against IsNullOrEmpty is for a literal string, rather than the variable, which causes exception to never fire, e.g.

                if (string.IsNullOrEmpty("alias"))
                {
                    throw new Exception("Missing alias for attribute in aggregate fetch xml");
                }
                if (string.IsNullOrEmpty("name"))
                {
                    throw new Exception("Missing name for attribute in aggregate fetch xml");
                }
@filcole filcole changed the title Alias and name checks on aggregates do not fire Alias and name checks on aggregates do not raise exception Mar 6, 2020
@filcole filcole linked a pull request Mar 6, 2020 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant