search

diefenbach / django-lfs

An online-shop based on Django
http://www.getlfs.com
BSD 3-Clause "New" or "Revised" License
622 stars 222 forks source link

BUG: message might not be set before reference #21

Closed honovation closed 12 years ago

honovation commented 12 years ago

It is a simple bug. When no property, the loop will not be executed, so the message is not defined.

diefenbach commented 12 years ago

Thanks,

I can't see, why the for loop should not be executed ever.

Kai

Zagrebelin commented 12 years ago

AFAIR this could be happen when you change type of product from plain product to variant, but do not specify properties. Or something like this. I had this bug too, but it was wrong usage of manage interface.

diefenbach commented 12 years ago

Thanks, I will examine further. OTOH @taowen's doesn't do any harm.