PigPen-divinity
Sorceror
Lokai;683333 said:I know this post is a little old, but this was never addressed, and I found this script, and checked, and you are correct. It does affect Gems and Scrolls. In LootPack.cs we find this line:
PHP:if ( item.Stackable ) item.Amount = m_Quantity.Roll();
Since Gems and Scrolls are stackable, when 1 gem or 1 scroll is dropped, the 90% is applied to it, and rounded down, causing it to result in a quantity of zero.
The fix would be to change the return value to return the Math.Ceiling result of the calculation, rather than the (int) "explicit cast" of the calculation.
I just came across this thread while contemplating how best to adjust gold drops on monsters. This script may not be a full economy solution but it certainly is a quick bandage fix if some farming is getting out of control.
And yes - this script does prohibit selling any Gems and Scrolls. That is a problem (not so much the gems but Scribes should be able to sell their wares.
Lokai's suggestion of "change the return value to return the Math.Ceiling result of the calculation, rather than the (int) "explicit cast" of the calculation" is beyond my coding abilities. The alternative for me looks like manually adjusting individual monster drops.
Has anyone solved this on their version of this script?