debug message when configuring a product is confusing

Issue #742 resolved
Anonymous created an issue

Hi there Satchmo devs!

I was trying to get AJAX working, and kept getting a product not found error. It turns out, that the sample product I chose, happened to be configurable. The configuration was failing, but the log message didn't mention that.

Can I suggest the following verbiage? {{{log.debug('Could not fully configure a ConfigurablePoduct [%s] with: [%s]', product, chosenOptions)}}}


Terry {{{ Index: satchmo/apps/satchmo_store/shop/views/ =================================================================== --- satchmo/apps/satchmo_store/shop/views/ (revision 1956) +++ satchmo/apps/satchmo_store/shop/views/ (working copy) @@ -379,7 +379,7 @@ chosenOptions = optionids_from_post(cp, formdata) optproduct = cp.get_product_from_options(chosenOptions) if not optproduct: - log.debug('Could not find a product for: %s [%s]', product, chosenOptions) + log.debug('Could not fully configure a ConfigurablePoduct [%s] with: [%s]', product, chosenOptions) raise Product.DoesNotExist() else: product = optproduct }}}

Reported by tezza2k09

Comments (1)

  1. Log in to comment