Answered

Need Help with Add Columns Formula

(0) Share

Report

Posted on by Community Power Pla... Microsoft Employee

I need help with a formula. If anyone has a better way to do this then that would be awesome.

ClearCollect(ReviewList, 
AddColumns(Filter(FSDetailSupport_, Review_Status.Value="Sent to Reviewer3" || Review_Status.Value="Sent to Reviewer2" || Review_Status.Value="Sent to Reviewer1"),
"CurrentReviewer",
If(Review_Status.Value="Sent to Reviewer1",LookUp(FSDetailSupport_,ID=ID,Reviewer_1.DisplayName),
If(Review_Status.Value="Sent to Reviewer2",LookUp(FSDetailSupport_,ID=ID,Reviewer_2.DisplayName),
If(Review_Status.Value="Sent to Reviewer3",LookUp(FSDetailSupport_,ID=ID,Reviewer_3.DisplayName))))));

I currently am using the above collection formula to add a column called "Current User" that looks up the name of the user based on the value in the Review_Status Column. For whatever reason it will not pull in the user for column when the review status is "Sent to Reviewer3". I have tried everything. It's not formatting. I don't know how to fix it, but I really need it to do this.

Is there a better formula I could use?

Categories:

Building Power Apps

I have the same question (0)

All responses (5)

Answers (1)

Sort by

Verified answer

v-bofeng-msft Microsoft Employee on at

Like
a
(0)

Report
Copy link

Link copied!
Hi @Becca51178 :
I guess this condition "ID=ID" may cause ambiguity.Please Try:

ClearCollect(ReviewList,
            AddColumns(
                 RenameColumns(Filter(FSDetailSupport_,Review_Status.Value="Sent to Reviewer3" || Review_Status.Value="Sent to Reviewer2" || Review_Status.Value="Sent to Reviewer1"),"ID","IDD"),
                 "CurrentReviewer",
                 If(Review_Status.Value="Sent to Reviewer1",
                    LookUp(FSDetailSupport_,ID=IDD,Reviewer_1.DisplayName),
                    If(Review_Status.Value="Sent to Reviewer2",
                       LookUp(FSDetailSupport_,ID=IDD,Reviewer_2.DisplayName),
                       If(Review_Status.Value="Sent to Reviewer3",
                       LookUp(FSDetailSupport_,ID=IDD,Reviewer_3.DisplayName))))));
or
ClearCollect(ReviewList, AddColumns( RenameColumns(Filter(FSDetailSupport_,Review_Status.Value="Sent to Reviewer3" || Review_Status.Value="Sent to Reviewer2" || Review_Status.Value="Sent to Reviewer1"),"ID","IDD"), "CurrentReviewer", If(Review_Status.Value="Sent to Reviewer1", Reviewer_1.DisplayName, If(Review_Status.Value="Sent to Reviewer2", Reviewer_2.DisplayName, If(Review_Status.Value="Sent to Reviewer3", Reviewer_3.DisplayName)))));
or
ClearCollect(ReviewList, AddColumns( RenameColumns(Filter(FSDetailSupport_,Review_Status.Value="Sent to Reviewer3" || Review_Status.Value="Sent to Reviewer2" || Review_Status.Value="Sent to Reviewer1"),"ID","IDD"), "CurrentReviewer", Switch(Review_Status.Value, "Sent to Reviewer1", Reviewer_1.DisplayName, "Sent to Reviewer2", Reviewer_2.DisplayName, "Sent to Reviewer3", Reviewer_3.DisplayName)));
If you still have any question,please don't hesitate to let me know.
Best Regards,
Bof

Was this reply helpful? Yes No
WarrenBelz 155,927 Most Valuable Professional on at

Like
a
(0)

Report
Copy link

Link copied!
Hi @Becca51178 ,
Just for something different - you might try the new As statement to cure the ambiguity. I have also done a With() and a Switch() to condense the code a bit.
ClearCollect( ReviewList, AddColumns( Filter( FSDetailSupport_, Review_Status.Value="Sent to Reviewer3" || Review_Status.Value="Sent to Reviewer2" || Review_Status.Value="Sent to Reviewer1" ) As vReview, "CurrentReviewer", With( { vName: LookUp( FSDetailSupport_, ID=vReview.ID ) }, Switch( Review_Status.Value, "Sent to Reviewer1", vName.Reviewer_1.DisplayName, "Sent to Reviewer2", vName.Reviewer_2.DisplayName, "Sent to Reviewer3", vName.Reviewer_3.DisplayName ) ) ) )

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

Was this reply helpful? Yes No
Community Power Pla... Microsoft Employee on at

Like
a
(0)

Report
Copy link

Link copied!

I really like this code but its throwing an error on the Switch part

Was this reply helpful? Yes No
Community Power Pla... Microsoft Employee on at

Like
a
(0)

Report
Copy link

Link copied!

Thanks! These both worked great!

Was this reply helpful? Yes No
WarrenBelz 155,927 Most Valuable Professional on at

Like
a
(0)

Report
Copy link

Link copied!

Thanks @Becca51178 ,
You do have the option (up to you entirely) of accepting more than one solution, especially if they are helpful in the future for others.

Was this reply helpful? Yes No