dogtail-devel [Bug 321196] Commit of 2005-11-08 breaks all of the existing object-orientated dogtail scripts

["dogtail (bugzilla.gnome.org)" <bugzilla-daemon bugzilla gnome org>]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=321196
 dogtail | Framework | Ver: CVS HEAD





------- Additional Comments From Dave Malcolm  2005-11-18 21:05 -------
As discussed on IRC, I think you should commit this but leave the bug open:  the
patch is a big improvement, and gets the scripts closer to working, and restores
most of the usability of the OO API; however the script removes the synthesized
"action" wiring (which most of the OO wrappers and OO example scripts rely on),
hence they still won't work, I think.

The use of a dict rather than a list is cool.

IMHO we should retain the synthesized action methods in spite of the ambiguity
over the "menu" method, but we should mark them clearly in the documentation as
a shortcut to calling: node.action['name-of-action'].do()  and be clear in the
documentation about where this breaks down.   I'm thinking about this from an
ease-of-use POV for people using the OO API.

(I think we badly need unit tests, there's a separate bug open about this)

------- You are receiving this mail because: -------
You are watching the assignee for the bug.
You are watching the QA contact.