From wapcaplet88 on June 05, 2010 21:41:12

The root problem here is in gui.py, in the load_args method. This method naively sets
Control values based on their vartype--in this case, since FlagOpt is a bool type, it
simply sets the flag and moves on. This is a design flaw that might be better handled
if Controls were responsible for parsing their own arguments (since they are
responsible for setting them in the first place).

Consider adding a counterpart to get_args, called set_args. Currently, the
Application starts off the chain of get_args, by calling all the Panels' get_args
methods, which in turn call their Controls' get_args methods. A set_args method
should work in much the same way, starting with a list of args that will be processed
down the chain, until a Control finds options/arguments it's interested in, and pops
them off the list. This chain of set_args calls should completely replace the current
Application.load_args method.

From wapcaplet88 on June 06, 2010 20:56:36

Labels: Priority-High

From grepper@gmail.com on August 28, 2010 18:52:09

Just a note that a SpacedText "list" will also fail the load_args test.
Example: tovid gui -files 1.mpg 2.mpg -rotate-thumbs 10 12
will produce a:
"1" "0"
in the text area,
and a "Unrecognized argument: 12" in the error output.

From grepper@gmail.com on August 28, 2010 18:54:52

Summary: tovid gui "Load script" fails to load FlagOpts or SpacedText