always log warning in case a field is set on after new/save

throwing exception (even if only in developer mode) is quite aggresive for now.

#5609

de.metas.adempiere.adempiere/base/src/main/java-legacy/org/compiere/model/PO.java

@@ -1039,11 +1039,10 @@ private final boolean set_Value(final int index, final Object value)
 		}
 		if(m_currentChangeType != null && m_currentChangeType.isAfter())
 		{
-			final AdempiereException ex = new AdempiereException("Changing "+this+" on AFTER NEW/CHANGE shall be avoided because those changes won't be persisted in database.")
+			AdempiereException.logWarningIfDeveloperMode(log, () -> new AdempiereException("Changing " + this + " on AFTER NEW/CHANGE shall be avoided because those changes won't be persisted in database.")
 					.appendParametersToMessage()
 					.setParameter("columnName", ColumnName)
-					.setParameter("value", value);
-			ex.throwIfDeveloperModeOrLogWarningElse(log);
+					.setParameter("value", value));
 			// NOTE: don't return, allow setting the value because maybe some legacy code depends on it. At least we informed the developer.
 		}