PDA

View Full Version : Crom dice string for MoreCore



Ranzarok
June 29th, 2019, 05:21
Heya MoreCore folks. I noticed the /crom dice string is returning an error that 'sSize' is a nil value in the manager_custom_crom.lua

Versions:
MoreCore - 1.51
CoreRPG - 3.37
FG - 3.3.7

No extension involved except for the MoreCore ruleset.

Thanks for the great ruleset BTW.

damned
June 29th, 2019, 08:34
Thanks - Ill investigate.

Ranzarok
June 29th, 2019, 22:51
Great, thanks damned. When you do get it ironed out, would you mind sharing what it was? I am trying to teach myself this stuff as I would like to do some extension work. Been through all of your vids, the MoreCore and FG dev stuff. It's coming, but slooow. :)

Thanks again.

Ranzarok
July 7th, 2019, 03:09
I think I found the issue.

The match pattern was missing the repetition modifier * in the string. manager_custom_crom.lua (line 36)


-- Now we check that we have a properly formatted parameter, or we set the sDesc for the roll with a message.
if not sParams:match("(%d+)d([%dF]*)%s*(.*)") then
Debug.console("sParams: ", sParams);
rRoll.sDesc = "Parameters not in correct format. Should be in the format of \"#d# <desc>\"";
return rRoll;
end

after the space characters class %s

All good now.

damned
July 7th, 2019, 10:29
Ho Ranzarok

I apologise - this slipped my mind.
The Crom roll is very simple - it is just /crom #d6 and it does some counting of dice faces.
It does work correctly as far as I have tested.

What exactly are you putting in your roll?

Ranzarok
July 7th, 2019, 20:11
No worries.

Pretty much what you said /crom #d6 that's it. It was returning s.Size = nil. I did get it working though by adding the repetition modifier * like I mentioned in the post above.