Page 1 of 1

[b31] cursed Alchemist agate

Posted: Thu Jul 28, 2011 8:57 am
by lukep
Cursed alchemist agate still happens.

Re: [b31] cursed Alchemist agate

Posted: Thu Jul 28, 2011 8:24 pm
by lukep
I think that I found the bug. Cursed Touch has this restriction:

Code: Select all

if item.type == "ammo" or item.type == "gem" then return end
where it should have

Code: Select all

if item.type == "ammo" or item.type == "alchemist-gem" then return end

Re: [b31] cursed Alchemist agate

Posted: Mon Sep 12, 2011 6:47 pm
by lukep
bump, still present in b33.

Re: [b31] cursed Alchemist agate

Posted: Mon Sep 12, 2011 8:40 pm
by Hedrachi
alchemist gems are ammo iirc.. if ammo can be cursed, alchemist gems can be cursed, because some of the annoying ***** elites with more than one class can have alchy and cursed/doomed on the same class.

Re: [b31] cursed Alchemist agate

Posted: Mon Sep 12, 2011 11:10 pm
by lukep
Other types of ammo can't be cursed, it is just alchemist gems. This allows an extra boost for (non-ranged) Cursed classes from the extra curse slot.

Also, that code snippet is lacking context here, it is for what isn't affected by Fateful Aura.

Re: [b31] cursed Alchemist agate

Posted: Tue Sep 13, 2011 12:24 am
by Aquillion
Wait, don't regular gems still have to be forbidden? Or are they caught elsewhere?

Shouldn't it be this?

Code: Select all

if item.type == "ammo" or item.type == "alchemist-gem" or item.type == "gem" then return end

Re: [b31] cursed Alchemist agate

Posted: Tue Sep 13, 2011 12:34 am
by lukep
I believe it is caught in the previous line. I edited my beta 31, and it seemed to work perfectly when I did it. Here is a more complete look at the code.

Code: Select all

	curseItem = function(self, t, item)
		if item.cursed_touch then return end
		if item.unique then return end
		if item.quest then return end
		if not item:wornInven() then return end
		if item.type == "ammo" or item.type == "gem" then return end

Re: [b31] cursed Alchemist agate

Posted: Tue Sep 13, 2011 8:54 am
by darkgod
fixed