Power FX - Bulk Update Price List Items

[LukeASartain]

Hi

I've created a custom page in Power Apps to accept a percentage increase / decrease which will be applied to all price list items for a given price list in CDS. The issue is that the price always gets set to 0 whenever a formula is applied to the amount within the patch function. 

I can successfully patch the records using the existing value with following command. Using Fiddler confirms that the original value is being used in the patch.

IfError(    
        ForAll(
            gblPriceList.'Price List Items',
            Patch(
                'Price List Items',
                ThisRecord,
                {Amount: ThisRecord.Amount}
            )
        ),
   
    Notify(
        FirstError.Message,
        NotificationType.Error
    );    
);
Back();

 However as soon as I try to apply any calculations, however simple, the amount always gets set to 0. For example:

IfError(    
        ForAll(
            gblPriceList.'Price List Items',
            Patch(
                'Price List Items',
                ThisRecord,
                {Amount: ThisRecord.Amount * 1.05}
            )
        ),
   
    Notify(
        FirstError.Message,
        NotificationType.Error
    );    
);
Back();

I've tried a lot of different ways to get this to work including encasing the calculation within the value function {Amount: Value(ThisRecord.Amount * 1.05)} but nothing has worked. 

Any suggestions?

[AhmedSalih]

Hello, @LukeASartain, are you sure that ThisRecord.Amount has values that are more than 0? Use a collection to collect the values to test. 

[LukeASartain]

Hi @AhmedSalih 

Thanks for your prompt response. I have now amended my OnStart command to populate a priceListItems collection and inspecting this it only contains the price list item guids, not the amount so this may be where the issue is. 

Set(
    gblPriceListId,
    //GUID(Mid(Param("recordId"),2,Len(Param("recordId"))-2))    
    GUID("3cb5a11b-4be3-ed11-8846-6045bd0d6764")
);
Set(
    gblPriceList,
    LookUp(
        'Price Lists',
        'Price List' = gblPriceListId
    )
);
ClearCollect(priceListItems,gblPriceList.'Price List Items')

However when I used a similar Power FX command to copy a price list and it's associated price list items all attributes were returned for the price list items when I used a patch inside a ForAll('Price List'.'Price List Items').

I'm not sure what the expected behaviour is for this but I guess one solution would be to do a lookup inside my patch to return the full record but am concerned about the performance hit of doing so if there are a large number of price list items.

[AhmedSalih]

@LukeASartain, Lookup would indeed impact your performance. Have you though of using Power Automate Flow to achieve this price Changing? You can trigger the Power Automate Flow from within the PowerApps and pass the GUIDs to the flow.