possible bug in GenericFilter

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

possible bug in GenericFilter

Richard Taylor-2
I stumbled across a possible bug in _GenericFilter.py. All of the get_cursor
methods appear to have their param the wrong way around. This has caught me
out when I was writing code to use GenericFilter but I have not seen any
errors in the existing filter UI.

Is this a bug? and if not why is it done this way?

    def get_cursor(db, self):
        return db.get_person_cursor()

Regards

Richard

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Gramps-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/gramps-devel
Reply | Threaded
Open this post in threaded view
|

Re: possible bug in GenericFilter

Alex Roitman
Richard,

On Tue, 2007-01-16 at 08:48 +0000, Richard Taylor wrote:
> Is this a bug? and if not why is it done this way?
>
>     def get_cursor(db, self):
>         return db.get_person_cursor()

Looks like a typo to me. I don't think any of the existing filters
uses the cursor, that's why it has not been caught earlier. I think
the existing filters all operate on a list of handles, so the
cursor clause has not been invoked by the existing filters.

Alex

P.S. Looks like every cursor (family, source, etc) has this typo
in _GenericFilter.py

--
Alexander Roitman   http://www.gramps-project.org


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Gramps-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/gramps-devel

signature.asc (196 bytes) Download Attachment