Viewing Issue Simple Details Jump to Notes ] Wiki ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0007171 [DCSS] Patches minor N/A 2013-06-07 02:40 2013-12-11 09:32
Reporter Medar View Status public  
Assigned To neil
Priority normal Resolution done  
Status closed   Product Branch 0.13 ancient branch
Summary 0007171: Allow potions/cards of experience train restricted skills
Description Allow potions/cards of experience train restricted skills. These are magical items and shouldn't require you to have the right type of item in your inventory.

Potentially invocations and/or spell schools should still be restricted.

Interface clutter is also an issue, but I think it's bearable.
Additional Information
Tags patch
Attached Files ? file icon 0001-Allow-potions-cards-of-exp-train-restricted-skills.patch [^] (2,159 bytes) 2013-06-07 02:40 [Show Content]
? file icon exp-hide-untrainable.diff [^] (2,043 bytes) 2013-09-12 22:00 [Show Content]

- Relationships

-  Notes
(0023089)
neil (administrator)
2013-06-07 03:17

In trunk (0.13-a0-1481-g48b0c40), thanks!
(0024006)
neil (administrator)
2013-09-12 21:59

elliptic has suggested hiding the untrainable skills by default, so that (for example) Nemelexites don't select Invocations under the mistaken impression that it is useful. * would display and allow you to select everything, as it does in the reskill-to menu. I am attaching a patch that implements that; but it's buggy: you can end up with a skill selected but 0% of the XP devoted to it.
(0024009)
KiloByte (manager)
2013-09-13 06:06

Doesn't sound like a good idea to me: you typically do such things upon returning to your stash, and at that moment your shield is often unworn, weapon unwielded, etc. Blocking a huge chunk of experience (a potion gives lots early on) just because you didn't suspect it's experience -- or worse, because you were unspoiled or unwary, doesn't make for good gameplay.

Even hiding the skill but allowing selecting it with some magic command wouldn't be good as most players don't read docs.

Thus, I'd leave status quo as is.
(0024684)
Medar (developer)
2013-12-11 09:31

This has been in for a while now.

The interface surely could be improved, but no need to keep this report open for that.

- Issue History
Date Modified Username Field Change
2013-06-07 02:40 Medar New Issue
2013-06-07 02:40 Medar File Added: 0001-Allow-potions-cards-of-exp-train-restricted-skills.patch
2013-06-07 02:41 Medar Tag Attached: patch
2013-06-07 03:17 neil Note Added: 0023089
2013-06-07 03:17 neil Status new => resolved
2013-06-07 03:17 neil Fixed in Branch => 0.13 development branch
2013-06-07 03:17 neil Resolution open => done
2013-06-07 03:17 neil Assigned To => neil
2013-06-07 03:17 Medar Status resolved => closed
2013-09-12 21:59 neil Note Added: 0024006
2013-09-12 22:00 neil File Added: exp-hide-untrainable.diff
2013-09-12 22:00 neil Status closed => new
2013-09-12 22:00 neil Resolution done => open
2013-09-13 06:06 KiloByte Note Added: 0024009
2013-12-11 09:31 Medar Note Added: 0024684
2013-12-11 09:31 Medar Status new => resolved
2013-12-11 09:31 Medar Resolution open => done
2013-12-11 09:32 Medar Status resolved => closed


Mantis 1.1.8[^]
Copyright © 2000 - 2009 Mantis Group
Powered by Mantis Bugtracker