Bushido, Ninjitsu and Spellweaving

[Aeros]

Hi guys,

Thanks for these great additions to RC2. However, the following observations:

1) For bushido and ninjitsu, the books are empty. I then assumed you need scrolls to fill them, and proceed to go through InsideUO to find these scrolls. However, I found no instances of these scrolls. For both books, I have the following settings in the itemdescs (Do note, these books are ALL defined as Spellbook in my itemdescs):

For Bushido:

Code: Select all

    SpellType						Bushido

and for Ninjitsu:

Code: Select all

    SpellType						Ninjitsu

2) For the Spellweaving book, I can't drag any of its scrolls into the book. It keeps telling me the item is too heavy for the container.

The spelltype entry for the spellweaving book:

Code: Select all

    SpellType						Spellweaving

And the itemdesc entry for one of the scrolls:

Code: Select all

Item 0x2D51
{
    Name           arcanecirclescroll
    Desc           Arcane Circle scroll
    Script         spellweavingscroll
    CProp Spellid  i601
    VendorBuysFor  6
    VendorSellsFor 12
    Weight         1/10
}

Also tried the following as an itemdesc entry for a scroll:

Code: Select all

Item 0x2970
{
    Name           arcanecirclescroll
    Desc           Arcane Circle scroll
		Graphic				 0x2D51
    Script         spellweavingscroll
    CProp Spellid  i601
    VendorBuysFor  6
    VendorSellsFor 12
    Weight         1/10
}

0x2D51 being the InsideUO graphic number for the scroll, 0x2970 being my internal objtype for the scroll. Neither entry worked, still got the same error - object too heavy for the container.

I should also mention here that I used the exact same methods with the Chivalry and Necromancy books, and they worked there. So I assume something changed, or something may be wrong by how core controls the Bushido, Ninjitsu and Spellweaving books.

[MuadDib]

Unsure if you have looked into the 096 Distro any yet, so will post it here just so others can see it too.

Itemdesc.cfg entry for a spellbook/ninjitsu.etc using the new systems.

Code: Select all

Spellbook 0xEFA
{
	Name			Spellbook
	Desc			spellbook

	Gump			0xFFFF
	MinX			40
	MaxX			60
	MinY			20
	MaxY			80

	SpellType		Magic

	Newbie			1
	BlocksCastingIfInHand	0
	RequiresAttention	0
	
	VendorSellsFor		22
	VendorBuysFor		11

	Weight			1

	CanRemoveScript		spellBook/canRemove
}

Notice the way SpellID is, and SpellType. They are entries, not cprops, old container info, etc etc.

Also, for spell scroll's itemdesc, you MUST list spelltype there also, so the core knows WHICH book it is for. I would suggest making your system use SpellID as an entry also, instead of a cprop. You can see Distro 096 SVN for how we are doing magery there, for any extra help.

Also, bear in mind, the spell scroll IDs are hardcoded, via being the same objtype as in the UO files (using InsideUO to get their id).

Now, that info is for Spellweaving. Give me a bit on the Bushido and Ninjitsu, as it appears, while on OSI they are identical to the other books (Paladin, Necro, and magery), they come with the spells already inside them when you create a char, AND buy them (fully loaded with all the moves). This might require some hardcoding in the core since it uses the same packet stuff, but, they are spawned on OSI with all the stuff in them.

And yes, I too am not finding "scrolls" in the client for these 2 books, very very odd since it uses the same book system. But I will keep you posted on my progress.

[MuadDib]

Oh yeah, forgot, they don't have "scrolls", but they use the "icons" from the tiledata instead.

If you search for their names in InsideUO you will find them, right after the bushido book, and ninjitsu book, respectively. I have now added those objtype id's to the core for the next release. (RC3 or Re-publish of RC2 and in Final).

Remember, you MUST use these objtypes!! You CAN change the graphic iirc, to that of anything you wish (correct me if wrong anyone). I'll send you a test core to work with on testing this if you wish. No biggie for me.

[Aeros]

Redefined the scrolls, same problem. Let me list my entries for you.

The spellbook:

Code: Select all

Spellbook 0xB9FE
{
    Name                spellweavingbook
    desc                Book of Bushido
    Graphic          		0x2D50
    SpellType						Spellweaving
    Gump                0xFFFF
    MinX                40
    MaxX                60
    MinY                20
    MaxY                80
    Newbie              1
    BlocksCastingIfInHand 0
    RequiresAttention   0
    VendorSellsFor      22
    VendorBuysFor       11
    weight              1    
}

Then, my scroll entries are now:

Code: Select all

Item 0x2D51
{
    Name           arcanecirclescroll
    Desc           Arcane Circle scroll
    Script         spellweavingscroll
    SpellType			 Spellweaving
    SpellID			   601
    VendorBuysFor  6
    VendorSellsFor 12
    Weight         1/10
}

And yeah.. same error.

EDIT: I know the desc for the spellweaving book is wrong, btw. Noticed that now.

[MuadDib]

So is the graphic. no need. Make sure the Objtype is the same as the one given in InsideUO. Then, also no need to set the Graphic entry too

[Aeros]

Have I got bad news for you...

That didnt help either. Spellbook entry is:

Code: Select all

Spellbook 0x2D50
{
    Name                spellweavingbook
    desc                Book of Spellweaving
    SpellType						Spellweaving
    Gump                0xFFFF
    MinX                40
    MaxX                60
    MinY                20
    MaxY                80
    Newbie              1
    BlocksCastingIfInHand 0
    RequiresAttention   0
    VendorSellsFor      22
    VendorBuysFor       11
    weight              1    
}

Same error. Item is too heavy for this container.

[MuadDib]

That's cuz you need the next release for the stuff to fully work right

[MuadDib]

Just an update to the post. In a "test" core (with the updates) Aeros got it working perfectly. So in the final release of Core, it will have the fully working version of all 3. I also made some heavy notes in the Core-Changes.txt to make things easier in setting these up.

[Shinigami]

nice work...

Shinigami

[MuadDib]

Thx, at least you haven't shot me yet

[Pierce]

Before you shoot each other, RC2 works fine for me
*smiles*

[CWO]

How about having at least RC3 before they consider even hurting eachother...

[MuadDib]

lol