Inconsistencies when creating records via graphs

Thank you for replying @darylbowman
I like the idea of using SetValueExt to ensure the events fire. I’ve adjusted the code to use your suggestions, this is shown below.

BAccount bAccount1 = Base.Select<BAccount>().FirstOrDefault(x => x.AcctCD == "VEND001");
POOrderEntry pOOrderEntry = PXGraph.CreateInstance<POOrderEntry>();
POOrder pOOrder = pOOrderEntry.Document.Insert();
pOOrderEntry.Document.SetValueExt<POOrder.vendorID>(pOOrder, bAccount1.BAccountID);
pOOrderEntry.Document.Update(pOOrder);
try
{
	pOOrderEntry.Actions.PressSave();
}
catch (Exception ex)
{
	string s = ex.Message;
}            

Unfortunately I get the same result. An error saying "The currency 'USD' of the pay-to Supplier 'VEND001' differs from currency 'GBP' of the document."
This is because the pOOrder.CuryID property is set to “GBP”
Using the watch window I can see that:
After the insert line, CuryID defaults to GBP. 
After the SetValueExt line and the Update line, CuryID remains it GBP.

Don’t suppose you have any other ideas?

I have had a look at the source code for POOrderEntry as you suggested and I may have found something. 

Below is a very cut down version of the source code showing that the FieldUpdated event for VendorID is defined in the main POOrderEntry graph. But the event is overridden by the MultiCurrency class, the currency is set by the overriding class, which then calls the base method.

public class POOrderEntry : PXGraph<POOrderEntry, POOrder>, PXImportAttribute.IPXPrepareItems, IGraphWithInitialization
{
	protected virtual void POOrder_VendorID_FieldUpdated(PXCache sender, PXFieldUpdatedEventArgs e)
	{
		//set a lot of values such as default location and terms etc
	}

	public class MultiCurrency : MultiCurrencyGraph<POOrderEntry, POOrder>
	{
		[PXOverride]
		public virtual void POOrder_VendorID_FieldUpdated(PXCache sender, PXFieldUpdatedEventArgs e, PXFieldUpdated baseMethod)
		{
			POOrder pOOrder = (POOrder)e.Row;
			if (e.ExternalCall || pOOrder == null || pOOrder.CuryID == null || pOOrder.OverrideCurrency == true)
			{
				SourceFieldUpdated<POOrder.curyInfoID, POOrder.curyID, POOrder.orderDate>(sender, pOOrder);
			}

			baseMethod(sender, e);
		}
	}
}

When I use the following to update the vendor id I think the FieldUpdated event from POOrderEntry is being called and not the overridden FieldUpdated event from the MultiCurrency class. This would explain why location is set but currency is not.

pOOrderEntry.Document.SetValueExt<POOrder.vendorID>(pOOrder, bAccount1.BAccountID);

Problem is, I don’t know how to instantiate POOrderEntry as the MultiCurrency class, or whether I should need to. Does anyone have any suggestions?

Hi @Chris Hackett , I don’t really have a ‘solution’ as such. I was really trying to understand why using a graph in code produces different results to using a graph in the UI. I’m running into this problem quite a lot in different parts of the system but this PO example seems to be a good but simple example, requiring only small code sample. I really expected the graph to populate the vendor’s currency for me in this case. If it doesn’t I would like to understand why and how to make that happen. The alternative is to search for the currency and set it myself, but surely this the correct approach - I thought the idea of using a graph was that we can also use the inbuilt business logic of the graph. Problems like this leave me feeling less confident in developing using Acumatica as it can sometimes feel a bit random. If you had a contact who could explain what is happening that would be much appreciated.