[Bug]: Importing Group Rolls throws an error

[Ainias]

Description

When importing an exported Roll with GroupRolls throws an error TypeError: value must be one of ResultGroup, RollResults, string, or number

Reproduceable with the following code:

DiceRoll.import(new DiceRoll("{1d20}").export());

Dice Roller version

5.4.1

Bundle type

ESM

Node version

v18.18.2

Browser name and version

Brave v1.60.114

[GreenImp]

Thanks for raising, I'm able to replicate.

For reference, this is because, when exported, the results is an array of objects like:

{
  "averageTotal": 10.5,
  "maxTotal": 20,
  "minTotal": 1,
  "notation": "{1d20}",
  "output": "{1d20}: {[5]} = 5",
  "rolls": [
    {
      "calculationValue": 5,
      "isRollGroup": true,
      "modifierFlags": "",
      "modifiers": [],
      "results": [
        {
          "calculationValue": 5,
          "isRollGroup": false,
          "modifierFlags": "",
          "modifiers": [],
          "results": [
            {
              "rolls": [
                {
                  "calculationValue": 5,
                  "initialValue": 5,
                  "modifierFlags": "",
                  "modifiers": [],
                  "type": "result",
                  "useInTotal": true,
                  "value": 5
                }
              ],
              "type": "roll-results",
              "value": 5
            }
          ],
          "type": "result-group",
          "useInTotal": true,
          "value": 5
        }
      ],
      "type": "result-group",
      "useInTotal": true,
      "value": 5
    }
  ],
  "total": 5,
  "type": "dice-roll"
}

But by the time this is passed down to the ResultGroup.addResult() method, it expects the data to already be cast to a ResultGroup or RollResults object.

This will need to be changed to handle standard objects in this format as well.