Slash commands and hiding minimap button lead to error

[Nercus]

By either using the Slash commands /mdt or /manbabydungeontools or going into the options to hide the minimap button leads to following error:

2x ...eManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua:1042: bad argument #1 to 'random' (interval is empty)
[string "=[C]"]: in function `random'
[string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:1042: in function `UpdateBottomText'
[string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:676: in function `ShowInterface'
[string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:32: in function `OnClick'
[string "@BigWigs\Libs\LibDBIcon-1.0\LibDBIcon-1.0-44.lua"]:144: in function <BigWigs\Libs\LibDBIcon-1.0\LibDBIcon-1.0.lua:142>

Locals:
(*temporary) = 0

Edit: Clicking on the minimap button leads to same error (opens the window though)

[T3ll1nG]

The same error is also thrown when using the minimap button.

[mmpashby]

this should fix the issue.

[T3ll1nG]

It didn't fix it for me.

[mmpashby]

@T3ll1nG Can you provide the error you get please? This fix did work for me on testing.

[T3ll1nG]

@Pashbee
You suggested changing line 570 in MDT/Locales/enUS

I run Win10 en-US and the gameclient is en-US too.

I get the same error message as before.

If you click the minimap button:
2x ...eManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua:1042: bad argument #1 to 'random' (interval is empty)
[string "=[C]"]: in function random' [string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:1042: in function UpdateBottomText'
[string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:676: in function ShowInterface' [string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:32: in function OnClick'
[string "@BigWigs\Libs\LibDBIcon-1.0\LibDBIcon-1.0-44.lua"]:144: in function <BigWigs\Libs\LibDBIcon-1.0\LibDBIcon-1.0.lua:142>

Locals:
(*temporary) = 0

If you use /mdt or /manbabydungeontools after wiping saved bugs
1x ...eManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua:1042: bad argument #1 to 'random' (interval is empty)
[string "=[C]"]: in function random' [string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:1042: in function UpdateBottomText'
[string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:676: in function ShowInterface' [string "@ManbabyDungeonTools\ManbabyDungeonTools-3.3.4.lua"]:62: in function ?'
[string "@FrameXML\ChatFrame.lua"]:5093: in function ChatEdit_ParseText' [string "@FrameXML\ChatFrame.lua"]:4757: in function ChatEdit_SendText'
[string "@FrameXML\ChatFrame.lua"]:4793: in function <FrameXML\ChatFrame.lua:4789>
[string "=[C]"]: in function `ChatEdit_OnEnterPressed'
[string ":OnEnterPressed"]:1: in function <[string ":OnEnterPressed"]:1>

Locals:
(*temporary) = 0

[mmpashby]

the PR got merged but you're right it does not work because of case sensitivity on the lookup. I am sorry. I will fix and put a PR in again.

[mmpashby]

@T3ll1nG fix here - I could re-produce the error you were seeing. Please let me know if that works.

[T3ll1nG]

@Pashbee that fix works just fine.
Nice 👍

[lexaprofessional]

@Pashbee i had the same issue but your fix + changing the casing worked for me as well