Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify UIArmory_Loadout so its inventory hook fires in all cases. #127

Closed
RealityMachina opened this issue Dec 3, 2017 · 0 comments
Closed

Comments

@RealityMachina
Copy link
Contributor

Currently, the X2DLCInfo hook in that script is only checked if the game hasn't already gave a vanilla reason to disable the weapon.

Since the added expansion to the inventory hook means there's a greater chance of more mods that wants to enable an item to be equipped by a unit that normally can't, (heck I'm already running into this problem and my mod isn't reliant on the added expansion) the UI end should be changed to account for this, since it's the only method that has this kind of restriction on firing.

RealityMachina added a commit to RealityMachina/X2WOTCCommunityHighlander that referenced this issue Dec 3, 2017
MalucoMarinero pushed a commit that referenced this issue Dec 18, 2017
#127

* Edited inventory hook to accomdate ItemState

* Added new hook so older mods won't crash games

* Updated XCGS_Unit with new hook

* Replaced function body with one call to CanAddItemToInventory_CH

* Modified XCGS_Unit inventory hook to use itemstate when available

Also added Helpers.uc

* Added X2StrategyGame_SimCombat just in case

* Edited UIArmory_Loadout for Issue #127
MalucoMarinero added a commit that referenced this issue Dec 18, 2017
MalucoMarinero pushed a commit that referenced this issue Dec 20, 2017
* Edited inventory hook to accomdate ItemState

* Added new hook so older mods won't crash games

* Updated XCGS_Unit with new hook

* Replaced function body with one call to CanAddItemToInventory_CH

* Modified XCGS_Unit inventory hook to use itemstate when available

Also added Helpers.uc

* Added X2StrategyGame_SimCombat just in case

* Edited UIArmory_Loadout for Issue #127

* Altered X2DLCInfo for inventory hook changes

* Additional fixes.

* Removed quote marks that weren't needed.

* Fixed X2StrategyGame_SimCombat hook not having GetMyTemplate()

* Fix: literals can't be left blank as an out variable

* X2StrategyGame_SimCombat should now be properly fixed
MalucoMarinero pushed a commit that referenced this issue Jan 23, 2018
* Edited inventory hook to accomdate ItemState

* Added new hook so older mods won't crash games

* Updated XCGS_Unit with new hook

* Replaced function body with one call to CanAddItemToInventory_CH

* Modified XCGS_Unit inventory hook to use itemstate when available

Also added Helpers.uc

* Added X2StrategyGame_SimCombat just in case

* Edited UIArmory_Loadout for Issue #127

* Cosmetic System Fixes: handles Issues #155, #154, and #150

Adds UIOptionsPCScreen and has it show all applicable sliders if possible.

Adds XGCharacterGenerator and fixed issue with limit being impossible to meet by part pack sliders.

Modifies XCGS_Unit in `AddItemToInventory`, to filter out part packs player doesn't want the game to use.
robojumper pushed a commit that referenced this issue Mar 14, 2018
* Edited inventory hook to accomdate ItemState

* Added new hook so older mods won't crash games

* Updated XCGS_Unit with new hook

* Replaced function body with one call to CanAddItemToInventory_CH

* Modified XCGS_Unit inventory hook to use itemstate when available

Also added Helpers.uc

* Added X2StrategyGame_SimCombat just in case

* Edited UIArmory_Loadout for Issue #127

* Additions for issues #160 and #183

* Reduced const a bit for performance reasons

It's at 100 because I have about 65 part packs the system can change. :v

* Changed method to rebuilding and clearing list when needed.

* Menu now auto-adjusts depending on items
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants