Hi Scott,

On May 21, 2008, at 8:16 AM, Scott Gray wrote:

Hi Jacopo

Here's my opinion:
1. a null itemDescription should be fine for returns if it is fine for
orders

Ok, in rev. 658607 I did some changes so that the returns can work well with null itemDescriptions.

Thanks for your advices,

Jacopo



2. productName should take be more prominent when creating a new product than internalName, so that if they are the same then the user can leave internalName blank. Then in the back-end apps if internalName is null then
productName should be displayed

So to clarify, instead of always having internalName set and falling back to that for customers, we should always have productName set and fall back to
that for back-end users.

What do you think?

Scott

2008/5/21 Jacopo Cappellato <[EMAIL PROTECTED]>:

Scott, thanks for your comment... I understand your point of view.
The other options we have, if both productName and descriptions are null: 1) put the productId in the itemDescription (but it seems not a good option
to me) or
2) fix the code to work well with a null itemDescription
Or we may require that when we create/update the product, the productName
is set (directly or thru the content... but that would be trickier)

Hmmmm... I'm not sure which one is the best approach... probably #2

Jacopo




On May 20, 2008, at 9:16 PM, David E Jones wrote:


Yes, this is a good point. We probably shouldn't populate something the
customer sees with text from that field.

-David


On May 20, 2008, at 12:38 PM, Scott Gray wrote:

Hi Jacopo

It doesn't really bother me that much, but it seems funny to me that the system would use something explicitly defined as internal for customer facing pages. As a user I would expect that the customer would never see
the contents of that field.

Regards
Scott

2008/5/21 Jacopo Cappellato <[EMAIL PROTECTED]>:

Hi all,

I've noticed that the OrderItem.itemDescription is only set if the
product's productName is available; I would like to add code so that, if
the
productName is not set, then the internalName is used.
I need this because the OrderItem.itemDescription seems to be required
by
some screens and some of the return procedures (I got errors trying to
return orders with null itemDescriptions).

Is it ok?

Jacopo






Reply via email to