On Sun, Mar 03, 2002, [EMAIL PROTECTED] wrote:
> I was looking at that the other day too. It can easily be changed to only choose
> autogenerate if the doc_file=PLUCKERDIR/channels/<CHANNEL NAME>/<CHANNEL
> NAME>
Why do you have to change the setting? If I as a user make a choice
that would make the name the same as it would have been if
autogenerated then it won't add anything but confusion if the setting
is changed the next time I visit the destination dialog. I (the user)
decide what the setting should be, not the program.
> Creation of a new channel via GUI puts creates a new channels
> doc_file=PLUCKERDIR/channels/<CHANNEL NAME>/<CHANNEL NAME>
> so they are fine that way. Channels that are created and manipulated entirely by GUI
> are fine.
Only if you use the autogenerated name. Otherwise it will end up in
either PLUCKERDIR or at the location given by a full path.
> I guess would be the no doc_file key exists would also need to be
> handled in some way. What do you suggest there?
No idea, since that sentence didn't make sense to me. Can you repeat
it in English, please ;-)
> Off the top of my head, the flow of the current way is to check somewhere in the
> startup to see if a NEW_CHANNEL_TEMPLATE section / subdir exists, and if not
> then create it.
Yes it does that, but it also *always* append '_' making it always
fail when looking for NEW_CHANNEL_TEMPLATE.
> It's probably better to just fix this bug in the toolkit than to
> hack around it.
I don't know where the bug is, but when I compile the desktop code
with debug info I get three assertions when I start the program
("invalid index in wxListBase::Item" in src/common/list.cpp). They
happen when the main_dialog ctor calls AttachUnknownControl for
main_dialog_channels_tab_main_listctr.
I don't think I will have time to debug this, though.
/Mike
