[Bugfix][Script] Too fast Gizmo "runs" lead to crash

[saywaking 289]

Apparently, too fast Gizmo invokes lead to crash.

function main()
  cmd = gizmo.run(table);
  if cmd.Result == "refresh" then
    main();
  end
end

I can workaround this, when i put wait(500) before the recursion.

But it would be better, if there is a Queue-System for Gizmo-Scenes.

Edit: Also, Script.ShowMessageBox() in Gizmo Scenes is not possible.

[MaesterRowen 14]

I'll look into a solution for running (2) gizmo scenes too fast.   For the message box, the gizmo object has a special message box function that should be used when in a gizmo.  Infact, this applies to keyboard and passcode as well.

static const luaL_Reg l_gizmoMethods[] = {
	// Methods added in 0.6b
	{ "RegisterCallback",			l_gizmoRegisterCallback },
	{ "RegisterAnimationCallback",	l_gizmoRegisterAnimationCallback },
	{ "Dismiss",					l_gizmoDismiss },
	{ "SetXLScene",					l_gizmoSetXLScene },
	{ "SetCommandText",				l_gizmoSetCommandText },
	{ "SetCommandEnabled",			l_gizmoSetCommandEnabled },
	{ "InvokeUI",					l_gizmoInvokeUI },
	{ "RegisterControl",			l_gizmoRegisterControl },
	{ "SetTimer",					l_gizmoSetTimer },
	{ "KillTimer",					l_gizmoKillTimer },
	{ "PlayTimeline",				l_gizmoPlayTimeline },
	{ "ShowKeyboard",				l_gizmoShowKeyboard },
	{ "ShowMessagebox",				l_gizmoShowMessagebox },
	{ "ShowPasscode",				l_gizmoShowPasscode },
	{ "Notify",						l_gizmoNotify },
	// Methods added in 0.7b
	{ "SetXLScene",					l_gizmoSetXLScene },
	{ nullptr,						nullptr }
};

 

[saywaking 289]

I see, that's why i got nil. I listed here, that it was not working.

There is 

Script.ShowMessageBox() and

userdata:ShowMessagebox()

In the Readme, there is userdata:ShowMessageBox()

edit: it appears, that gizmo:ShowMessagebox() needs 4 arguments, and the first one must be a number.

In that method-list, i also see "Notify", which is not listed in the Readme (correctly). Well i decided to use Script.ShowNotification() and no MessageBox for that Temperature Manager Script.

Also SetXLScene is nice to see. I hope it's full screen xD

[MaesterRowen 14]

Its not full screen, but the same size as the file manager scene so significantly bigger.

[MaesterRowen 14]

2 hours ago, saywaking said:

edit: it appears, that gizmo:ShowMessagebox() needs 4 arguments, and the first one must be a number.

The first argument is an arbitrary identifier number.   This number is supplied back to you in the OnMessageBoxReturn callback so you can tell which message box was closed in the event you have more than one message box to detect responses from.

[saywaking 289]

Another one:

When i work with timer and miliseconds, and close the Gizmo Scene, the console freezes.

"this:SetTimer(1, 1);"

I guess that's not a good idea.