G2G preregistering - one small issue

[Mastiff]

When pre-registering a client, I would expect the focus to go name, IP, Port, Password when tabbing through them. But it doesn't go to password at all. Shouldn't that happen?

[JohnHind]

The password entry is in the tab loop, by you are right - it is way out of order. There is no obvious way of controlling tab order in the DUI designer. I guess it might be achieved by manually editing the XML file, but that sounds like a lot of work - is it this important?

[Promixis]

John,

if you edit the xml file and change the order of the nodes, tabbing will be set right. I can get you the name of free one later today if you want.

[JohnHind]

Ah! Manually editing unformatted XML, what larks :roll:

Consider my penance served for the above bug!

[Promixis]

Ah! Manually editing unformatted XML, what larks :roll:

Consider my penance served for the above bug!

:D no kidding...

[Mastiff]

Was that G2GUI.lua meant to fix the taborder? It didn't on any of my systems. Or was that for something else? Or should I put it in another directory then the main luascript?

[JohnHind]

Sorry, uploaded the wrong file - this extracts to an xml file which needs to replaced the like-named file in \plugins\UI. BB will not let me upload an xml file directly.

Updated in original post.

[Mastiff]

That helped! :D

[Mastiff]

OK, this is screwy! I pre-registered a client, but with the wrong IP address. So I tried to remove it again. And this is what I got for my trouble:

TreeScript (golua): ...Files\Promixis\Girder\/plugins/treescript/G2G UI.lua:39: bad argument #4 to `format' (string expected, got nil)
stack traceback:
[C]: in function `format'
...Files\Promixis\Girder\/plugins/treescript/G2G UI.lua&#58;39&#58; in function <...Files\Promixis\Girder\/plugins/treescript/G2G UI.lua&#58;31>
...Files\Promixis\Girder\/plugins/treescript/G2G UI.lua&#58;144&#58; in function <...Files\Promixis\Girder\/plugins/treescript/G2G UI.lua&#58;103>


Also it seems it's possible to have two clients with the same name and IP pre-registered. This is rather messy here at the moment.

[Promixis]

Tor, with the new comserver can you check and see if it still crashes with phidgets?

And how are you initializing the phidget com object -> ie what kind of event?

[Mastiff]

Mike, I haven't yet put in the new comserver. I thought after John's comment that this was internal for him and Ron, but I can download and install it. Or do you mean the comserver in Beta 16?

And I initalize the phidgets on script enable and Girder enable.

[JohnHind]

Tor,

Could you please help me narrow this down with by running this in the Lua console and posting the result:

table.print(g2g.GetClients(true))

Also a screen capture of the config screen with the duplicate clients would be useful.

Thanks, John.

[Mastiff]

John, this is not your lucky day... I had to reboot the server because of a harddisk change (not the OS, just some ripped DVDs), and when it came back up, there were no duplicate clients. Actually there were no preregistered clients at all, even though I had pre-registered three clients! I have now restartet Girder again after having pre-registered the clients correctly, and now they stay. And I was unable to dual register any clients...

[JohnHind]

OK, thanks anyway. I'll try to find it with a code review. Pre-registrations are stored locally as soon as you enter them, but only persisted if and when you press "Apply". With all the other crashes we've been getting I'd guess it fell over un-applied.

[Mastiff]

Man, that's counter intuitive! You shouldn't have to click apply after clicking pre-register. Can't you make that button do everything?

[JohnHind]

Actually, it's NOT having to press apply to pre-register for that session that should be counter-intuitive. Normally settings changes have to be applied. It will not let you out of the config screen without being prompted to apply, it's only if you elect not to apply or if Girder crashes with the config still open that they'll not get persisted.

[Mastiff]

I still disagree! Nice night for a fight, so rumble, young man, rumble! 8) One button should be enough for everybody, just like 640 kB of memory should be enough for everybody! :wink:

[Mastiff]

Mike, are you listening? You saw the question I asked after your last message in this thread?

[JohnHind]

One button should be enough for everybody, just like 640 kB of memory should be enough for everybody! :wink:

Says man with 10 computers and two networks :P

Seriously though, Ron is on top of the problem and there should be a new version of comsvr.dll shortly, so I'd hold off further experiments until then.

[Mastiff]

13 computers in use from time to time at the moment... Some of them as advanced TV's (so I can watch Internet TV while I'm in the garage working with the car or something). OK, I'll just wait until Ron has something new, then I'll do another test.