Ultima VIII - Sorcery reagents of different sizes do not stack

[bliznik]

In the original Ultima VIII, there were sorcery reagents of different shapes/sizes. For example, a Demon Bone could look like a skull, a rib, or a pile of femurs. But when you dragged a Demon Bone on top of another Demon Bone, they would stack and adopt one of the two shapes/sizes.

In ScummVM, the sorcery reagents are of different shapes/sizes, but you can only stack sorcery reagents of the same shape/size. For example, you can only stack two Demon Bones that are shaped like a skull, or two Demon Bones that are shaped like a rib, but you cannot stack a Demon bone that is shaped like a skull on top of a Demon Bone shaped like a rib.

Not a fatal defect, but it definitely makes inventory management more difficult.

[mduggan]

This is an interesting problem, and I looked into it in detail a while back - but I'm 90% sure the current behavior is correct unless there's been a regression.

Annoyingly, only earth/necromancy reagents have the engine code to enable them to stack. The others don't have the code to even print the plural form in the game (unlike say, blackmoor which is either pile or piles depending on quantity). The fire/sorcery reagents have no code to print plurals in their look() function. I even have some commented out code that could enable it if it worked:
​https://github.com/scummvm/scummvm/blob/7919e9f0d025b634fb6bc9e477285f56524cca1f/engines/ultima/ultima8/world/item.cpp#L2361

[bliznik]

Huh, interesting. They can stack in the original when playing it using DOSBox. I've taken a quick video as an example. ​https://youtu.be/CEVpg_pIAn4

I'm unsure why there's no code for it. Maybe it's hidden in a completely separate area that is buried in the code? That's a shame...thanks for checking!

[bliznik]

You are correct that the usecode doesn't show how many there are. In DOSBox, if I single-click on a pile of 8 blackmoor, the GUI shows me "8 blackmoor" But if I single-click on a pile of 4 Daemon bones, the GUI just shows me "Daemon Bone"

[mduggan]

Oh cool, thanks for checking!

In that case I can just enable that code - I used to have it enabled but then it was not clear how many I was carrying so I assumed I was wrong for adding it - I guess I should have spent more time checking the original.

[mduggan]

Fixed in 9cdd15f9859d2635de3f528ebfa6b272f97cce5c.

[bliznik]

The latest fix merges Sorcery reagents, but does not stack them. Compare the behavior of the ScummVM behavior here (​https://youtu.be/NfgdjUSdzY0) against the DosBOX behavior here (​https://youtu.be/CEVpg_pIAn4)

One stacks 4 reagents, and the other merges 4 reagents. They should be stacked, not merged.

[mduggan]

Sorry for the slow response. There was a stupid bug in my change so the quantity did not increase, it should be fixed now in be7ce1b6deeef80a184f06591a7d8a9481436e25.