Yes, I'm 102% sure.

I'll pass, thanks. :)

I feel your pain. :java:

Is this supposed to say something different? It's been this way for quite a while.

Why do you think I posted it in this forum? Did you miss the mistake? :)

99% of the time,

customer.DataExtRetList != null

. This was supposed to fix the edge cases when

customer.DataExtRetList == null

.

if (customer.DataExtRetList != null) {
this.QBInfo.DateCatSent = "";
this.Source = "";
} else {
for (int i = 0; i < customer.DataExtRetList.Count; i++) {
IDataExtRet ext = customer.DataExtRetList.GetAt(i);
if (ext.DataExtName.GetValueOrEmpty() == "Date Cat. Sent")
this.QBInfo.DateCatSent = ext.DataExtValue.GetValueOrEmpty();
else if (ext.DataExtName.GetValueOrEmpty() == "Source")
this.Source = ext.DataExtValue.GetValueOrEmpty();
}
}

Agreed. The problem is, we will never have an ideal world.