|
Viewing Issue Simple Details
[ Jump to Notes ]
[ Wiki ]
|
[ View Advanced ]
[ Issue History ]
[ Print ]
|
|
ID |
Category |
Severity |
Reproducibility |
Date Submitted |
Last Update |
|
0011092 |
[DCSS] Bug Report |
minor |
sometimes |
2017-06-07 18:49 |
2017-06-22 18:09 |
|
|
Reporter |
cut1less |
View Status |
public |
|
|
Assigned To |
advil |
|
Priority |
normal |
Resolution |
done |
|
|
Status |
closed |
|
Product Branch |
experimental branch |
|
|
Summary |
0011092: Viable species/background option ("+" key) returning random species/backgrounds |
|
Description |
At the new game character creation screen, using the viable species/background option (the + key) gives a random and potentially non-viable species/background even if the other choice (the background or species) has already been chosen.
Some examples that I've gotten are HuSk, SpNe, TeNe, and VsCK, which are not viable in either direction but sometimes come up as completed characters from using the + key on the species or the background screens.
Expected behavior:
1) If both species and background are unchosen, + key would give a random option for one.
2) If a species/background has been chosen (either by the player or randomly, as above or via *), then choose a viable background or species from what is recommended by the species/background.
Note: The viable character button ("#") works as intended, giving combos that are viable in at least one direction. |
|
Additional Information |
|
| Tags |
No tags attached. |
|
|
Attached Files |
 0001-Fix-Viable-species-background-new-game-option.patch [^] (990 bytes) 2017-06-08 00:24 [Show Content] [Hide Content]From 3341e9ebc9853a41bf6e23557f4a185d0117eab9 Mon Sep 17 00:00:00 2001
From: cut1less <kazoon401@gmail.com>
Date: Wed, 7 Jun 2017 17:13:10 -0500
Subject: [PATCH] Fix Viable species/background new game option
---
crawl-ref/source/newgame.cc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/crawl-ref/source/newgame.cc b/crawl-ref/source/newgame.cc
index eddbf91ce3..006bea500d 100644
--- a/crawl-ref/source/newgame.cc
+++ b/crawl-ref/source/newgame.cc
@@ -695,8 +695,8 @@ static void _add_choice_menu_options(int choice_type,
tmp->set_fg_colour(BROWN);
tmp->add_hotkey('+');
// If the player has species chosen, use VIABLE, otherwise use RANDOM
- if ((choice_type == C_SPECIES && ng.species != SP_UNKNOWN)
- || (choice_type == C_JOB && ng.job != JOB_UNKNOWN))
+ if ((choice_type == C_SPECIES && ng.job != JOB_UNKNOWN)
+ || (choice_type == C_JOB && ng.species != SP_UNKNOWN))
{
tmp->set_id(M_VIABLE);
}
--
2.13.0
|
|
|
