Secrets of Faydwer - Bug Tracking

[Secrets]

Originally Posted by Secrets

It's probably the disc type, since all of the spells on KMRA were hand-made and it would seem in SoF some of them are hardcoded to have a specific disc re-use as out of combat only. I'll try and get the number on the one that doesn't work, but some discs do work, some don't... i'm willing to bet it's spell file dependant.

Just an update on this, for reference:

I ended up importing the spells, exporting them, and some spells were working because the number of fields was correct for titanium. Some of the older spells Raid Addicts has did *not* have all the required fields for titanium, however the titanium client still read the spell. When I converted this over to SoF with the import/export technique, those spells got corrupted. I ended up taking my broken file and opening it up in Ailia's Spell Editor, and then save it without doing any changes, and that added the missing fields.

It's a good thing to know, because if I didn't have those fields, they would have got parsed incorrectly into the DB (which they did initially) and then broken in the end SoF client. So, I imported and exported again after saving the spell file again, and this time it made a very nice Secrets of Faydwer compatable spell file.

Just wanted to post my progress on this for reference.

[Shendare]

I'm sure this is a low priority, but I notice that with the SoF client and rev488 codebase, the npc_types fields luclin_hairstyle, luclin_haircolor, luclin_beard, and luclin_beardcolor are not being recognized.

All Luclin-model playable-race NPCs display the defaults for these values, regardless of the database's settings.

The Titanium client appears to be able to work with the aforementioned fields' values, and both clients are able to acknowledge face, luclin_eyecolor, and luclin_eyecolor2.

[trevius]

Thanks for the report, Shendare, but that is already a known issue as posted here:

http://www.eqemulator.net/forums/showthread.php?t=27429

Cosmetic Work:
1. Spawn Structure - The spawn structure could use a few more fields to be identified (hair/beard color, hair, other facial features, sneak, and invis).

That post is updated regularly, so please make sure to check there as well as the first page in this thread before posting new bug reports.

[Shendare]

I totally missed that while looking over the list. Sorry about that! Feel free to delete my post!

[warhawk]

Hi

I've noticed a bug with the potion belt using the SOF client. If i have a stack of 6 potions it shows 36 in the potion belt icon. Also ,when clicked it gives the error message Error: item not found in inventory slot 272..

This is on PEQ using version 491

[trevius]

Sounds like we need to set a slot encode for the potion belt as well. The whole slot change thing from Titanium to SoF is probably one of the biggest pains of anything lol. I will add that to the list of bugs and try to take a look at it later tonight. Thanks for the report.

Edit: After checking on this a bit, it seems that the slot problem is caused by OP_CastSpell. Looks like we just need to set an encode to convert all of the inventory slots (including inside bags) from Titanium to SoF, since almost all slot numbers changed. This should be a fairly simple change. I am not sure what else is going to need to be done to allow potions to be cast while inside bags as that might be a restriction by castspell itself. If so, I am sure we can correct it, but it will probably mean doing some type of check for if the item is a potion or not, because normal clicky items shouldn't be clickable from inside bags.

[kimura0715]

not sure if its reported yet, but on my server, if using SoF, some of the augs are invisible....if u do link all on the mob's corpse it will show an aug, but its invisible...i can even loot it and put it in a bag, but just cant see it