[Biopython-dev] [Bug 2967] AbstractCommandline silently accepts invalid parameter options

bugzilla-daemon at portal.open-bio.org bugzilla-daemon at portal.open-bio.org
Wed Dec 9 12:28:07 UTC 2009


http://bugzilla.open-bio.org/show_bug.cgi?id=2967





------- Comment #4 from lpritc at scri.sari.ac.uk  2009-12-09 07:28 EST -------
(In reply to comment #3)
> (In reply to comment #2)
>
> > We might still be able to block this via __setattr__, this needs some
> > experimentation.
> 
> That seems to be the best route to me, initially.  It might be worth removing
> the property magic in the AbstractCommandline.__init__(), and instead use
> __setattr__, __getattr__, and __delattr__, having them behave appropriately for
> known parameter names.
> 
> I'll have a go at doing that and put it in with the EMBOSS stuff I'm working
> on, just now.

Peter has pointed out that he'd like to retain discoverability, and so restrict
the change to a validating __setattr__ - which seems reasonable.

L.


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the Biopython-dev mailing list