Check if a value in a collection exists in another collection if not patch

[Rosie]

Hello,

I am making an app that is using 2 collections.  The user gets a list of items from a SharePoint list, they will be doing an inspection on a group of items, they select 3 items from the list.  These are added to an collection called cltInspecting.

There are a number of inspections to perform, each inspection type is on a separate screen.  When a contaminate is found the result is written to the collection cltInspectingResult. This is working fine.

When all the inspections are completed, the items that had no contaminant need to be patched to the cltInspectingResult collection.

For each item in cltInspecting I want to check if it exists in cltInspectingResult, if it doesn't exit I want to patch it to cltInspectingResult.

I just can't seem to get it right, I have a few errors but the interesting one being that the type (being a table) is not compatible.  I thought the ForAll would take care of that.

I have read a few posts in the forum about this, but I can't seem to follow them to get it right for my situation.

Patch(
    cltInspectionResults,ForAll(
        IsBlank(
            First(
                Filter(cltInspectionResults,cltInspecting.cltContainerID = ThisRecord.ContainerID)
            )
        ),
        {ContainerID:cltInspecting.cltContainerID}
    )
)

This is my latest iteration. 

The gallery in the app showing the items is using the collection cltInspecting

Item 1 has a contaminant - this has been patched to cltInspectionResults

Item 2 has a contaminant - this has been patched to cltInspectionResults

Item 3 has no contaminants - this needs to be written to cltInspectionResults.
The "Next Button" is where I am trying to identify the records showing in the app (cltInspecting) that are not in cltInspectingResults and then add them.

Many thanks

[WarrenBelz · Accepted solution]

Hi @Rosie ,

Try the below - you can also use Patch instead of Collect if you want.

ForAll(
   cltInspecting As aPatch,
   If(
      IsBlank(
         LookUp(
            cltInspectionResults,
            cltContainerID = aPatch.cltContainerID
         ).cltContainerID
      ),		 
      Collect(
         cltInspectionResults,
         {cltContainerID:aPatch.cltContainerID}
      )
   )
)

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.

Visit my blog Practical Power Apps

[Rosie]

ForAll(
   cltInspecting As aPatch,
   If(
      IsBlank(
         LookUp(
            cltInspectionResults,
            ContainerID = aPatch.cltContainerID
         ).ContainerID
      ),		 
      Collect(
         cltInspectionResults,
         {ContainerID:aPatch.cltContainerID}
      )
   )
)

Hello,

Thanks so much, I modified it a bit to get the names of the columns correct then the comparison worked and item 3 was added.  Thanks so much!

I did wonder it is was simpler to use Lookup instead of filter.
Do you use the As aPatch to make it easier to refer to the collection?

[WarrenBelz]

Hi @Rosie ,

I generally use the As statement to handle the Ambiguity and specifically state what value comes from where - it its not really necessary in your case, but needs to be used when the fields in the source and target have the same name.

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.

Visit my blog Practical Power Apps