ForAll Loop Issue (Counting each iteration)

(0) Share

Report

Posted on by dyee4614

283

Background:

- Submit data is the collection where I store my data.
- Success Count is where I want to count each record that had an empty error table. However, my code currently records ALL the items in SubmitData for each loop instead of just the count that went through without an error. How can I filter it to only the item that is being evaluated in the ForAll loop?

ForAll(
 SubmitData,
 If(
 IsEmpty(
 Errors('H2A Housing Inspection',
 (Patch('H2A Housing Inspection'
 ,Defaults('H2A Housing Inspection'),
 {Title: "",
 DDate: Now()}
 )
 )
 )
 ), 
 Collect(SuccessCount,SubmitData.Counter)
 )
 );
Notify(
 CountA(SuccessCount.Counter) & " out of " & 
 CountA(SubmitData.region5),NotificationType.Success)

Categories:

Building Power Apps

All responses (3)

Answers (2)

Sort by

Verified answer

mdevaney 29,993 Moderator on at

Like
a
(2)

Report
Copy link

Link copied!
@dyee4614

My approach to the solution is different from yours since it increments a Value stored in a 1 row collection each time there is a success and it does not rely on filters. Please study the changes I made carefully and let me know if you have any questions.

// reset success counter to 0 ClearCollect(colSuccessCount,{Value: 0}) ForAll( SubmitData, If( IsEmpty( Errors('H2A Housing Inspection', ( Patch( 'H2A Housing Inspection', Defaults('H2A Housing Inspection'), {Title: "", DDate: Now()} ) ) ) ), // increment success counter if there are no error Patch(colSuccessCount,{Value: First(colSuccessCount).Value + 1}) ) ); Notify( // display the counter result First(colSuccessCount).Value & " out of " & CountA(SubmitData.region5),NotificationType.Success )

---
Please click "Accept as Solution" if my post answered your question so that others may find it more quickly. If you found this post helpful consider giving it a "Thumbs Up."
Verified answer

v-siky-msft Microsoft Employee on at

Like
a
(1)

Report
Copy link

Link copied!
Hi @dyee4614 ,

Please modify your codes as below:

ForAll( SubmitData, If( IsEmpty( Errors('H2A Housing Inspection', (Patch('H2A Housing Inspection' ,Defaults('H2A Housing Inspection'), {Title: "", DDate: Now()} ) ) ) ), Collect(SuccessCount,Counter) ) ); Notify( CountA(SuccessCount.Value) & " out of " & CountA(SubmitData.region5),NotificationType.Success)

I modify 'SubmitData.Counter' to Counter, because 'SubmitData.Counter' returns a single-line table, so every time it loops, the whole single-line table is stored into the SuccessCount collection.
If you change to use Counter, each time it loops, it will only return the counter value of the currently traversed record without error, and collect to SuccessCount collection.
Meanwhile, you need to modify 'SuccessCount.Counter' to 'SuccessCount.Value', because the column name is changed.
Sik
dyee4614 283 on at

Like
a
(0)

Report
Copy link

Link copied!

It worked! Thanks for your help.