From 450ed7120b0ccee4fa4affaa45f7b34c81aa1452 Mon Sep 17 00:00:00 2001
From: Glenn Maynard <gmaynard@users.sourceforge.net>
Date: Wed, 25 Apr 2007 00:23:09 +0000
Subject: [PATCH] remove m_bPickExtraStage, use false behavior.  Extra stage
 stuff is way too complicated for too many niche cases.

---
 stepmania/src/GameState.cpp         | 22 ++--------------------
 stepmania/src/MusicWheel.cpp        |  2 +-
 stepmania/src/PrefsManager.cpp      |  1 -
 stepmania/src/PrefsManager.h        |  1 -
 stepmania/src/ScreenSelectMusic.cpp |  4 ++--
 5 files changed, 5 insertions(+), 25 deletions(-)

diff --git a/stepmania/src/GameState.cpp b/stepmania/src/GameState.cpp
index feda35f003..7407a3af78 100644
--- a/stepmania/src/GameState.cpp
+++ b/stepmania/src/GameState.cpp
@@ -1195,8 +1195,6 @@ bool GameState::HasEarnedExtraStage() const
 	if( !IsFinalStage() && !IsExtraStage() )
 		return false;
 	
-	bool bOnePassed = false;
-	
 	FOREACH_EnabledPlayer( pn )
 	{
 		if( m_pCurSteps[pn]->GetDifficulty() != Difficulty_Hard && 
@@ -1206,27 +1204,11 @@ bool GameState::HasEarnedExtraStage() const
 		if( (IsExtraStage() && STATSMAN->m_CurStageStats.m_player[pn].GetGrade() <= GRADE_TIER_FOR_EXTRA_1) || 
 		    (IsExtraStage2() && STATSMAN->m_CurStageStats.m_player[pn].GetGrade() <= GRADE_TIER_FOR_EXTRA_2) )
 		{
-			bOnePassed = true;
-			break;
+			return true;
 		}
 	}
 	
-	if( !bOnePassed )
-		return false;
-
-	/* If PickExtraStage, allow EX2 if the chosen song was correct. */
-	if( PREFSMAN->m_bPickExtraStage && IsExtraStage2() )
-	{
-		Song* pSong;
-		Steps* pSteps;
-		SONGMAN->GetExtraStageInfo( false, GetCurrentStyle(), pSong, pSteps );
-		ASSERT(pSong);
-
-		const StageStats &stats = STATSMAN->m_CurStageStats;
-		return stats.m_vpPlayedSongs.size() && stats.m_vpPlayedSongs.back() == pSong;
-	}
-
-	return true;
+	return false;
 }
 
 PlayerNumber GameState::GetBestPlayer() const
diff --git a/stepmania/src/MusicWheel.cpp b/stepmania/src/MusicWheel.cpp
index 0ae69ec1fc..f0100c0a0b 100644
--- a/stepmania/src/MusicWheel.cpp
+++ b/stepmania/src/MusicWheel.cpp
@@ -127,7 +127,7 @@ void MusicWheel::BeginScreen()
 
 	WheelBase::BeginScreen();
 
-	if( (GAMESTATE->IsExtraStage() && !PREFSMAN->m_bPickExtraStage) || GAMESTATE->IsExtraStage2() )
+	if( GAMESTATE->IsExtraStage() || GAMESTATE->IsExtraStage2() )
 	{
 		m_WheelState = STATE_LOCKED;
 		SCREENMAN->PlayStartSound();
diff --git a/stepmania/src/PrefsManager.cpp b/stepmania/src/PrefsManager.cpp
index b0beb03c17..a73f24e2ba 100644
--- a/stepmania/src/PrefsManager.cpp
+++ b/stepmania/src/PrefsManager.cpp
@@ -189,7 +189,6 @@ PrefsManager::PrefsManager() :
 	m_iCoinsPerCredit		( "CoinsPerCredit",		1 ),
 	m_iSongsPerPlay			( "SongsPerPlay",		3, ValidateSongsPerPlay ),
 	m_bDelayedCreditsReconcile	( "DelayedCreditsReconcile",	false ),
-	m_bPickExtraStage		( "PickExtraStage",		false ),
 	m_bComboContinuesBetweenSongs	( "ComboContinuesBetweenSongs",	false ),
 	m_ShowSongOptions		( "ShowSongOptions",		Maybe_YES ),
 	m_bDancePointsForOni		( "DancePointsForOni",		false ),
diff --git a/stepmania/src/PrefsManager.h b/stepmania/src/PrefsManager.h
index f83961b322..1f7ca85f47 100644
--- a/stepmania/src/PrefsManager.h
+++ b/stepmania/src/PrefsManager.h
@@ -151,7 +151,6 @@ public:
 	Preference<int>		m_iCoinsPerCredit;
 	Preference<int>		m_iSongsPerPlay;
 	Preference<bool>	m_bDelayedCreditsReconcile;
-	Preference<bool>	m_bPickExtraStage;
 	Preference<bool>	m_bComboContinuesBetweenSongs;
 	Preference<Maybe>		m_ShowSongOptions;
 	Preference<bool>	m_bDancePointsForOni;
diff --git a/stepmania/src/ScreenSelectMusic.cpp b/stepmania/src/ScreenSelectMusic.cpp
index 3894ebc667..ffd34cdbf7 100644
--- a/stepmania/src/ScreenSelectMusic.cpp
+++ b/stepmania/src/ScreenSelectMusic.cpp
@@ -581,7 +581,7 @@ bool ScreenSelectMusic::DetectCodes( const InputEventPlus &input )
 	}
 	else if( CodeDetector::EnteredNextSort(input.GameI.controller) )
 	{
-		if( ( GAMESTATE->IsExtraStage() && !PREFSMAN->m_bPickExtraStage ) || GAMESTATE->IsExtraStage2() )
+		if( GAMESTATE->IsExtraStage() || GAMESTATE->IsExtraStage2() )
 			m_soundLocked.Play();
 		else
 			m_MusicWheel.NextSort();
@@ -781,7 +781,7 @@ void ScreenSelectMusic::MenuStart( const InputEventPlus &input )
 					bIsRepeat = true;
 
 			/* Don't complain about repeats if the user didn't get to pick. */
-			if( GAMESTATE->IsExtraStage() && !PREFSMAN->m_bPickExtraStage )
+			if( GAMESTATE->IsAnExtraStage() )
 				bIsRepeat = false;
 
 			if( bIsRepeat )