Skip to content

Add dynamic advanced repair menu#2

Draft
Ravenger2709 wants to merge 1 commit into
Bones50:masterfrom
Ravenger2709:codex/advanced-repair-dynamic-menu
Draft

Add dynamic advanced repair menu#2
Ravenger2709 wants to merge 1 commit into
Bones50:masterfrom
Ravenger2709:codex/advanced-repair-dynamic-menu

Conversation

@Ravenger2709

Copy link
Copy Markdown

Summary

This PR modernizes the Advanced Repair script while keeping the original Exile commanding-menu installation flow intact.

Changes included:

  • Adds a dynamic hitpoint-based repair menu instead of static car/heli action lists.
  • Groups actions into Salvage, Repair, and Replace submenus.
  • Adds nested wheel and glass submenus so vehicles with many wheels/windows stay readable.
  • Shows part condition as a percentage where 100% is healthy and 0% is fully broken or removed.
  • Adds quantity-aware repair recipes through _actionConfigs.
  • Adds anti-exploit salvage checks so removed/destroyed parts cannot be salvaged repeatedly.
  • Returns raw materials for heavily damaged salvaged parts instead of pristine replacement parts.
  • Keeps the original Bones_fnc_salvageAndRepairMenuCar and Bones_fnc_salvageAndRepairMenuHelo entrypoints as compatibility wrappers.
  • Updates README and install instructions, including infiSTAR allowed commanding menu entries.

Notes

The recipe defaults use standard Exile item classes only. Server owners using custom part items can edit _actionConfigs in custom\advancedRepair\Bones_fnc_salvageAndRepair.sqf.

Validation

  • Checked the staged diff with git diff --cached --check.
  • Checked for accidental custom/private item class references such as DDR_*.
  • Verified the install instructions still use the original car/heli function names.

I could not run an in-game Arma/Exile validation in this fork; the same dynamic menu approach was tested on a live Exile server before adapting it to standard Exile item defaults for this PR.

@Ravenger2709 Ravenger2709 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on my server

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