LookUp Inside Patch Inside ForAll With Multiple Conditions

(0) Share

Report

Posted on by TrevorMorrice

Hey!

I'm having an issue with patching old records from a combobox.

This is my current code that isn't working its giving a network error code at the lookup

ForAll(
 ComboBoxAssets_1.SelectedItems As _item,
 Patch(
 [@'Asset Trackings'],
 LookUp(
 [@'Asset Trackings'],
 _item.Asset = AssetLookup.Asset && 
 IsBlank('End Date')),
 {
 Active: 'Active (Asset Trackings)'.Yes,
 'End Date': DatePickerStart_4.Value
 }
 )
);

It runs without errors if I remove one of the two conditions, doesn't matter which one. (obvsly grabs the wrong record)

I'm fairly sure I'm allowed to have && in there.

🙂

Thanks for any help!

Categories:

Building Power Apps

All responses (2)

Answers (1)

Sort by

Verified answer

TrevorMorrice 81 on at

Like
a
(0)

Report
Copy link

Link copied!
I figured out a solution,
Had to create another collection, and do the conditions in there instead?
Clear(OldAssets); ForAll( ComboBoxAssets_1.SelectedItems As _item, Collect( OldAssets, LookUp( [@'Asset Trackings'], AssetLookup.Asset = _item.Asset && IsBlank('End Date') ) ) ); ForAll( OldAssets As _item, Patch( [@'Asset Trackings'], _item, {'End Date': DatePickerStart_4.Value} ) );
Ethan_009 4,840 Moderator on at

Like
a
(0)

Report
Copy link

Link copied!
Hi @TrevorMorrice ,

I'm not sure where things are going wrong but I hope that there aren't multiple record based on your filter in Lookup
Basically, Lookup is used to catch 1 record and not multiple, in this case you are looping each item & those should have 1 Item lookup based on condition provided.
ForAll( ComboBoxAssets_1.SelectedItems As _item, With( { ItemRecord: LookUp('Asset Trackings', _item.Asset = AssetLookup.Asset && 'End Date' = Blank(), ThisRecord) }, Patch( 'Asset Trackings', ItemRecord, { Active: 'Active (Asset Trackings)'.Yes, 'End Date': DatePickerStart_4.Value } ) ) );

Hope this code helps also you can directly use "yes" instead of choice enum for Active.