From 390801f53c4accea6bf1d944933d0c52d0deba16 Mon Sep 17 00:00:00 2001
From: Sarjuuk <junkdump@gmx.net>
Date: Tue, 25 Mar 2025 17:30:33 +0100
Subject: [PATCH] NpcDetailPage/Skinning Skill  * fix displayed skill
 requirement for non-standard skinning (engineering, herbalism)

---
 includes/game.php | 10 ++++++----
 pages/npc.php     |  2 +-
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/includes/game.php b/includes/game.php
index 1418d67f..3f5179be 100644
--- a/includes/game.php
+++ b/includes/game.php
@@ -468,6 +468,7 @@ class Game
         switch ($skillId)
         {
             case SKILL_SKINNING:
+                $reqLevel /= 5;                             // we pass creature level * 5 (so, skill value), but formula depends on actual creature level
                 if ($reqLevel < 10)
                     $reqLevel = 0;
                 else if ($reqLevel < 20)
@@ -479,15 +480,16 @@ class Game
             case SKILL_JEWELCRAFTING:
             case SKILL_INSCRIPTION:
             case SKILL_MINING:
-                $points = [$reqLevel];                              // red/orange
+            case SKILL_ENGINEERING:
+                $points = [$reqLevel];                      // red/orange
 
-                if ($reqLevel + 25 <= MAX_SKILL)                    // orange/yellow
+                if ($reqLevel + 25 <= MAX_SKILL)            // orange/yellow
                     $points[] = $reqLevel + 25;
 
-                if ($reqLevel + 50 <= MAX_SKILL)                    // yellow/green
+                if ($reqLevel + 50 <= MAX_SKILL)            // yellow/green
                     $points[] = $reqLevel + 50;
 
-                if ($reqLevel + 100 <= MAX_SKILL)                   // green/grey
+                if ($reqLevel + 100 <= MAX_SKILL)           // green/grey
                     $points[] = $reqLevel + 100;
 
                 return $points;
diff --git a/pages/npc.php b/pages/npc.php
index 87547c0e..1d523650 100644
--- a/pages/npc.php
+++ b/pages/npc.php
@@ -744,7 +744,7 @@ class NpcPage extends GenericPage
                 if ($note)
                     $tabData['note'] = $note;
                 else if ($lootTpl == LOOT_SKINNING)
-                    $tabData['note'] = '<b>'.Lang::formatSkillBreakpoints(Game::getBreakpointsForSkill($skinTab[2], $this->subject->getField('maxLevel')), Lang::FMT_HTML).'</b>';
+                    $tabData['note'] = '<b>'.Lang::formatSkillBreakpoints(Game::getBreakpointsForSkill($skinTab[2], $this->subject->getField('maxLevel') * 5), Lang::FMT_HTML).'</b>';
 
                 if ($hiddenCols)
                     $tabData['hiddenCols'] = $hiddenCols;