Skip to content

[armory] Use item stats directly from the API for correct catalyzed item stats - #11850

Open
seriallos wants to merge 1 commit into
midnightfrom
armory-redirected-stats
Open

[armory] Use item stats directly from the API for correct catalyzed item stats#11850
seriallos wants to merge 1 commit into
midnightfrom
armory-redirected-stats

Conversation

@seriallos

Copy link
Copy Markdown
Contributor

There's no indication from the API that a tier item was catalyzed from something else except for the actual stats on the item. No bonus ID or additional property in the JSON shows up. So the simplest way to get things working is just to accept all the stats from the Armory and push them using the stats= attribute on an item.

Example API JSON for the stats of a catalyzed feral druid chest below. I didn't try to match the AgiInt hybrid stat, just drops any stat not appropriate for the spec and keeps the main stat.

[
    {
        "type": {
            "type": "AGILITY",
            "name": "Agility"
        },
        "value": 148,
        "display": {
            "display_string": "+148 Agility",
            "color": {
                "r": 255,
                "g": 255,
                "b": 255,
                "a": 1
            }
        }
    },
    {
        "type": {
            "type": "INTELLECT",
            "name": "Intellect"
        },
        "value": 148,
        "is_negated": true,
        "display": {
            "display_string": "+148 Intellect",
            "color": {
                "r": 128,
                "g": 128,
                "b": 128,
                "a": 1
            }
        }
    },
    {
        "type": {
            "type": "STAMINA",
            "name": "Stamina"
        },
        "value": 2896,
        "display": {
            "display_string": "+2,896 Stamina",
            "color": {
                "r": 255,
                "g": 255,
                "b": 255,
                "a": 1
            }
        }
    },
    {
        "type": {
            "type": "CRIT_RATING",
            "name": "Critical Strike"
        },
        "value": 74,
        "is_equip_bonus": true,
        "display": {
            "display_string": "+74 Critical Strike",
            "color": {
                "r": 0,
                "g": 255,
                "b": 0,
                "a": 1
            }
        }
    },
    {
        "type": {
            "type": "MASTERY_RATING",
            "name": "Mastery"
        },
        "value": 106,
        "is_equip_bonus": true,
        "display": {
            "display_string": "+106 Mastery",
            "color": {
                "r": 0,
                "g": 255,
                "b": 0,
                "a": 1
            }
        }
    }
]

…tem stats

There's no indication from the API that a tier item was catalyzed from
something else except for the actual stats on the item. No bonus ID or
additional property in the JSON shows up. So the simplest way to get
things working is just to accept all the stats from the Armory and push
them using the stats= attribute on an item.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant