Change how TOTAME=0 works?

Want to discuss changes to the UOX3 source code? Got a code-snippet you'd like to post? Anything related to coding/programming goes here!
Post Reply
User avatar
Xuri
Site Admin
Posts: 3704
Joined: Mon Jun 02, 2003 9:11 am
Location: Norway
Has thanked: 48 times
Been thanked: 8 times
Contact:

Change how TOTAME=0 works?

Post by Xuri »

According to both stratics and uo.com's bestiary, there are animals that require 0.0 taming skill to be able to tame. Cats, for instane. This is currently impossible in UOX3 as any creature with a TOTAME value of 0 is impossible to tame.

The question is - should we keep it that way, and set a minimum required TOTAME=1 taming those animals, or allow taming of creatures with TOTAME=0 and find a different way of disallowing taming something?
-= Ho Eyo He Hum =-
giwo
Developer
Posts: 1780
Joined: Fri Jun 18, 2004 4:17 pm
Location: California
Has thanked: 0
Been thanked: 0

Post by giwo »

Well, technically a player could have 0.0 taming skill and want to train by taming animals.... that being the case It would be a good idea to default TOTAME to, maybe 0xFFFF, and then anything with TOTAME=0 is tamable even at 0 skill.
Scott
User avatar
Xuri
Site Admin
Posts: 3704
Joined: Mon Jun 02, 2003 9:11 am
Location: Norway
Has thanked: 48 times
Been thanked: 8 times
Contact:

Post by Xuri »

Default it to 0xFFFF in the DFNs or in the source?
-= Ho Eyo He Hum =-
Grimson
Developer
Posts: 802
Joined: Sat Jun 04, 2005 1:52 am
Location: Germany
Has thanked: 0
Been thanked: 0

Post by Grimson »

Xuri wrote:Default it to 0xFFFF in the DFNs or in the source?
I guess in the source. And change the following lines in taming.js:

Code: Select all

		else if( skillToTame == 0 || skillToTame > 1000 )
			pSock.SysMessage( GetDictionaryEntry( 1593, pLanguage ) );
to:

Code: Select all

		else if( skillToTame > 1000 )
			pSock.SysMessage( GetDictionaryEntry( 1593, pLanguage ) );
Grimson
Developer
Posts: 802
Joined: Sat Jun 04, 2005 1:52 am
Location: Germany
Has thanked: 0
Been thanked: 0

Post by Grimson »

Grimson wrote: I guess in the source. And change the following lines in taming.js:

Code: Select all

		else if( skillToTame == 0 || skillToTame > 1000 )
			pSock.SysMessage( GetDictionaryEntry( 1593, pLanguage ) );
to:

Code: Select all

		else if( skillToTame > 1000 )
			pSock.SysMessage( GetDictionaryEntry( 1593, pLanguage ) );
Done it, tested it and will commit it to the CVS in a few moments.
User avatar
Xuri
Site Admin
Posts: 3704
Joined: Mon Jun 02, 2003 9:11 am
Location: Norway
Has thanked: 48 times
Been thanked: 8 times
Contact:

Post by Xuri »

Might want to set that to > 1500 or something like that, as some creatures require more than 100.0 taming to tame :P I've removed all TOTAME tags from non-tameable creatures already, so there should no longer be any non-tameable creatures with TOTAME tags above 1000.

Also, please default TOPROV and TOPEACE to 0xFFFF in the source as well.
-= Ho Eyo He Hum =-
Grimson
Developer
Posts: 802
Joined: Sat Jun 04, 2005 1:52 am
Location: Germany
Has thanked: 0
Been thanked: 0

Post by Grimson »

Xuri wrote:Might want to set that to > 1500 or something like that, as some creatures require more than 100.0 taming to tame :P I've removed all TOTAME tags from non-tameable creatures already, so there should no longer be any non-tameable creatures with TOTAME tags above 1000.
It defaults to 0x7FFF (maximum value for a SI16) which is 32767, should be high enough ;).
Also, please default TOPROV and TOPEACE to 0xFFFF in the source as well.
Ok.
Post Reply