Function GroupBy has some invalid arguments

(0) Share

Report

Posted on by pmwhelan

377

I am trying to display a pie chart by using this blog.

https://blog.atwork.at/post/Tips-for-PowerApps-8-Group-and-chart

I've a table ProjectThamesRequests and it has a lookup column called "Firm" which looks up to Firm.Title

I want to group by firm and display the number of requests per firm.

I try to do this ...

ClearCollect(RequestsByFirm,
AddColumns(
GroupBy(ProjectThamesRequests, "Firm","Rows"),
"RowCount",
CountRows(Rows)
)
);

but get the error "Function GroupBy has some invalid arguments"

Thanks

Categories:

Building Power Apps

I have the same question (0)

All responses (12)

Answers (1)

TheRobRush 11,128 Moderator on at

Like (0)

Report
Copy link

Link copied!

Tested out on my side just to make sure, but the syntax is right. Maybe hover over each part of your groupby until it gives you the little hovericon that tells you this part is wrong like this

Was this reply helpful? Yes No
pmwhelan 377 on at

Like (0)

Report
Copy link

Link copied!

Thanks
It says "Expected a text, numeric, boolean or datetime"
It must not like a Lookup column?

Was this reply helpful? Yes No
TheRobRush 11,128 Moderator on at

Like (0)

Report
Copy link

Link copied!

Lookup column will probably need a modifier on it something like ColumnName.Selected.Title or Concat(ColumnName, Title, ", "). You might try narrowing that down by tossing in a quick glalery, setting it to your datasource, and make a label control in it, and start putting in variations of modifiers for that specific column until you get one that returns a string. I'd set up a test on my side but none of the splists I have setup right now use lookup columns

Was this reply helpful? Yes No
pmwhelan 377 on at

Like (0)

Report
Copy link

Link copied!

Thanks but I don't really follow.

Was this reply helpful? Yes No
TheRobRush 11,128 Moderator on at

Like (0)

Report
Copy link

Link copied!

A lookup column is a table. Kind of like if you wanted to reference a value in the title col,umn of a splist you would have to type SPList.Title, not just splist.

Since a lookup table is a table you have to reference the column you are wanting to look at inside of it the same way.

Was this reply helpful? Yes No
pmwhelan 377 on at

Like (0)

Report
Copy link

Link copied!

I try this ...

ClearCollect(RequestsByFirm,
AddColumns(
//GroupBy(ProjectThamesRequests, "Firm","Rows"),
GroupBy(
AddColumns(
ProjectThamesRequests,
"Firm", 'Firm'.Value
),
"Firm",
"Rows"
)
"RowCount",
CountRows(Rows)
)
);

Was this reply helpful? Yes No
TheRobRush 11,128 Moderator on at

Like (0)

Report
Copy link

Link copied!

Check out this post, and more importantly the solution provided by @RandyHayes you will see he has added the modifier .Value to the end of the lookup field used in the add columns formula

Was this reply helpful? Yes No
pmwhelan 377 on at

Like (0)

Report
Copy link

Link copied!

Yes I have done that.
It's in my code above.
Thanks

Was this reply helpful? Yes No
WarrenBelz 154,795 Most Valuable Professional on at

Like (1)

Report
Copy link

Link copied!
@pmwhelan ,

Just helping @TheRobRush who has done the hard work here - the only issue with your code is that you cannot add a field name that already exists - the RenameColumns part is only necessary if you want Firm as the field name resolved

ClearCollect( RequestsByFirm, RenameColumns( AddColumns( GroupBy( AddColumns( ProjectThamesRequests, "MyFirm", Firm.Value ), "MyFirm", "Rows" ), "RowCount", CountRows(Rows) ), "MyFirm", "Firm" ) );

Was this reply helpful? Yes No
pmwhelan 377 on at

Like (0)

Report
Copy link

Link copied!

Thanks but the function RenameColumns has some invalid arguments.

Was this reply helpful? Yes No