[Arianne-bugs] [ arianne-Bugs-1745967 ] 2 rings of life - none working ?!

[SourceForge.net <no...@so...>]

Bugs item #1745967, was opened at 2007-07-01 09:21
Message generated for change (Comment added) made by seather
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101111&aid=1745967&group_id=1111

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: World
Group: None
>Status: Closed
Resolution: None
Priority: 8
Private: No
Submitted By: KoSch (koschi)
>Assigned to: Seather (seather)
Summary: 2 rings of life - none working ?!

Initial Comment:
Hi dudes,
before i knew how the Ring of Life looks, i traded one - i thought. The description shows "Ring Of Life" but it seems to be the Skullring.

And i got another one (not looking like the original one):
After my wedding the engagement ring got the discription of "Ring of Life" too!

Now i got 2 rings of life - but none is working correctly. I still loose much XP after dieing.

Maybe some admin wanna trade this against Chaos-Helmet,  Chaos-Cloak and Chaos Legs?? I need everything in Chaos :) No -joking- i would trade it against the original ring of life. (at least the skull looking one)

Now i will improve my XP.
If you don't believe this RingofLife-Thing just find me in Stendhal - i'll show you.

CU
kosch

----------------------------------------------------------------------

>Comment By: Seather (seather)
Date: 2007-08-05 22:01

Message:
Logged In: YES 
user_id=1152155
Originator: NO

committed "Correctly handle situations where there is more than one
emerald ring, handle charges/amount greater than 1 correctly"

----------------------------------------------------------------------

Comment By: Seather (seather)
Date: 2007-08-01 01:43

Message:
Logged In: YES 
user_id=1152155
Originator: NO

I committed a quick fix for ring descriptions

	@Override
	public String describe() {
		if(has("amount")) {
			if(getInt("amount") == 0) {
				return "You see the ring of life. The gleam is lost from the stone and
it has no powers.";
			} else {
				return "You see the ring of life. Wear it, and you risk less from
death.";
			}
		}
		return super.describe();
	}


----------------------------------------------------------------------

Comment By: Seather (seather)
Date: 2007-07-04 06:05

Message:
Logged In: YES 
user_id=1152155
Originator: NO

If you really do have two "ring of life" there will be a problem; the
onDead code checks one ring at random(sort of, bag always gets checked
before finger) and won't check any other rings regardless of the "amount"
the checked ring had.

A creature drops this ring(at least since May 15).

(Subject: "amount") death code changes any non zero number to zero, I
expected new = old - 1;

----------------------------------------------------------------------

Comment By: Katie (kymara)
Date: 2007-07-01 13:40

Message:
Logged In: YES 
user_id=1560922
Originator: NO

>From Ring.java:

Can I leave you to fix please miguel?	
@Override

	public String describe() {

		if(has("amount") && getInt("amount")==0) {
			return "You see the ring of life. The gleam is lost from the stone and
it has no powers.";

		} else {

			return "You see the ring of life. Wear it, and you risk less from
death.";

		}

	}


----------------------------------------------------------------------

Comment By: Katie (kymara)
Date: 2007-07-01 13:38

Message:
Logged In: YES 
user_id=1560922
Originator: NO

Ok there must be something in some generic Ring java file that sets the
description. FYI, kosch, you get the ring of life from a quest and you need
to be pretty high level to complete it. It has a green gem



----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101111&aid=1745967&group_id=1111