#561: Display more user-friendly error page when navigating to a non-existing
product
---------------------------+-----------------------
  Reporter:  rjollos       |      Owner:  rjollos
      Type:  enhancement   |     Status:  review
  Priority:  major         |  Milestone:  Release 6
 Component:  multiproduct  |    Version:
Resolution:                |   Keywords:  hooks
---------------------------+-----------------------

Comment (by olemis):

 Replying to [comment:7 rjollos]:
 > There are a few issues with the patch:
 >  * Warning message is displayed when `href` is `/products` or
 `/products/`.
 >   * `/products`: Warning Product None not found
 >   * `/products/`: Warning Product not found
 >  * The //Create products// button no longer works.
 >
 > Since I committed some of the cleanup/refactoring changes already, I've
 attached a revised version of the patch:
 [attachment:t561_r1493739_hooks_no_product-1.diff]
 >

 Ok, I'll take a look . Notice that patches for both #561 and #541 are
 needed .

 > Unrelated to your patch, at the end of `process_request`,
 `product_view.html` is returned.

 > However I can't find a way to cause the product view page to be
 displayed.

 Two options

   1. /products/x/products/x
   2. set dashboard as default handler for product home page by using
 TracIni options

 > I would have guessed that the products listed on the `product_list` page
 (`/products`) would link to the `product_view` page, but the links seem to
 direct to the default handler for the product.

 AFAICT, that's expected behavior if using the default bootstrap handlers
 together with environment / request factories distributed with Bloodhound
 . May be different in other deployments by implementing custom bootstrap
 handlers .

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/561#comment:8>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker

Reply via email to