WowAce

I was able to fix this to not showing missing enchants for MoP helms and MoP characters that are not Enchanters. This required a simple version string check, and to simply check the characters professions.

I also fixed the player doll frame average item level display to include upgrades, but I cannot fix the inspect doll frame average item level. There is a bug with created item links where they don't include any upgrade info, and this is what is needed to manually parse upgrade levels. There is a player only "GetAverageItemLevel" API that I used to fix the character screen.

in addon.lua

add a new variable at the top, around line 5

> `local isMoP = WOW_PROJECT_ID == WOW_PROJECT_MISTS_CLASSIC`

then find this line lower in the file

> ` local enchantable = isClassic and {`

and just under it, change `INVTYPE_HEAD = true` to be `INVTYPE_HEAD = not isMoP`

In reply to siegester03: you are god

In reply to mythct: I lost these changes after an auto update, so I posted them to github! https://github.com/siegester03/wow-simpleitemlevel-mop

In reply to Kemayo: It is, thanks! I will delete my fork of MoP hacks!

In reply to Kemayo: On initial load for a character, it still shows rings as enchantable. Once you reload UI is properly hides the red "E". This is due to the "GetProfessions()" call returning nil,nil on initial load. I think your code is running before all the player data has loaded

The new patch for Mists Classic added item upgrades. Will it be possible to update this add-on to accommodate for upgraded items when calculating overall item level?

In reply to kriminy: If you want, you can go to the addon folder, and look for the file: addon.lua

search for:

```

AddAverageLevelToFontString("player", self.avglevel)

```

replace the above with:

```

-- AddAverageLevelToFontString("player", self.avglevel)

local avgItemLevel, avgItemLevelEquipped, _ = GetAverageItemLevel()

self.avglevel:SetFormattedText("Item Level %.2f", avgItemLevelEquipped)

```

This will get you the upgraded item level, and also get it to 2 decimals

In reply to Popsickelz: That's line 482 for me. I implemented the changes. I will test them later. Thank you!

These changes work properly for me!

In reply to Popsickelz: And is there a way to do the same for an inspected player? Right now if I inspect someone the item lvl displayed doesn't take into account upgrades.

did this used to show gems on the character sheet? MY gems havent shown up for a few days and i cant figure out which addon updated that did

Is there a way to not show "E" for helm enchant (doesn't exist in MOP) and rings when you are not an enchanter?

In reply to TearinItUpSon: Second this, also if it's possible to detect that the character does not have Enchanting as a profession, could it not show the "E" for ring enchants?

In Mists of Pandaria Classic, my chat log is getting spammed with "fell back to itemid 1048590" every 0.5 to 1 sec when the trinket fly-out menu is open.

https://i.imgur.com/PQ8HKbc.png

There are no LUA errors when this event occurs.

In reply to Kemayo: Thank you so much!

Unusable. can only enable when needed otherwise it randomly but eventually crashes the game (MoP Classic).