Thanks @Paul_C
Your assessment is spot on with the behavior I'm witnessing. I did some tests just now and tried to summarize what I'm seeing and the steps I took. I hope this helps.
Session ID
Session ID: e4116311-e996-475d-a7a1-72d3a278ea71
PowerApps 3.18111.15
Testing and results
Code indicated in blue is supplied at bottom of post
- Run Full code - this code first gets the latest qualifying record and updates it before writing a brand new record. All subsequent runs, the lookup does not "see" the latest record that was added, even after waiting several minutes between runs.
- Run Button 1 > then wait > then run Button 2 - the lookup still does not "see" the latest record that was added. If however I run Button 2 again (without running Button 1, it returns the correct result every time). This appears to be the "smoking gun" but not sure what it means.
- Add item meeting criteria directly to SP list (Events) > then run Button 2 - the lookup still does not "see" the latest record that was added. If however I run Button 2 again (without running Button 1, it returns the correct result every time). This is the same result as the test above.
Actual code used in test
Full code - all operations together (note highlighted block)
Set(varStartOperationTime,Now());
//put code below to test
//**********************************************
//Set(varTask,LookUp(Tasks,ID=Value(TextInput_varOperation1.Text))); //set TaskID var manually
Refresh(Tasks);
Set(varTask,GalleryCurrentUserTasks.Selected); //set TaskID var the same as in prod
//SET EVENT STATUS MINUTES VARIABLES***************************************************************************
//New
//set the event class of the current event so we know which type to update later with the minutes in this status. choices are 'Employee' or 'Task'
Set(varCurrentEventClass,"Task"); // ** update to match current event class **
//find last qualifying record by looking up the first one in a sorted, filtered set
Refresh(Events);
Set(varPreviousEvent,
LookUp(Sort(Events,ID, Descending),
EventClass = varCurrentEventClass && TaskID = varTask.ID
)
);
//END EVENT SET STATUS MINUTES VARIABLES***************************************************************************
// catch errors and record to a table
If(!IsEmpty(Errors(Events).Record),
Patch(
Errors, //don't change
{ID: Blank()}, //don't change
{Title: "Events table error"},
{Error: First(Errors(Events)).Error}, //don't change
{Column: First(Errors(Events)).Column}, //don't change
{Message: First(Errors(Events)).Message}, //don't change
{Record: First(Errors(Events)).Record.ID}, //don't change
{ErrorDate:Now()}, //don't change
{App:varThisAppName}, //don't change
{Comments:"Writing event status minutes during test operation 1"} // ** update this to indicate current context of error **
);
Revert(Events)
);
//WRITE PREVIOUS EVENT STATUS MINUTES ***************************************************************************
//set the event's time to use in the calculation
Set(varPreviousEventTime,varPreviousEvent.Time);
// calculated minutes between previous time & now
Set(varPreviousEventStatusMinutes,DateDiff(varPreviousEventTime,Now(),Minutes));
//now write calculated number of minutes to previous event row determined earlier
//Refresh(Events);
Patch(
Events,
varPreviousEvent,
{StatusMinutes:varPreviousEventStatusMinutes}
);
//END WRITE PREVIOUS EVENT STATUS MINUTES ***************************************************************************
If(CheckboxOperation1.Value=true, //allows for testing code with and without writing a new event more easily
//WRITE NEW EVENT ***************************************************************************
//write timestamped event for **current** update
//Refresh(Events);
Patch(
Events,
{ID: Blank()}, // faster than using "Defaults" as only one call is needed
{Title: varThisAppName & " New Test Event Operation 1"},
{EventClass:varCurrentEventClass}, // set earlier, no need to change
{EventType:"New Event Test operation 1"}, // ** update this to indicate current context **
{TaskID:varTask.ID}, // auto number row ID from SharePoint list
{TaskStatus:"New Event Test Assigned"},
{EmpStatus:"New Event Test Assigned"},
{Time:Now()}
);
// catch errors and record to a table
If(!IsEmpty(Errors(Events)), //don't change
Patch(
Errors, //don't change
{ID: Blank()}, //don't change
{Title: "Events Error"}, //don't change
{Error: First(Errors(Events)).Error}, //don't change
{Column: First(Errors(Events)).Column}, //don't change
{Message: First(Errors(Events)).Message}, //don't change
{Record: First(Errors(Events)).Record.ID}, //don't change
{ErrorDate:Now()}, //don't change
{App:varThisAppName}, //don't change
{Comments:"Writing new event during test operation 1"} // ** update this to indicate current context **
);
Revert(Events) //don't change
)
//END WRITE NEW EVENT ***************************************************************************
);
//**********************************************
//put code above to test
Set(varEndOperationTime,Now()); //don't change this
Set(varOperationSeconds,DateDiff(varStartOperationTime,varEndOperationTime,Milliseconds))//don't change this
Button 1 (highlighted code removed)
Thanks @Paul_C
Your assessment is spot on with the behavior I'm witnessing. The difficult thing is getting it to reliably repeat in order to troubleshoot. I'm fairly certain that I already performed the operations in separate operations as you suggested and it proved to fix the issue but I am going to perform the test again. I will provide an update and session info if I can reproduce.
Session ID of app after recreating the issue at 11/20/2008 10:14 am EST
Session ID: e4116311-e996-475d-a7a1-72d3a278ea71
PowerApps 3.18111.15
Full code - all operations together
Set(varStartOperationTime,Now());
//put code below to test
//**********************************************
//Set(varTask,LookUp(Tasks,ID=Value(TextInput_varOperation1.Text))); //set TaskID var manually
Refresh(Tasks);
Set(varTask,GalleryCurrentUserTasks.Selected); //set TaskID var the same as in prod
//SET EVENT STATUS MINUTES VARIABLES***************************************************************************
//New
//set the event class of the current event so we know which type to update later with the minutes in this status. choices are 'Employee' or 'Task'
Set(varCurrentEventClass,"Task"); // ** update to match current event class **
//END EVENT SET STATUS MINUTES VARIABLES***************************************************************************
// catch errors and record to a table
If(!IsEmpty(Errors(Events).Record),
Patch(
Errors, //don't change
{ID: Blank()}, //don't change
{Title: "Events table error"},
{Error: First(Errors(Events)).Error}, //don't change
{Column: First(Errors(Events)).Column}, //don't change
{Message: First(Errors(Events)).Message}, //don't change
{Record: First(Errors(Events)).Record.ID}, //don't change
{ErrorDate:Now()}, //don't change
{App:varThisAppName}, //don't change
{Comments:"Writing event status minutes during test operation 1"} // ** update this to indicate current context of error **
);
Revert(Events)
);
//WRITE PREVIOUS EVENT STATUS MINUTES ***************************************************************************
//set the event's time to use in the calculation
Set(varPreviousEventTime,varPreviousEvent.Time);
// calculated minutes between previous time & now
Set(varPreviousEventStatusMinutes,DateDiff(varPreviousEventTime,Now(),Minutes));
//now write calculated number of minutes to previous event row determined earlier
//Refresh(Events);
Patch(
Events,
varPreviousEvent,
{StatusMinutes:varPreviousEventStatusMinutes}
);
//END WRITE PREVIOUS EVENT STATUS MINUTES ***************************************************************************
If(CheckboxOperation1.Value=true, //allows for testing code with and without writing a new event more easily
//WRITE NEW EVENT ***************************************************************************
//write timestamped event for **current** update
//Refresh(Events);
Patch(
Events,
{ID: Blank()}, // faster than using "Defaults" as only one call is needed
{Title: varThisAppName & " New Test Event Operation 1"},
{EventClass:varCurrentEventClass}, // set earlier, no need to change
{EventType:"New Event Test operation 1"}, // ** update this to indicate current context **
{TaskID:varTask.ID}, // auto number row ID from SharePoint list
{TaskStatus:"New Event Test Assigned"},
{EmpStatus:"New Event Test Assigned"},
{Time:Now()}
);
// catch errors and record to a table
If(!IsEmpty(Errors(Events)), //don't change
Patch(
Errors, //don't change
{ID: Blank()}, //don't change
{Title: "Events Error"}, //don't change
{Error: First(Errors(Events)).Error}, //don't change
{Column: First(Errors(Events)).Column}, //don't change
{Message: First(Errors(Events)).Message}, //don't change
{Record: First(Errors(Events)).Record.ID}, //don't change
{ErrorDate:Now()}, //don't change
{App:varThisAppName}, //don't change
{Comments:"Writing new event during test operation 1"} // ** update this to indicate current context **
);
Revert(Events) //don't change
)
//END WRITE NEW EVENT ***************************************************************************
);
//**********************************************
//put code above to test
Set(varEndOperationTime,Now()); //don't change this
Set(varOperationSeconds,DateDiff(varStartOperationTime,varEndOperationTime,Milliseconds))//don't change this
Button 2 (removed highlighted code run in separate operation)
//find last qualifying record by looking up the first one in a sorted, filtered set
Refresh(Events);
Set(varPreviousEvent,
LookUp(Sort(Events,ID, Descending),
EventClass = varCurrentEventClass && TaskID = varTask.ID
)
);
