Merge remote-tracking branch 'upstream/master' into SquidHack

Author: realSquidCoder

changelog.txt

@@ -36,6 +36,19 @@ Template for new versions:
 
 ## Removed
 
+# 52.03-r1
+
+## New Tools
+
+## New Features
+
+## Fixes
+- `make-legendary`: ``make-legendary all`` will no longer corrupt souls
+
+## Misc Improvements
+
+## Removed
+
 # 52.02-r2
 
 ## New Tools

make-legendary.lua

@@ -7,9 +7,11 @@ function getName(unit)
 end
 
 function legendize(unit, skill_idx)
-    utils.insert_or_update(unit.status.current_soul.skills,
-        {new=true, id=skill_idx, rating=df.skill_rating.Legendary5},
-        'id')
+    if skill_idx >= 0 and skill_idx <= df.job_skill._last_item then
+        utils.insert_or_update(unit.status.current_soul.skills,
+            {new=true, id=skill_idx, rating=df.skill_rating.Legendary5},
+            'id')
+    end
 end
 
 function make_legendary(skillname)
@@ -50,7 +52,9 @@ function BreathOfArmok()
         return
     end
     for i in ipairs(df.job_skill) do
-        legendize(unit, i)
+        if i >= 0 then
+            legendize(unit, i)
+        end
     end
     print('The breath of Armok has engulfed ' .. getName(unit))
 end