Menu

#5166 greater potions request gets substituted with potions

closed-fixed
5
2011-04-15
2011-04-12
Anonymous
No

Hi folks,

after the update today, Feelgood won't sell me greater pots:

[00:30] <tuebix> buy 10 greater potion
[00:30] <Dr. Feelgood> 10 bottles of potion will cost 2000. Do you want to buy them?

best
tuebix

Discussion

  • Katie Russell

    Katie Russell - 2011-04-12

    00:50 <+postman-bot> tuebix asks for support to ADMIN: i cannot comment on my bug, since i forgot my sourceforge login data. Carmen has the same problem as
    feelgood. Selling assassin daggers (also 2 words) works however

     
  • Katie Russell

    Katie Russell - 2011-04-12
    • assigned_to: nobody --> martinfuchs
     
  • Katie Russell

    Katie Russell - 2011-04-12
    • summary: no greater pots for sale? --> greater potions request gets substituted with potions
     
  •  tuebix

    tuebix - 2011-04-13

    ok, i created a new account to save the support guys work ;-)

    i haven't found the precise location of the bug since it is 3am now, but the algorithm seems to be like this:

    [02:53] <tuebix> buy minor potion

    [02:53] <Carmen> A bottle of minor potion will cost 100. Do you want to buy it?

    [02:53] <tuebix> no

    [02:53] <Carmen> Ok, how else may I help you?

    [02:53] <tuebix> buy potion

    [02:53] <Carmen> A bottle of potion will cost 250. Do you want to buy it?

    [02:53] <tuebix> no

    [02:53] <Carmen> Ok, how else may I help you?

    [02:54] <tuebix> buy greater potion

    i wanna buy "greater potion":
    1) "greater potion" doesn't end with "minor potion"
    2) "greater potion" ends with "potion", the full name of the next bigger article
    3) there is a hit in point 2 and no further search is done.

    also works for things like "asdfae potion" and so on

     
  • Katie Russell

    Katie Russell - 2011-04-13

    Hi, thanks for your efforts in debugging this problem and for reporting it to us.

    I am pretty sure this is to do with the change listed in the change log as:

    - improvements in understanding item names when talking with merchandizers, e.g. "solid plate shields"

    To explain this change and the reasoning: The actual item name was plate shield, but the description had an adjective: 'solid', so players would try 'solid plate shield' and that was not recognized as an item. Likewise 'mighty broadsword'.

    Outcome here causing this bug: I think that the adjective 'greater' is being dropped, even though 'greater potion' is actually an item itself. 'assassin' is not an adjective, perhaps that is why 'assassin dagger' works even though 'dagger' is also an item. Likewise 'minor' is perhaps not listed as an adjective.

    I have assigned this bug to the developer who made that change but in the event he can't work on it then another team member can look into it.

    Thank you again.

     
  • Katie Russell

    Katie Russell - 2011-04-13
    • priority: 5 --> 7
     
  • Katie Russell

    Katie Russell - 2011-04-13

    OK, thanks for the error report.
    Problem is now fixed in CVS.

     
  • Katie Russell

    Katie Russell - 2011-04-13
    • assigned_to: martinfuchs --> kymara
    • status: open --> closed-fixed
     
  • Hendrik Brummermann

    • priority: 7 --> 5
     

Anonymous
Anonymous

Add attachments
Cancel