Call Set on the whole MusicWheelItem, not just TextBanner.

Fix HandleMessage not being propagated.

stepmania/src/MusicWheelItem.cpp

@@ -173,6 +173,7 @@ void MusicWheelItem::LoadFromWheelItemData( const WheelItemBaseData *pWIBD, int
 	// init and unhide type specific stuff
 	switch( pWID->m_Type )
 	{
+	DEFAULT_FAIL( pWID->m_Type );
 	case TYPE_SECTION:
 	case TYPE_COURSE:
 	case TYPE_SORT:
@@ -228,8 +229,13 @@ void MusicWheelItem::LoadFromWheelItemData( const WheelItemBaseData *pWIBD, int
 		m_textRoulette.SetText( THEME->GetString("MusicWheel","Portal") );
 		m_textRoulette.SetVisible( true );
 		break;
+	}
 
-	DEFAULT_FAIL( pWID->m_Type );	// invalid type
+	// Call "Set" so that elements can react to the change in song.
+	{
+		Message msg( "Set" );
+		msg.SetParam( "Song", data->m_pSong );
+		this->HandleMessage( msg );
 	}
 
 	Actor *pBars[] = { m_sprBar, m_sprExpandedBar, m_sprSectionBar, m_sprModeBar, m_sprSortBar, m_sprSongBar, NULL };
@@ -324,7 +330,7 @@ void MusicWheelItem::HandleMessage( const Message &msg )
 		RefreshGrades();
 	}
 
-	Actor::HandleMessage( msg );
+	WheelItemBase::HandleMessage( msg );
 }
 
 /*

stepmania/src/TextBanner.cpp

@@ -92,14 +92,12 @@ void TextBanner::LoadFromString(
 	const RString &sDisplayArtist, const RString &sTranslitArtist )
 {
 	LoadInternal( 
-		NULL,
 		sDisplayTitle, sTranslitTitle, 
 		sDisplaySubTitle, sTranslitSubTitle, 
 		sDisplayArtist, sTranslitArtist );
 }
 
 void TextBanner::LoadInternal( 
-	Song *pSong,
 	const RString &sDisplayTitle, const RString &sTranslitTitle, 
 	const RString &sDisplaySubTitle, const RString &sTranslitSubTitle, 
 	const RString &sDisplayArtist, const RString &sTranslitArtist )
@@ -110,9 +108,7 @@ void TextBanner::LoadInternal(
 	m_textSubTitle.SetText( sDisplaySubTitle, sTranslitSubTitle );
 	m_textArtist.SetText( sDisplayArtist, sTranslitArtist );
 
-	Message msg( "Set" );
+	// Set command is called by parent on the whole music wheel item
-	msg.SetParam( "Song", pSong );
-	this->HandleMessage( msg );
 }
 
 void TextBanner::LoadFromSong( Song *pSong )
@@ -127,7 +123,6 @@ void TextBanner::LoadFromSong( Song *pSong )
 	RString sTranslitArtist		= pSong ? m_sArtistPrependString + pSong->GetTranslitArtist() : RString("");
 
 	LoadInternal(
-		pSong,
 		sDisplayTitle, sTranslitTitle, 
 		sDisplaySubTitle, sTranslitSubTitle, 
 		sDisplayArtist, sTranslitArtist );

stepmania/src/TextBanner.h

@@ -28,7 +28,6 @@ public:
 
 private:
 	void LoadInternal( 
-		Song *pSong,
 		const RString &sDisplayTitle, const RString &sTranslitTitle, 
 		const RString &sDisplaySubTitle, const RString &sTranslitSubTitle, 
 		const RString &sDisplayArtist, const RString &sTranslitArtist );