Some fixes for Craft System

[Lord_Greywolf]

Well most people know you can require Mana for crafting, but it does not show the requirements
(it will say if you need mana only if it also requires a scroll, etc - not fully usefull)

but most did not know - you can also require Stam or Hit Points as well
but will not show that it is required, unless you do not have enough and fail, little late lol

well these couple of mods will allow you to see that they are required or not, and how much

I do not suggest you require all 3, only because there is only 4 spots for lines to be shown in there, and would also include the blank scroll part and or if material changes hue, makers mark, etc etc

over 4 lines and they are "lost"

anyways - 1 place to add to and 1 place to modify in GraftGumpitem.cs file
(it is in the engines/craft/core directory)

Add in:
find this:

			if ( craftItem.UseAllRes )
				AddHtmlLocalized( 170, 302 + (m_OtherCount++ * 20), 310, 18, 1048176, LabelColor, false, false );

and right below it add in this:

			if ( craftItem.Mana > 0 )
			{
				string manastr = "Requires Mana: " +  Convert.ToString(craftItem.Mana);
				AddHtmlLocalized( 170, 302 + (m_OtherCount++ * 20), 310, 18, 1042971, manastr, LabelColor, false, false );
			}

			if ( craftItem.Stam > 0 )
			{
				string stamstr = "Requires Stamina: " +  Convert.ToString(craftItem.Stam);
				AddHtmlLocalized( 170, 302 + (m_OtherCount++ * 20), 310, 18, 1042971, stamstr, LabelColor, false, false );
			}

			if ( craftItem.Hits > 0 )
			{
				string hitsstr = "Requires Hit Points: " +  Convert.ToString(craftItem.Hits);
				AddHtmlLocalized( 170, 302 + (m_OtherCount++ * 20), 310, 18, 1042971, hitsstr, LabelColor, false, false );
			}

then for the mod:

find:

			if ( cropScroll )
				AddHtmlLocalized( 170, 302 + (m_OtherCount++ * 20), 360, 18, 1044379, LabelColor, false, false );

and replace it with this:

			if ( cropScroll )
			{
				string scrollstr = "Requires a Blank Scroll:";
				AddHtmlLocalized( 170, 302 + (m_OtherCount++ * 20), 310, 18, 1042971, scrollstr, LabelColor, false, false );
			}

and restart the server

and to use them, incase you did not know:

make sure you have it with an index = AddCraft( typeof( for the begining of the craft
and then just add one of these:
SetManaReq( index, 100 );
SetStamReq( index, 100 );
SetHitsReq( index, 100 );
just replace the 100 with the mana required

after the addskill or addres, etc part of it, but before the next item

and here is what it looks like then in the inscription area and on a new craft with all 3 being used

[Greystar]

Why did you change

1044379 // Inscribing scrolls also requires a blank scroll and mana.

to

1042971 //~1_NOTHING~

I get the other change but why add the

string scrollstr = "Requires a Blank Scroll:";

when you don't need it.. Or was that because you split off the showing of required mana, so you changed it to show something else?

Just want to know why I'm changing something.

[Lord_Greywolf]

if you do not change it it then shows:

requires mana 8
this also requires mana and a blank scroll

so it shows it requires mana twice then
you do not have to do the mod, but then like i said - it repeats itself

and the ~1nothing~ just means - what ever you put in for it to say - it says - nothing is added to it
so it works for this situation perfectly

[Greystar]

Originally Posted by Lord_Greywolf

if you do not change it it then shows:

requires mana 8
this also requires mana and a blank scroll

so it shows it requires mana twice then
you do not have to do the mod, but then like i said - it repeats itself

and the ~1nothing~ just means - what ever you put in for it to say - it says - nothing is added to it
so it works for this situation perfectly

I know how the clilocs work I was just asking, thanks for clarifying!

Thanks for the mod. I was just thinking about adding a couple of craft items that require hits to craft too.

PS) have some Karma++