Answered

Patch to Sharepoint not competing before I perform ClearCollect in next statement

(0) Share

Report

Posted on by brown_1892

I am patching a Sharepoint list with records that have changed and then performing a ClearCollect to get the whole list back. I have sometimes had an issue where it has patched to Sharepoint but is not in my Collection.

Is it performing the ClearCollect before Sharepoint has finished Patching?

// Patch Planned lines to SharePoint
ForAll (
 Filter(
 colPlannedLines,
 State = "Updated" || State = "New"
 ),
 Patch(
 Planned,
 Defaults(Planned),
 {
 MonthCommencing: locMonthCommencing,
 Person: {
 '@odata.type': "#Microsoft.Azure.Connectors.SharePoint.SPListExpandedUser",
 Claims: Person.Claims,
 Department: "",
 DisplayName: Person.DisplayName,
 Email: Person.Email,
 JobTitle: "",
 Picture: ""
 },
 Project: {
 '@odata.type': "#Microsoft.Azure.Connectors.SharePoint.SPListExpandedReference",
 Id: Project.Id,
 Value: Project.Value
 },
 PlannedDays: PlannedDays
 }
 )
);
//Get current months Planned Hours
ClearCollect(
 colPlannedLines,
 Filter(
 Planned,
 (MonthCommencing = locMonthCommencing)
 ),
 "ID",
 "MonthCommencing",
 "Person",
 "Project",
 "PlannedDays"
);

Categories:

Building Power Apps

All responses (11)

Answers (1)

Sort by

CNT 10,921 Super User 2024 Season 1 on at

Like
a
(0)

Report
Copy link

Link copied!

@brown_1892 If all that you want is to add the new record to your collection, just patch it in the collection as well rather than getting the whole list again. This will be more performant and also help to avoid the issue you are having.
brown_1892 38 on at

Like
a
(0)

Report
Copy link

Link copied!
Hi, sorry I cut out some of the code, I'm doing an Update if it exists or an Insert if it is New

// Patch Planned lines to SharePoint ForAll ( Filter( colPlannedLines, Not IsBlank(Person.Claims) And (State = "Updated" || State = "New") ), Patch( Planned, // Is this a new line If( State = "New", // This is a new line so use defaults Defaults(Planned), // Else, this is an exisiting line so patch (update) to SharePoint {ID: ID} ), { MonthCommencing: locMonthCommencing, Person: { '@odata.type': "#Microsoft.Azure.Connectors.SharePoint.SPListExpandedUser", Claims: Person.Claims, Department: "", DisplayName: Person.DisplayName, Email: Person.Email, JobTitle: "", Picture: "" }, Project: { '@odata.type': "#Microsoft.Azure.Connectors.SharePoint.SPListExpandedReference", Id: Project.Id, Value: Project.Value } PlannedDays: PlannedDays } ) ); //Get current months Planned Hours ClearCollect( colPlannedLines, Filter( Planned, (MonthCommencing = locMonthCommencing) ), "ID", "MonthCommencing", "Person", "Project", "PlannedDays" );
CNT 10,921 Super User 2024 Season 1 on at

Like
a
(0)

Report
Copy link

Link copied!

@brown_1892 You can do the same update/patch to your collection as well!
brown_1892 38 on at

Like
a
(0)

Report
Copy link

Link copied!

Hi, in my collection I have a dummy ID for 'new' records and I am trying to get the real ID back from Sharepoint and this is the reason why I'm doing the ClearCollect. Are you saying that I don't need to do this and if so how do I get the real Sharepoint ID to use in my App going forward?
CNT 10,921 Super User 2024 Season 1 on at

Like
a
(0)

Report
Copy link

Link copied!

@brown_1892 The return value of Patch is the record that you modified or created. You can use this to update/patch your collection.
brown_1892 38 on at

Like
a
(0)

Report
Copy link

Link copied!

Hi, thanks for all your help on this. Where in my code would I need to add the new Patch to my collection
CNT 10,921 Super User 2024 Season 1 on at

Like
a
(0)

Report
Copy link

Link copied!

@brown_1892 It would be better to store the record returned by the Patch in a variable and than use it in the Patch to the collection. You could do it in the same place where your ClearCollect is now.
brown_1892 38 on at

Like
a
(0)

Report
Copy link

Link copied!

Hi, my Patch is in a For All loop, am I still able to store the record in a variable?
Verified answer

CNT 10,921 Super User 2024 Season 1 on at

Like
a
(0)

Report
Copy link

Link copied!

@brown_1892 You can do it by using the With statement.
ForAll(Filter(...),
With({varPatchedRecord:Patch(SPList,.......)},
Patch(collection...)
))
brown_1892 38 on at

Like
a
(0)

Report
Copy link

Link copied!

This has worked. Thank you so much!