Thorondor wrote:
RE adverse status things that could adjust appearance; I quite like the stunned / confused ones on TE4 main game, and also a red warning when you're about to get eaten
Let's see, is stunned the wobbly-screen one or the spinning-stars one? Confused, I think, is the blurry-screen one. And… "eaten"?

Thorondor wrote:
Interesting: Selling un-id'd scrolls / potions etc is now allowed; the price is agreed (e.g. 15 gold) but the actual remuneration reflects the value of the sale (as if it had been ID'd previous to trying to sell it). The same doesn't seem to be true for ammo.
[sound F/X: source diving] Ah, right, it's recomputing the sale price after the object is identified during the selling process. That will be tricky to disentangle… ah, since I'm already overriding Store:doSell(), I can just have it pass the computed price through to Store;onSell(). Fixed in SVN.
Thorondor wrote:
Also, selling 1 un-ID'd ammo from a stack doesn't identify the rest of the stack in your inventory.
Hmm, presumably I can handle that by ID'ing the stack before peeling off the one to sell. I forget, does doing this fully *ID* the remainder of the unsold stack, or just ID it? The latter is what I've put in SVN; if it's supposed to be the former, that's easy enough to fix for the next release.
Thorondor wrote:
Now, there I'm out of my depth. I know very little about how the engine's auto-explore works; I'm just putting scaffolding around it that I copied from elsewhere. Maybe we need to have it check for 'block_run' rather than 'block_move', which would stop it running across lava unless you're immune to fire; I have no idea whether that's even possible, though.
Thorondor wrote:
This happened when I walked on ice created by an ice ball (that I'd cast over lava)
error = "Lua Error: /data/damage_types.lua:200: attempt to compare number with nil\
[sound F/X: source diving] Ah, typo in the temporary effect conferred by standing on ice; I'd copy/pasted it from the ones for lava, which do random damage, and forgot to adapt it for ice, which does fixed damage. Got nether mist wrong the same way. Fixed in SVN.
Thorondor wrote:
Outside of Bree, are there any other mayors etc that are currently activated? (it's not currently possible to test the other house quests as the other 'Mayors' aren't implemented yet)
Not yet; that's what I'm working on for the next release.
Thorondor wrote:
Are you able to implement an inventory re-stock for shops?
That's also what I'm working on for the next release.

Thorondor wrote:
Used a staff of disarm (unidentified) - Thorondor fails to disarm an unknown trap
[sound F/X: source diving] Oh, right, this is
why I added the 'force' flag to Trap:disarm(); I should probably use it, shouldn't I?

Fixed in SVN.
Thorondor wrote:
Read a scroll of light
error = "Lua Error: /mod/class/Player.lua:1349: /data/general/objects/scrolls.lua:262: attempt to index global 'DamageType' (a nil value)\
[…]
Activating Rilia caused an error
error = "Lua Error: /mod/class/Player.lua:1349: /data/general/objects/artifacts.lua:2567: attempt to index global 'DamageType' (a nil value)\
Yeah, these are the same bug as
before. It really is going to crop up in all sorts of annoying places; I guess this calls for an emergency bugfix release:
Release 0.2.9 "We're Gonna Be Meaner Now"As noted, this is mostly a bugfix release:
- Fixed a lot of object activations that weren't defining damage types properly.

- Fixed the tooltips for some talents related to Summoning and Possession. [Not that they do anything yet, but at least they won't stack dump anymore…]
- Small typo in the level-up dialog was screwing up the check for whether to show a skill; in particular, we need to show Geomancy for non-Geomancer Mage-related classes even they they can't learn it, because Fire/Water/Air/Earth all live under it.
- Tweak the messages for a couple timed effects that assumed the player was the target of the effect.
- Evil monsters are supposed to be in the "Enemies" faction — including that Spectral/Zombie/etc. version of something that used to be in a friendly faction.

- Fix the Copy Spell inventory dialog so that you can actually select an item to copy onto.
- Relatedly, properly describe spell-copyable items as "A spell (can be/has been) copied onto this item."
- Fix grid damage for standing on ice or in nether mist.
- Couple fixes for selling things to stores:
- When selling one item from a stack, ID the rest of the stack in inventory. This is mainly for selling ammo, to get the (+x,+y) bonuses.
- Bit of internal juggling to ensure that the un-ID'd price the store offered you is the same price it actually pays you after identifying what it just bought from you.
- The Disarm spell (and Staffs thereof) obviously shouldn't have to check against the player's disarming ability, nor should it give the player the usual trap-disarming experience.
We also reproduce T2's "There is a bright flash of light!" message for this case. - The usual raft of stupid typos that I really wish Lua did "compile-time" checking for, notably one in the Galadriel chat.
I did add one feature, though:
- As long as I was in the area of the Copy Spell talent, might as well hook it into the use-item dialog, as new action 'Copy spell onto item' for suitable items.
Now, as before, since this is mostly an emergency bugfix release, I'm pretty sure any bugs in it were already there and just haven't been found yet.

As always, here is a good place to report them.
And as always, the
.team file for the release is available from my site, along with the
Subversion repo and
WebSVN front-end thereto. And for those of you following along on SVN, the
0.2.9 release branch will get bug fixes for all the aforementioned inevitable bugs.
Feature work for the main release is proceeding slowly but surely; I should be close to a feature release, but I've said that before.
