Comment 10 for bug 69200

Revision history for this message
Leo Milano (lmilano) wrote :

What I was proposing has not been applied. The first
few lines of the function should looke like this:
    #######################################################################
     def __selectGroup(self,groupid):
- if groupid:
- self.selectedgroupid = groupid
- lvi = self.groupidsToListItems[groupid]
- self.grouplist.setSelected(lvi,True)

Everything else in the function should be enclosed in the same "if groupid". That is, the validation for the function belongs to the function, not to the functon callers.

The version I am seeing in websvn is rev=621399. Here is the full path:

http://websvn.kde.org/trunk/playground/base/guidance/userconfig/userconfig.py?rev=621399&view=markup

Am I looking at the wrong place ?

Thanks!
-- Leo