Support for Custom Tiers in EpicLoot?

I am working through the final touches on combining both EpicLoot and ItemPerks Legendary items, but noticed that your information overlay does not account for custom Tier values in EpicLoot. For instance, I have the following set for my EpicLoot config:

  "Tier information": {
    "Tier display names that show up in the menu": {
      "e": "Exquisite",
      "s": "Legendary",
      "a": "Epic",
      "b": "Rare",
      "c": "Uncommon"
    },
    "Tier colours": {
      "e": "#09FC05",
      "s": "#FFF900",
      "a": "#9500BA",
      "b": "#077E93",
      "c": "#AE5403"

However, your UI will only show the default values:

Any chance you can un-hard code those and make it read off of the config set?

TY for letting me know.
Actually, i already built it with this in mind and thought it was working. Ill look into it asap.

Changed Status from Pending to Work in Progress

Changed Fixed In to Next Version

have you added the tier configurations to the epic buffs itself?

3 hours ago, molokatan said:

have you added the tier configurations to the epic buffs itself?

If you mean via the config file yes - you can have any amount of tiers you want basically, then I just adjust the chances down as I add more



 

i added it now like you did + the chance for builders roll.

    "Builders": {
      "enabled": true,
      "item_blacklist": null,
      "item_whitelist": [
        "attire.hide.boots",
        "attire.hide.helterneck",
        "attire.hide.pants",
        "attire.hide.poncho",
        "attire.hide.vest",
        "hat.wolf",
        "bone.armor.suit",
        "boots.frog",
        "bucket.helmet",
        "burlap.gloves",
        "burlap.headwrap",
        "burlap.shirt",
        "burlap.shoes",
        "burlap.trousers",
        "coffeecan.helmet",
        "deer.skull.mask",
        "diving.fins",
        "diving.mask",
        "diving.tank",
        "diving.wetsuit",
        "hat.beenie",
        "hat.boonie",
        "hat.cap",
        "hat.dragonmask",
        "hat.miner",
        "hazmatsuit",
        "hazmatsuit.arcticsuit",
        "hazmatsuit.nomadsuit",
        "hazmatsuit.spacesuit",
        "heavy.plate.helmet",
        "heavy.plate.jacket",
        "heavy.plate.pants",
        "hoodie",
        "jacket",
        "jacket.snow",
        "mask.balaclava",
        "mask.bandana",
        "metal.facemask",
        "metal.plate.torso",
        "pants",
        "pants.shorts",
        "riot.helmet",
        "roadsign.jacket",
        "roadsign.kilt",
        "roadsign.gloves",
        "shirt.collared",
        "shoes.boots",
        "tactical.gloves",
        "tshirt",
        "tshirt.long",
        "wood.armor.helmet",
        "wood.armor.pants",
        "wood.armor.jacket"
      ],
      "tierInfo": {
        "e": {
          "chance_weight": 1,
          "min_value": 0.17,
          "max_value": 0.20,
          "required_crafting_perm": null
        },
        "s": {
          "chance_weight": 5,
          "min_value": 0.13,
          "max_value": 0.16,
          "required_crafting_perm": null
        },
        "a": {
          "chance_weight": 15,
          "min_value": 0.09,
          "max_value": 0.12,
          "required_crafting_perm": null
        },
        "b": {
          "chance_weight": 40,
          "min_value": 0.05,
          "max_value": 0.08,
          "required_crafting_perm": null
        },
        "c": {
          "chance_weight": 100,
          "min_value": 0.01,
          "max_value": 0.04,
          "required_crafting_perm": null
        }
      },

since the chance is quite low, you can barely read the value and bar on the chances block.

But.... its working fine.

Changed Status from Work in Progress to Not a Bug

Changed Fixed In from Next Version to 1.0.8

when you changed it, you have to reload also my plugin. Its reading the config from EpicLoot and takes over the chances etc from it.

@molokatan

I have had those values in my config for many months now - I just started using your plugin, and it does not work like it is for you. The only significant difference in my config is that I have a permission defined for "required_crafting_perm". Here is my full EpicLoot config for reference

 

EpicLoot.json

I will look into it later today. Maybe i made a mistake with that perm stuff or its reading it wrong.

Will let you know.

How it was planned with that permissions:

A player can only craft and see the stats if they got the permission assigned. So if a player dont has that permission, they should only see the 4 other stats.

For sure not the best option to hide it in the screen, but i was also not sure how it makes sense for players without the perm.

I thought best is to only show what a player can get for real.

Will still double check if everything is working like this or if we got a bug here.

I can confirm that is what is happening - I enabled the permission and it shows up. To your point, I think your intent to hide was correct - I just use it differently. I make the rarest Tiers non-craftable, that way they can only be looted (intentional). To that end, it would be less confusing for players if the bar remained unhidden, or captured in a separate "loot chance" section vs. "craft chance", considering they will be referencing this for all of their stats.

I did manage to alter the code to behave the way I described - if you want it for a reference just DM me and I will send it over for you to check out.

I can add a flag "show disabled tiers" or so. thats basically 3 lines of code and then its configurable.

Maybe even show the tier with a lock icon to indicate that it cannot be used by crafting.