EQEmulator Forums - Clicky items recast timer bug and fix

EQEmulator Forums (https://www.eqemulator.org/forums/index.php)

- Development::Bug Reports (https://www.eqemulator.org/forums/forumdisplay.php?f=591)

- - Clicky items recast timer bug and fix (https://www.eqemulator.org/forums/showthread.php?t=22532)

Clicky items recast timer bug and fix

I noticed that items with clicky effects (in my case and most noticeably, the shadowknight epic) ignored the recastdelay field in the items database, so that it was possible to re-click such items incessantly. I worked up a quick fix to this. I don't 100% like it because if the clicky is cancelled (e.g. by ducking during a long cast) the re-use timer is still set, but I can't work out how to access the m_inv variable (thus accessing the inventory and then the original item to get the recast delay) later on once the spell cast is finished - any ideas appreciated. Anyway for now this might be useful:

Code:

--- client_packet.cpp.orig      2007-03-23 15:33:40.000000000 +0000

+++ client_packet.cpp   2007-03-23 15:31:27.000000000 +0000

@@ -2935,6 +2935,16 @@

                        safe_delete(outapp);

                        if ((item->Click.Type == ET_ClickEffect) || (item->Click.Type == ET_Expendable) || (item->Click.Type == ET_EquipClick) || (item->Click.Type == ET_ClickEffect2))

                        {

+                               // here is where we can check recast times

+                               if (item->RecastDelay > 0) {

+                                       // check to see if it's in progress

+                                       if(!CastToClient()->GetPTimers().Expired(&database, pTimerSpellStart + item->Click.Effect, false)) {

+                                               InterruptSpell(item->Click.Effect);

+                                               Message(13,"Item recast time not yet met, %d seconds left.",CastToClient()->GetPTimers().GetRemainingTime(pTimerSpellStart + item->Click.Effect));

+                                               return;

+                                       }

+                                       CastToClient()->GetPTimers().Start(pTimerSpellStart + item->Click.Effect, item->RecastDelay);

+                               }

                                CastSpell(item->Click.Effect, castspell->target_id, castspell->slot, item->CastTime, 0, 0, castspell->inventoryslot);

                        }

                        else

Cheers

Paul.

Does anyone read this forum? I'm a bit disheartened by the "1 views" tag that I'm seeing :( Should I send code contributions somewhere else?

Paul.

I read it.. in fact I'm pretty sure I wrote out a reply at one point not sure why I never posted it.. anyway. I had it fixed after initially reading this, tho I do it a bit differently, actually checking for the timer type on the item instead of spell timer.

Thanks :)

Having looked at the forums a bit more, I have a suspicion that the forum software is putting the number of posts in the thread in where it should be showing the number of views, which is a bit strange!

Cheers

Paul.

I am new to all of this and I was wanting to enable the timers on the items, I am confused as to where this code is placed. is it in the world.exe or zone.exe or do I just have to creat another file and place it in a directory somewhere.

I am hoping that it is the create a file and place in a directory fix :)

You can't fix it unless you know how to compile and such, wait for the next build that has the fix.

I am sorry to bump this thread, but I looked at log files and cannot seem to find any mention about recast issue been fixed (unless you simply forgot to add it to the log file)

another thing, if you could tell what is the diffirence bitween diffirent recast types? I seen types 1, 6 and 8 on most items, but have no idea what importance they have.

thanks =)

This is still happening with my custom item. I guess the fix never made it in?

Probably best to make a new post with details and questions. Bumping posts from 8 years ago isn't all that great an idea.