Opened 3 years ago

Closed 3 years ago

#1891 closed bug (fixed (in master))

*Enchant* prompts twice

Reported by: nckmccnnll Owned by:
Milestone: 4.0 Keywords:
Cc:

Description

"*Enchant Weapon* scrolls are now prompting twice - it'll ask you to choose which weapon to enchant, apply the to hit bonus to that, then prompt you to choose again before applying the damage bonus, so you can use it on two different weapons if you want." - Nomad

Change History (3)

comment:1 Changed 3 years ago by nckmccnnll

  • Summary changed from *Emchant* prompts twice to *Enchant* prompts twice

comment:2 Changed 3 years ago by fizzix

The "easiest" route would be to add an additional enchant flag to handle *enchant*. But I'm not sure if this is really the solution that is most desirable.

A second option might be to pass a null object to each enchant call, and if an object has been selected, to pass that object to the next call. I'm not sure if this is cleaner though.

Also, it seems a message will prompt if either half of the enchantment fails, that is easier to fix though.

comment:3 Changed 3 years ago by nckmccnnll

  • Resolution set to fixed (in master)
  • Status changed from new to closed

Fixed in 4cb2aad

Note: See TracTickets for help on using tickets.