It doesn't work for me. I use 1.5.0.12 version of the game (the newest for now). Tried to remove all another mods (except hook), but still nothing. And tried both attached files. :(
I really love this mod, it worked for about 10 minutes when i first downloaded it. Now whenever the mod is active it just straight up doesnt let me open the inventory or roster screen. without the mod on the same savefile it lets me open up the inventory screen.
Excellent mod! Love the version with the Blacksmith follower; it makes perfect sense they would be able to remove upgrades. Honestly kind of surprised the developers did not include this with the base game.
The issue with Quicker should be fixed with version 1.3 of that mod - it was eating keypresses in certain states (like the character screen). Tested and confirmed it works with this mod. Still a little wonky with the keypress, you have be pretty deliberate with Alt+D as other comments here mention, but that's back to default behavior for this mod.
I'm adding my hat in the ring to say the mod is not working for me. Got the latest mod hooks installed as well as this mod. Don't have Quicker mod installed. But neither the replacing attachment nor pressing alt-D does anything and I end up loosing the previous attachment I had on the armour.
Alt-D doesn't work, but returning to the inventory when changing to a new attachment does. Found a reason: installed "Quicker" mod (JCSato) prevents Alt-D from working.
Hmm...removing Quicker mod only stopped the player selection from shifting to the right like other people have mentioned on this page, but did not allow the removal of the upgrade.
Does ALT+D hotkey still work? It goes to the character on the right when I stop pressing D (read the comment chain below, didn't work for me). I tried setting it to ALT-S in .nut file but still nothing happens. Replacing armor attachments works.
Yes, it works. I just tested it. Either it's not being pressed correctly, or there's a conflict with another mod. The way it should be pushed (IIRC): Press Alt, press D, release D, release Alt
I adore this idea, but there seems to be an exploit that allows for free armor repairs. Was this intended?
Here are some screenshots. I can equip Metal Pauldrons (+40 durability) to a Basic Mail Shirt (115) that's missing 29 points of durability, then unequip the attachment with Alt+D and the armor will be fully repaired. I also tested this with the Leather Neckguard (+10 durability) and it's repairing 10 points each time I attach and remove it.
I do have several other mods installed, though the only one that modifies the inventory is Boyski's Reworked Inventory Stats.
Can you perchance do a test for me? Without the mod installed, can you take your damaged armor, equip metal pauldrons, and then equip the leather neckguard (destroying the metal pauldrons in the process). If that does not result in any repair, then I'll see about fixing it. (I tried testing it with bbedit, but it seems to not work with armor attachments...)
Of course it's less exploitable in the base game since you'd have to burn an armor attachment to get the repair, and those are limited. I don't have a lot of free time - very little actually - but I'll see if it's something I can do quickly.
Could you please add a line of code to update sprites when you remove attachments? Currently, after detachment, the image shows as if armor still has the attachment.
I thought I already did. Is this the same thing "nicholasgrimm" reported a while back (see below)? If it's a different case, let me know how to reproduce it.
There are two brother's portraits: one is at the upper left corner, the other is in the formation line of the character screen. I thought that if you add `armor.updateAppearance();` in method `detachArmorUpgrade`, it will do the trick. I tested it and it still does not updates the portraits. Needs more digging.
OK, I fixed that by adding the following lines: loadData(); // refresh the brother and stash displays (this is your line) armor.onUnequip(); armor.onEquip();
Hmm. I already have those lines, but in a different order:
armor.onUnequip(); // the armor won't update it's appearance in the roster unless we re-equip armor.onEquip(); loadData(); // refresh the brother and stash displays
It sounds like you're saying that you don't see those lines in the code? Or, is it that the order should be changed? Or are you talking about a different place?
I downloaded the latest version (the file This variant is always enabled.) I do not see the above mentioned lines in the code. Also the mod page shows version 3, but the latest file I downloaded has version 2.
Give Feedback
52 comments
Found a reason: installed "Quicker" mod (JCSato) prevents Alt-D from working.
Here are some screenshots. I can equip Metal Pauldrons (+40 durability) to a Basic Mail Shirt (115) that's missing 29 points of durability, then unequip the attachment with Alt+D and the armor will be fully repaired.
I also tested this with the Leather Neckguard (+10 durability) and it's repairing 10 points each time I attach and remove it.
I do have several other mods installed, though the only one that modifies the inventory is Boyski's Reworked Inventory Stats.
Here are the screenshots. The armor goes from 86/115 durability to 126/155, then back down to 125/125.
Either way, thanks for the attention, Adam :)
Currently, after detachment, the image shows as if armor still has the attachment.
OK, I fixed that by adding the following lines:
loadData(); // refresh the brother and stash displays (this is your line)
armor.onUnequip();
armor.onEquip();
Two lines instead of one.
armor.onUnequip(); // the armor won't update it's appearance in the roster unless we re-equip
armor.onEquip();
loadData(); // refresh the brother and stash displays
It sounds like you're saying that you don't see those lines in the code? Or, is it that the order should be changed? Or are you talking about a different place?