Page 1 of 2

spell icons in maintained window

Posted: Sat Feb 28, 2009 12:40 pm
by Zcoretri
Spell icons in maintained window no longer display, you just get a colored background.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 5:58 pm
by LethalEncounter
Yah I noticed that. I'll look into it.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 7:56 pm
by Zcoretri
I coded it how it was before and it worked. Was there a reason behind why you created new functions?

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 7:57 pm
by LethalEncounter
Huh? Did I change something? I was going to work on getting buffs working but then got sidetracked, but I don't think I changed anything.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:03 pm
by Zcoretri
Maybe those functions were already there, but here is what I am talking about...

Code: Select all

	//info->GetInfo()->maintained_effects[free_spell_maintained_slot].icon = (spell->GetSpellIcon()*-1)-1;
	info->GetInfo()->maintained_effects[free_spell_maintained_slot].icon = spell->GetSpellData()->icon;
	//info->GetInfo()->maintained_effects[free_spell_maintained_slot].icon_type = spell->GetSpellIconType();
	info->GetInfo()->maintained_effects[free_spell_maintained_slot].icon_type = spell->GetSpellData()->icontype;
commented out lines are what looked like were changed.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:05 pm
by LethalEncounter
Feel free to grab an earlier version of the repository and test to see if it works, but I'm fairly certain it was broken before I made any changes.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:07 pm
by John Adams
I remember debating the icons that showed up in Maintained before. If I cast a DoT on a mob, I would get a Maintained icon for that spell in my window - indefinitely. Zcoretri showed me that if my duration expired AFTER the mob died, the icon remained. But if before, the maintained icon would fade as expected.

Not sure if this helps any in remembering how it used to be.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:24 pm
by Zcoretri
LethalEncounter wrote:Feel free to grab an earlier version of the repository and test to see if it works, but I'm fairly certain it was broken before I made any changes.
Excuse my ignorance here, but how do you grab an earlier version?

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:27 pm
by LethalEncounter
I haven't done so it quite some time, but I think the process (if you are using tortoisesvn) is update to revision and then give the revision number. I think it will also show a log of the dates for each revision as well.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:33 pm
by Zcoretri
AH, that's what that command does :D

Thank LE

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:49 pm
by John Adams
Also using TortoiseSVN, you can highlight the exact file you wish to compare, and do a Show Log, and it will only show you that files changes. You can then highlight 2 versions and right-click Compare and see a nice diff of the two files complete with highlighted differences.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 8:58 pm
by Zcoretri
Yes I'm using TortoiseSVN...this sub version thing is pretty slick :)

OK, updated to rev 633 because 634 is where it was changed. Compiled and ran server, maintained window DOES show the icon correctly.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 9:03 pm
by LethalEncounter
OK, just do a diff and change it back then. It isn't a big deal though, because even if the icon does show correctly buffs need a ton of work and I'm going to be working on them soon.

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 9:08 pm
by Zcoretri
OK, just do a diff and change it back then. It isn't a big deal though, because even if the icon does show correctly buffs need a ton of work and I'm going to be working on them soon.
Not soon enough...been wanting to do buffs for awhile now :mrgreen:

OK will do a diff change

Re: spell icons in maintained window

Posted: Sat Feb 28, 2009 9:15 pm
by John Adams
Zcoretri wrote:Not soon enough...been wanting to do buffs for awhile now :mrgreen:
Ahh, finally others are stepping up to my level of dev abuse!

My work here is done.