On 5/30/13, Ryan Ollos <[email protected]> wrote:
> On Thu, May 16, 2013 at 11:41 PM, Olemis Lang <[email protected]> wrote:
>
>> On 5/16/13, Apache Bloodhound <[email protected]> wrote:
>> >
>> [...]
>> >
>> >  [attachment:t529_r1482461_hooks_missing_product.diff Attached patch]
>> >  solves this issue . Details
>> >
>>
>> Nevertheless IMO a better choice is to render :
>>
>>   1. an HTML error page
>>   2. products list with warning message on top
>>
>> What do you think ?
>>
>
> Either seem good to me. I agree that the behavior after the patch is
> better, but still not ideal.
>

Yes , I agree

> I studied the code for a short while, and it looks like you'd have to open
> the global environment and then raise a TracError. I'm sure you have a
> better understanding of this than I do.
>

I was thinking of redirecting to global products page and displaying a
custom message «Product %s not found» .

-- 
Regards,

Olemis.

Reply via email to