PXDefault Attribute is Updating Existing Records. Why?

Yes, you can cause this to happen in several ways:

• [PXFormula(typeof(Default<anotherField>))] (calls ‘defaulting’ when anotherField is updated)
• e.Cache.SetDefaultExt<thisField>(row)  (manually calls ‘defaulting’ on thisField)

What version are you running? I had a hunch, and there are some attributes that loop through the code and attempt to set field defaulting on all fields… maybe there is something in the version you are running that is causing it?


Example in 2025r1… in AP PaymentRefAttribute..

		protected virtual void InsertCheck(PXCache sender, APRegister payment, int CashAccountID, string PaymentMethodID, string CheckNbr)
		{
			PXCache<CashAccountCheck> cache = sender.Graph.Caches<CashAccountCheck>();
			CashAccountCheck check = new CashAccountCheck();

			List<PXDataFieldAssign> fields = new List<PXDataFieldAssign>();

			Dictionary<string, object> foreign_values = new Dictionary<string, object>
			{
				{typeof(CashAccountCheck.cashAccountID).Name.ToLower(), CashAccountID},
				{typeof(CashAccountCheck.paymentMethodID).Name.ToLower(), PaymentMethodID},
				{typeof(CashAccountCheck.checkNbr).Name.ToLower(), CheckNbr},
				{typeof(CashAccountCheck.docType).Name.ToLower(), payment.DocType},
				{typeof(CashAccountCheck.refNbr).Name.ToLower(), payment.RefNbr},
				{typeof(CashAccountCheck.finPeriodID).Name.ToLower(), payment.FinPeriodID},
				{typeof(CashAccountCheck.docDate).Name.ToLower(), payment.DocDate},
				{typeof(CashAccountCheck.vendorID).Name.ToLower(), payment.VendorID},
				{typeof(CashAccountCheck.Tstamp).Name.ToLower(), PXCache.NotSetValue},
			};

			foreach (string field in cache.Fields)
			{
				object NewValue;
				if (!foreign_values.TryGetValue(field.ToLower(), out NewValue))
				{
					cache.RaiseFieldDefaulting(field, check, out NewValue);
					if (NewValue == null)
					{
						cache.RaiseRowInserting(check);
						NewValue = cache.GetValue(check, field);
					}
				}
				if (NewValue != PXCache.NotSetValue)
				{
					PXCommandPreparingEventArgs.FieldDescription descr;
					cache.RaiseCommandPreparing(field, check, NewValue, PXDBOperation.Insert, typeof(CashAccountCheck), out descr);

					if (descr?.Expr != null)
					{
						fields.Add(new PXDataFieldAssign((Column)descr.Expr, descr.DataType, descr.DataLength, descr.DataValue));
					}
				}
			}
			PXDatabase.Insert<CashAccountCheck>(fields.ToArray());
		}

Theres the foreach for field and then Raises Field Defaulting. I assume some functionality like that may be hidden somewhere that is being called?


The other thought, is having a fielddefaulting event attribute, rather than the PXDefault, that checks the current value and then only sets if it currently is blank?

Yeah I’ll need to go this route. 

The problem I’m running into is, where do I define the event handler for a field of a record that doesn’t have a primary graph associated with it? 

​@Keith Richardson Man, you’re a life-saver! 

Here’s the code that ended up working. Just a slight adaptation. 
 

    public class SWItemConditionDefaultAttribute : PXEventSubscriberAttribute
    {
        public override void CacheAttached(PXCache sender)
        {
            base.CacheAttached(sender);
            sender.Graph.FieldDefaulting.AddHandler(
                BqlTable,
                _FieldName,
                OnDefaultItemCondition
            );
        }

        protected virtual void OnDefaultItemCondition(PXCache sender, PXFieldDefaultingEventArgs e)
        {
            if (e.Row is INItemLotSerial row)
            {
                var ext = PXCache<INItemLotSerial>.GetExtension<SWINItemLotSerialExt>(row);
                if (ext != null && string.IsNullOrEmpty(ext.UsrSWItemCondition))
                {
                    e.NewValue = SWItemCondition.NewItem;
                }
            }
        }
    }

And on the DAC. 
 

        #region UsrSWItemCondition
        [PXDBString(2, IsFixed = true)]
        [SWItemConditionDefault]
        [SWItemCondition.List]
        [PXUIField(DisplayName = "Item Condition", Visibility = PXUIVisibility.SelectorVisible)]
        public string UsrSWItemCondition { get; set; }
        public abstract class usrSWItemCondition : PX.Data.BQL.BqlString.Field<usrSWItemCondition> { }
        #endregion

I’m still confused about what’s actually happening with the INItemLotSerial record, because before attempting this override of the CacheAttached event, I attempted to implement a custom Field Defaulting event attribute that implemented IPXFieldDefaultingSubscriber. In it, I also added the check to make sure the value was null in the custom field before defaulting it to New. However it was still overriding the “Used” value. I did a bunch of debugging and somehow the custom field value was null in the cache, when the event was called, so it was always updating it to New.

Thanks again for the help. I owe you one!