[Zope3-dev] Re: Deleting utilities in site management doesn't work correct

[jürgen Kartnaller]

Hi Tres.

I could now manage to have a relatively simple test (see attached file).

I also found and fixed some other problems I encountered with the
utility registration.

Can you please have a look at the test implementation and let me know if
this look ok.

This test will not work with the current implementation of the
SiteManagementView !
I already fixed some problems in my sandbox.

Problems solved :
- adding a utility without name
- renaming utilities to a name another already renamed utility had
- deleting more than one utility
- better message support
   - show the message at the utility not just on top of the page
   - show UserError exception as message

Jürgen


Tres Seaver wrote:
> Stephan Richter wrote:
> 
>>>On Wednesday 05 October 2005 17:00, jürgen Kartnaller wrote:
>>>
>>>
>>>>When more than one utility is selected for deleting in the site
>>>>management view, only the first selected utility is deleted.
>>>>
>>>>The problem is that the content of regManager is changed inside the
>>>>while iteratating over it.
>>>>Here is the fix which works for me :
>>>>If this is applicable, can please someone check it into the trunk ?
>>>>The file is in zope.app.component.browser
>>>
>>>
>>>Could you create an issue in the bug collector for this? If you cannot create 
>>>a unit test for it, then I will. :-)
> 
> 
> Stephan,
> 
> I was in the midst of writing a 'tools.txt' doctest which would have
> provided the basis for such a test when I wrote my "keening" post.  In
> case you missed the attacment, here it is again.
> 
> Tres.
> --
> ===================================================================
> Tres Seaver          +1 202-558-7113          tseaver at palladion.com
> Palladion Software   "Excellence by Design"    http://palladion.com

-- 

---------------------------------------------------
Jürgen Kartnaller   mailto:juergen_at_kartnaller.at
                    http://www.kartnaller.at
                    http://www.mcb-bregenz.at
---------------------------------------------------