diff --git a/stepmania/src/CryptManager.cpp b/stepmania/src/CryptManager.cpp index fb145a04b1..5549bfa0d9 100644 --- a/stepmania/src/CryptManager.cpp +++ b/stepmania/src/CryptManager.cpp @@ -7,8 +7,8 @@ #include "crypto/CryptRSA.h" #include "crypto/CryptRand.h" +#include "crypto/CryptMD5.h" -static const CString SIGNATURE_APPEND = ".sig.rsa"; static const CString PRIVATE_KEY_PATH = "Data/private.key.rsa"; static const CString PUBLIC_KEY_PATH = "Data/public.key.rsa"; static const int KEY_LENGTH = 1024; @@ -63,8 +63,11 @@ void CryptManager::GenerateRSAKey( unsigned int keyLength, CString privFilename, out.Close(); } -void CryptManager::SignFile( CString sPath ) +void CryptManager::SignFileToFile( CString sPath, CString sSignatureFilename ) { + if( sSignatureFilename == "" ) + sSignatureFilename = sPath + SIGNATURE_APPEND; + LOG->Trace("SignFile(%s)", sPath.c_str()); ASSERT( PREFSMAN->m_bSignProfileData ); @@ -74,24 +77,25 @@ void CryptManager::SignFile( CString sPath ) if( !IsAFile(sPath) ) return; - const CString sig = GetFileSignature( sPath ); + const CString sig = Sign( sPath ); RageFile out; - const CString sSignatureFilename = sPath + SIGNATURE_APPEND; if( !out.Open( sSignatureFilename, RageFile::WRITE ) ) RageException::Throw( "Error opening %s: %s", sSignatureFilename.c_str(), out.GetError().c_str() ); out.Write( sig ); } -bool CryptManager::VerifyFile( CString sPath ) +bool CryptManager::VerifyFileWithFile( CString sPath, CString sSignatureFilename ) { + if( sSignatureFilename == "" ) + sSignatureFilename = sPath + SIGNATURE_APPEND; + LOG->Trace("VerifyFile(%s)", sPath.c_str()); ASSERT( PREFSMAN->m_bSignProfileData ); if( !IsAFile(PUBLIC_KEY_PATH) ) return false; - const CString sSignatureFilename = sPath + SIGNATURE_APPEND; if( !IsAFile(sSignatureFilename) ) return false; @@ -103,10 +107,10 @@ bool CryptManager::VerifyFile( CString sPath ) in.Read( sig ); } - return VerifyFile( sPath, sig ); + return Verify( sPath, sig ); } -CString CryptManager::GetFileSignature( CString sPath ) +CString CryptManager::Sign( CString sPath ) { ASSERT( PREFSMAN->m_bSignProfileData ); @@ -140,7 +144,7 @@ CString CryptManager::GetFileSignature( CString sPath ) return sig; } -bool CryptManager::VerifyFile( CString sPath, CString sSignature ) +bool CryptManager::Verify( CString sPath, CString sSignature ) { ASSERT( PREFSMAN->m_bSignProfileData ); @@ -171,21 +175,42 @@ bool CryptManager::VerifyFile( CString sPath, CString sSignature ) return key.Verify( data, sSignature ); } +CString BinaryToHex( const unsigned char *string, int iNumBytes ) +{ + CString s; + for( int i=0; im_bSignProfileData ); -// MD5 md5; -// HashFilter md5Filter(md5); -// -// auto_ptr channelSwitch(new ChannelSwitch); -// channelSwitch->AddDefaultRoute(md5Filter); -// RageFileSource(filename, true, channelSwitch.release()); -// -// HexEncoder encoder(new RageFileSink(cout), false); -// cout << "\nMD5: "; -// md5Filter.TransferTo(encoder); + struct MD5Context md5c; + unsigned char digest[16]; + int iBytesRead; + unsigned char buffer[1024]; + + RageFile file; + if( !file.Open( fn, RageFile::READ ) ) + { + LOG->Warn( "GetMD5: Failed to open file '%s'", fn.c_str() ); + return ""; + } + + MD5Init(&md5c); + while( !file.AtEOF() && file.GetError().empty() ) + { + iBytesRead = file.Read( buffer, sizeof(buffer) ); + MD5Update(&md5c, buffer, iBytesRead); + } + MD5Final(digest, &md5c); + + return BinaryToHex( digest, sizeof(digest) ); } CString CryptManager::GetPublicKeyFileName() diff --git a/stepmania/src/CryptManager.h b/stepmania/src/CryptManager.h index a14b38c533..34cab116f9 100644 --- a/stepmania/src/CryptManager.h +++ b/stepmania/src/CryptManager.h @@ -1,6 +1,8 @@ #ifndef CryptManager_H #define CryptManager_H +const CString SIGNATURE_APPEND = ".sig"; + class CryptManager { public: @@ -9,13 +11,13 @@ public: static void GenerateRSAKey( unsigned int keyLength, CString privFilename, CString pubFilename, CString seed ); - static void SignFile( CString sPath ); - static bool VerifyFile( CString sPath ); + static void SignFileToFile( CString sPath, CString sSignatureFile = "" ); + static bool VerifyFileWithFile( CString sPath, CString sSignatureFile = "" ); - static CString GetFileSignature( CString sPath ); - static bool VerifyFile( CString sPath, CString sSignature ); + static CString Sign( CString sPath ); + static bool Verify( CString sPath, CString sSignature ); - static void DigestFile( CString fn ); + static CString GetMD5( CString fn ); // in Hex static CString GetPublicKeyFileName(); }; diff --git a/stepmania/src/Profile.h b/stepmania/src/Profile.h index e419866584..ce21437e17 100644 --- a/stepmania/src/Profile.h +++ b/stepmania/src/Profile.h @@ -19,6 +19,36 @@ #include "XmlFile.h" #include "HighScore.h" + +// +// Current file versions +// +const CString STATS_XML = "Stats.xml"; + +const CString EDITABLE_INI = "Editable.ini"; +// Editable data is an INI because the default INI file association on Windows +// systems will open the ini file in an editor. The default association for +// XML will open in IE. Users have a much better chance of discovering how to +// edit this data if they don't have to fight against the file associations. + +const CString DONT_SHARE_SIG = "DontShare.sig"; +// The "don't share" file is something that the user should always keep private. +// They can safely share STATS_XML with STATS_XML's signature so that others +// can authenticate the STATS_XML data. However, others can't copy that data +// to their own profile for use in the game unless they also have the "don't +// share" file. DontShare contains a piece of information that we can +// construct using STATS_XML but the user can't construct using STATS_XML. +// The file contains a signature of the STATS_XML's signature. + +const CString PUBLIC_KEY_FILE = "public.key"; +const CString SCREENSHOTS_SUBDIR = "Screenshots/"; +const CString EDITS_SUBDIR = "Edits/"; + +#define REASONABLE_EDITABLE_INI_SIZE_BYTES 2*1000 // 2KB +#define REASONABLE_STATS_XML_SIZE_BYTES 5*1000*1000 // 5MB + + + class Song; class Steps; class Course; @@ -44,12 +74,12 @@ public: // Editable data // CString m_sDisplayName; + CString m_sLastUsedHighScoreName; // this doesn't really belong in "editable", but we need it in the smaller editable file so that it can be ready quickly. float m_fWeightPounds; // // General data // - CString m_sLastUsedHighScoreName; bool m_bUsingProfileDefaultModifiers; CString m_sDefaultModifiers; int m_iTotalPlays; @@ -116,7 +146,7 @@ public: struct Screenshot { CString sFileName; // no directory part - just the file name - CString sSignature; // sFile signed with the machine's private key + CString sMD5; // MD5 hash of the screenshot file HighScore highScore; // high score that the screenshot is taken of }; vector m_vScreenshots; @@ -153,14 +183,14 @@ public: void LoadCourseScoresFromDirSM390a12( CString sDir ); void LoadCategoryScoresFromDirSM390a12( CString sDir ); - void LoadEditableDataFromNode( const XNode* pNode ); + void LoadEditableDataFromDir( CString sDir ); void LoadGeneralDataFromNode( const XNode* pNode ); void LoadSongScoresFromNode( const XNode* pNode ); void LoadCourseScoresFromNode( const XNode* pNode ); void LoadCategoryScoresFromNode( const XNode* pNode ); void LoadScreenshotDataFromNode( const XNode* pNode ); - XNode* SaveEditableDataCreateNode() const; + void SaveEditableDataToDir( CString sDir ) const; XNode* SaveGeneralDataCreateNode() const; XNode* SaveSongScoresCreateNode() const; XNode* SaveCourseScoresCreateNode() const; diff --git a/stepmania/src/ProfileHtml.cpp b/stepmania/src/ProfileHtml.cpp index 0fa3a0ca8d..52cef8e0ed 100644 --- a/stepmania/src/ProfileHtml.cpp +++ b/stepmania/src/ProfileHtml.cpp @@ -22,10 +22,11 @@ #include "Course.h" #include "Bookkeeper.h" #include "PrefsManager.h" +#include "CryptManager.h" -#define STATS_HTML "stats.html" -#define STYLE_CSS "style.css" +const CString STATS_HTML = "Stats.html"; +const CString STYLE_CSS = "Style.css"; #define TITLE THEME->GetMetric("ProfileHtml","Title") #define FOOTER THEME->GetMetric("ProfileHtml","Footer") @@ -39,6 +40,7 @@ inline void PRINT_OPEN(RageFile &f,CString sName,bool bExpanded,CString sID){ g_ inline void PRINT_OPEN(RageFile &f,CString sName,bool bExpanded=false) { PRINT_OPEN(f,sName,bExpanded,MakeUniqueId()); } inline void PRINT_CLOSE(RageFile &f) { f.Write( "\n" "\n" ); g_Level--; ASSERT(g_Level>=0); } +#define PRETTY_PERCENT(numerator,denominator) ssprintf("%0.2f%%",(float)numerator/(float)denominator*100) struct Table { @@ -54,6 +56,9 @@ struct Table Line(int r,CString n,int v) { sRank = ssprintf("%d",r); sName = n; sValue = ssprintf("%d",v); } Line(int r,CString n,CString sn,int v) { sRank = ssprintf("%d",r); sName = n; sSubName = sn; sValue = ssprintf("%d",v); } Line(int r,CString n,CString sn,CString ssn,int v) { sRank = ssprintf("%d",r); sName = n; sSubName = sn; sSubSubName = ssn; sValue = ssprintf("%d",v); } + Line(int r,CString n,CString v) { sRank = ssprintf("%d",r); sName = n; sValue = v; } + Line(int r,CString n,CString sn,CString v) { sRank = ssprintf("%d",r); sName = n; sSubName = sn; sValue = v; } + Line(int r,CString n,CString sn,CString ssn,CString v) { sRank = ssprintf("%d",r); sName = n; sSubName = sn; sSubSubName = ssn; sValue = v; } CString sRank; CString sName; @@ -85,10 +90,15 @@ inline void PrintTable(RageFile &f,Table &table) return; bool bPrintRank = !vLines.empty() && !vLines[0].sRank.empty(); + bool bPrintInstructions = !vLines.empty() && !vLines[0].sRank.empty() && !vLines[0].sSubName.empty() && !vLines[0].sSubSubName.empty() && !vLines[0].sValue.empty(); int iMaxItemsPerCol = (vLines.size()+iNumCols-1) / iNumCols; iNumCols = (vLines.size()+iMaxItemsPerCol-1) / iMaxItemsPerCol; // round up - f.Write("\n"); + f.Write(ssprintf("
\n",bPrintInstructions?"instructions":"group")); + if( iNumCols == 0 ) + { + f.Write("\n"); + } for( int col=0; col\n"); @@ -114,9 +124,18 @@ inline void PrintTable(RageFile &f,Table &table) f.Write("

"); f.Write( line.sName ); f.Write("

"); - f.Write("

"); - f.Write( line.sSubName ); - f.Write("

"); + if( !line.sSubName.empty() ) + { + f.Write("

"); + f.Write( line.sSubName ); + f.Write("

"); + } + f.Write(""); + } + else if( line.sValue.empty() ) + { + f.Write(""); } else @@ -125,6 +144,17 @@ inline void PrintTable(RageFile &f,Table &table) f.Write( line.sName ); f.Write(""); } + + if( !line.sSubSubName.empty() ) + { + f.Write(""); + f.Write(""); + } + if( !line.sValue.empty() ) { f.Write(""); @@ -143,6 +173,95 @@ inline void PrintTable(RageFile &f,Table &table) f.Write("
empty"); + f.Write( line.sName ); f.Write(" "); + f.Write("

"); + f.Write( line.sSubSubName ); + f.Write("

"); + f.Write("		 
\n"); } +#define STRING_AS_LINK(s) CString(""+s+"") + +void PrintInstructions( RageFile &f, const Profile *pProfile, CString sTitle ) +{ + PRINT_OPEN(f,sTitle); + { + PRINT_OPEN(f,"Overview",true); + { + BEGIN_TABLE(1); + TABLE_LINE1("

This directory contains all your game profile data. Please read these instructions before modifying or moving any of these files. Modifying files may result in irreversible loss of your data.

\n"); + END_TABLE; + } + PRINT_CLOSE(f); + + PRINT_OPEN(f,"Description of Files"); + { + BEGIN_TABLE(1); + TABLE_LINE2(STRING_AS_LINK(EDITS_SUBDIR), CString("Place edit step files in this directory. See the Edits section below for more details.")); + TABLE_LINE2(STRING_AS_LINK(SCREENSHOTS_SUBDIR), CString("All screenshots that you take are saved in this directory.")); + TABLE_LINE2(DONT_SHARE_SIG, CString("This is a secret file that you should never share with anyone else. See the Sharing Your Data section below for more details.")); + TABLE_LINE2(STRING_AS_LINK(EDITABLE_INI), CString("Holds preferences that you can edit offline using your home computer. This file is not digitally signed.")); + TABLE_LINE2(STATS_HTML, CString("You're looking at this file now. It contains a formatted view of all your saved data, plus some data from the last machine you played on.")); + TABLE_LINE2(STATS_HTML+SIGNATURE_APPEND, CString("The signature file for "+STATS_HTML+".")); + TABLE_LINE2(STRING_AS_LINK(STATS_XML), CString("This is the primary data file. It contains all the score data and statistics, and is read by the game when you join.")); + TABLE_LINE2(STATS_XML+SIGNATURE_APPEND, CString("The signature file for "+STATS_XML+".")); + TABLE_LINE2(STYLE_CSS, CString("Contains style data used by "+STATS_HTML+".")); + END_TABLE; + } + PRINT_CLOSE(f); + + PRINT_OPEN(f,"Digital Signatures"); + { + BEGIN_TABLE(1); + TABLE_LINE1("

Some files on your memory card have a corresponding digital signature. Digital signatures are used to verify that your files haven't been modified outside of the game. This prevents cheaters from changing their score data and then passing it off as real.

\n"); + TABLE_LINE1("

Before the game reads your memory card data, it verifies that your data and digital signatures match. If the data and signatures don't match, then your data has been modified outside of the game. When the game detects this condition, it will completely ignore your tampered data. It is very important that you -do not- modify any file that has a digital signature because this will cause your data to be permanently unusable.

\n"); + TABLE_LINE1("

If someone else shares their profile data with you, you can verify their score data using digital signatures. To verify their data, you'll need 3 things:

\n"); + TABLE_LINE1("

- the "+STATS_XML+" data file

\n"); + TABLE_LINE1("

- the digital signature file "+STATS_XML+SIGNATURE_APPEND+"

\n"); + TABLE_LINE1("

- a small utility that will check data against a signature

\n"); + END_TABLE; + } + PRINT_CLOSE(f); + + PRINT_OPEN(f,"About Editable Preferences"); + { + BEGIN_TABLE(1); + TABLE_LINE1("

The file "+STRING_AS_LINK(EDITABLE_INI)+" contains settings that you can modify using your home computer. If you're using a Windows PC, you can click here to open the file for editing. This file is not digitally signed and the game will import any changes you make to this file.

\n"); + END_TABLE; + } + PRINT_CLOSE(f); + + PRINT_OPEN(f,"About Screenshots"); + { + BEGIN_TABLE(1); + TABLE_LINE1("

The "+STRING_AS_LINK(SCREENSHOTS_SUBDIR)+" directory contains all screenshots that you've captured while playing the game. See the Screenshots section later on this page to see thumbnails and more information captured at the time of the screenshot. The Screenshots section also lists an MD5 hash of the screenshot file. You can use the MD5 has to verify that the screenshot has not been modified since it was first saved.

\n"); + TABLE_LINE1("

If your memory card is full, you may delete files from this directory or the move files to another disk. If you move a screenshot to another disk, you can still verify the screenshot file using the MD5 hash.

\n"); + END_TABLE; + } + PRINT_CLOSE(f); + + PRINT_OPEN(f,"About Edits"); + { + BEGIN_TABLE(1); + TABLE_LINE1("

The "+STRING_AS_LINK(EDITS_SUBDIR)+" directory contains edit step files that you've created yourself or downloaded from the internet. See here for more information about edit files.

\n"); + END_TABLE; + } + PRINT_CLOSE(f); + + PRINT_OPEN(f,"Sharing Your Data"); + { + BEGIN_TABLE(1); + TABLE_LINE1("

You can share your score data with other players or submit it to a web site for an internet ranking contest. When sharing your data though, do -not- share the file "+DONT_SHARE_SIG+". "+DONT_SHARE_SIG+" is private digital signature required by the game before loading memory card data. Without "+DONT_SHARE_SIG+", the person you're sharing data with can verify that your data is original, but can't load your data using their memory card or pass your scores off as their own.

\n"); + TABLE_LINE1("

If you do share your "+DONT_SHARE_SIG+" with someone, then they can completely replicate your memory card and pass your scores off as their own.

\n"); + END_TABLE; + } + PRINT_CLOSE(f); + + PRINT_OPEN(f,"Backing Up/Moving Your Data"); + { + BEGIN_TABLE(1); + TABLE_LINE1("

To make a backup of your data, copy the entire "+PREFSMAN->m_sMemoryCardProfileSubdir+"/ directory on the root of your memory card to your local hard drive.

\n"); + TABLE_LINE1("

To move your data from the current memory card to a new memory card, move the entire "+PREFSMAN->m_sMemoryCardProfileSubdir+"/ directory on your current memory card to the root directory on the new memory card.

\n"); + END_TABLE; + } + PRINT_CLOSE(f); + } + PRINT_CLOSE(f); +} + void PrintStatistics( RageFile &f, const Profile *pProfile, CString sTitle, vector &vpSongs, vector &vpAllSteps, vector &vStepsTypesToShow, map mapStepsToSong, vector vpCourses ) { PRINT_OPEN(f,sTitle,true); @@ -257,6 +376,12 @@ void PrintPopularity( RageFile &f, const Profile *pProfile, CString sTitle, vect unsigned uMaxToShow = min( vpSongs.size(), (unsigned)100 ); + // compute total plays + int iTotalPlays = 0; + for( unsigned i=0; iGetSongNumTimesPlayed( vpSongs[i] ); + + { PRINT_OPEN(f, "Most Popular Songs" ); { @@ -267,8 +392,10 @@ void PrintPopularity( RageFile &f, const Profile *pProfile, CString sTitle, vect int iNumTimesPlayed = pProfile->GetSongNumTimesPlayed(pSong); if( iNumTimesPlayed == 0 || iNumTimesPlayed < iPopularNumPlaysThreshold ) // not popular break; // done searching - TABLE_LINE4(i+1, pSong->GetDisplayMainTitle(), pSong->GetDisplaySubTitle(), iNumTimesPlayed ); + TABLE_LINE4(i+1, pSong->GetDisplayMainTitle(), pSong->GetDisplaySubTitle(), PRETTY_PERCENT(iNumTimesPlayed,iTotalPlays) ); } + if( i == 0 ) + TABLE_LINE1("empty"); END_TABLE; } PRINT_CLOSE(f); @@ -285,8 +412,10 @@ void PrintPopularity( RageFile &f, const Profile *pProfile, CString sTitle, vect int iNumTimesPlayed = pProfile->GetSongNumTimesPlayed(pSong); if( iNumTimesPlayed >= iPopularNumPlaysThreshold ) // not unpopular break; // done searching - TABLE_LINE4(i+1, pSong->GetDisplayMainTitle(), pSong->GetDisplaySubTitle(), iNumTimesPlayed ); + TABLE_LINE4(i+1, pSong->GetDisplayMainTitle(), pSong->GetDisplaySubTitle(), PRETTY_PERCENT(iNumTimesPlayed,iTotalPlays) ); } + if( i == 0 ) + TABLE_LINE1("empty"); END_TABLE; } PRINT_CLOSE(f); @@ -302,15 +431,18 @@ void PrintPopularity( RageFile &f, const Profile *pProfile, CString sTitle, vect for( unsigned i=0; iGetStepsNumTimesPlayed(pSteps)==0 ) + int iNumTimesPlayed = pProfile->GetStepsNumTimesPlayed(pSteps); + if( iNumTimesPlayed==0 ) continue; // skip Song* pSong = mapStepsToSong[pSteps]; CString s; s += GAMEMAN->NotesTypeToString(pSteps->m_StepsType); s += " "; s += DifficultyToString(pSteps->GetDifficulty()); - TABLE_LINE5(i+1, pSong->GetDisplayMainTitle(), pSong->GetDisplaySubTitle(), s, pProfile->GetStepsNumTimesPlayed(pSteps) ); + TABLE_LINE5(i+1, pSong->GetDisplayMainTitle(), pSong->GetDisplaySubTitle(), s, PRETTY_PERCENT(iNumTimesPlayed,iTotalPlays) ); } + if( i == 0 ) + TABLE_LINE1("empty"); END_TABLE; } PRINT_CLOSE(f); @@ -326,8 +458,11 @@ void PrintPopularity( RageFile &f, const Profile *pProfile, CString sTitle, vect for( unsigned i=0; im_sName, pProfile->GetCourseNumTimesPlayed(pCourse) ); + int iNumTimesPlayed = pProfile->GetCourseNumTimesPlayed(pCourse); + TABLE_LINE3(i+1, pCourse->m_sName, PRETTY_PERCENT(iNumTimesPlayed,iTotalPlays) ); } + if( i == 0 ) + TABLE_LINE1("empty"); END_TABLE; } PRINT_CLOSE(f); @@ -337,46 +472,72 @@ void PrintPopularity( RageFile &f, const Profile *pProfile, CString sTitle, vect } -typedef void (*FnPrintSong)(RageFile &f, const Profile *pProfile, Song* pSong ); -typedef void (*FnPrintGroup)(RageFile &f, const Profile *pProfile, CString sGroup ); -typedef void (*FnPrintStepsType)(RageFile &f, const Profile *pProfile, StepsType st ); +// return true if anything was printed +typedef bool (*FnPrintSong)(RageFile &f, const Profile *pProfile, Song* pSong ); +typedef bool (*FnPrintGroup)(RageFile &f, const Profile *pProfile, CString sGroup ); +typedef bool (*FnPrintStepsType)(RageFile &f, const Profile *pProfile, StepsType st ); -void PrintSongsInGroup( RageFile &f, const Profile *pProfile, CString sGroup, FnPrintSong pFn ) +bool PrintSongsInGroup( RageFile &f, const Profile *pProfile, CString sGroup, FnPrintSong pFn ) { + vector vpSongs; + SONGMAN->GetSongs( vpSongs, sGroup ); + + if( vpSongs.empty() ) + return false; + PRINT_OPEN(f, sGroup ); { - vector vpSongs; - SONGMAN->GetSongs( vpSongs, sGroup ); - + bool bPrintedAny = false; for( unsigned i=0; iGetGroupNames( asGroups ); + if( asGroups.empty() ) + return false; + PRINT_OPEN(f, sTitle ); { + bool bPrintedAny = false; + for( unsigned g=0; g vStepsTypesToShow, FnPrintStepsType pFn ) +bool PrintStepsTypes( RageFile &f, const Profile *pProfile, CString sTitle, vector vStepsTypesToShow, FnPrintStepsType pFn ) { PRINT_OPEN(f, sTitle ); { @@ -388,22 +549,20 @@ void PrintStepsTypes( RageFile &f, const Profile *pProfile, CString sTitle, vect } } PRINT_CLOSE(f); + + return true; } -void PrintHighScoresForSong( RageFile &f, const Profile *pProfile, Song* pSong ) +bool PrintHighScoresForSong( RageFile &f, const Profile *pProfile, Song* pSong ) { int iNumTimesPlayed = pProfile->GetSongNumTimesPlayed(pSong); if( iNumTimesPlayed == 0 ) - return; // skip + return false; // skip vector vpSteps = pSong->GetAllSteps(); PRINT_OPEN(f, pSong->GetFullDisplayTitle() ); { - BEGIN_TABLE(2); - TABLE_LINE2( "NumTimesPlayed", iNumTimesPlayed ); - END_TABLE; - // // Print Steps list // @@ -422,10 +581,6 @@ void PrintHighScoresForSong( RageFile &f, const Profile *pProfile, Song* pSong ) DifficultyToString(pSteps->GetDifficulty()); PRINT_OPEN(f, s, true); { - BEGIN_TABLE(2); - TABLE_LINE2( "NumTimesPlayed", hsl.iNumTimesPlayed ); - END_TABLE; - BEGIN_TABLE(2); for( unsigned i=0; i &vpSongs, vector &vpAllSteps, vector &vStepsTypesToShow, map mapStepsToSong, vector vpCourses ) @@ -453,7 +610,7 @@ void PrintHighScores( RageFile &f, const Profile *pProfile, CString sTitle, vect PrintGroups( f, pProfile, sTitle, PrintHighScoresForGroup ); } -void PrintDifficultyTableForStepsType( RageFile &f, const Profile *pProfile, StepsType st ) +bool PrintGradeTableForStepsType( RageFile &f, const Profile *pProfile, StepsType st ) { unsigned i; const vector &vpSongs = SONGMAN->GetAllSongs(); @@ -468,7 +625,7 @@ void PrintDifficultyTableForStepsType( RageFile &f, const Profile *pProfile, Ste { if( dc == DIFFICULTY_EDIT ) continue; // skip - f.PutLine( ssprintf("%s", Capitalize(DifficultyToString(dc).Left(3)).c_str()) ); + f.PutLine( ssprintf("%s", Capitalize(DifficultyToString(dc)).c_str()) ); } f.PutLine( "" ); @@ -490,10 +647,22 @@ void PrintDifficultyTableForStepsType( RageFile &f, const Profile *pProfile, Ste continue; // skip Steps* pSteps = pSong->GetStepsByDifficulty( st, dc, false ); - if( pSteps ) - f.PutLine( ssprintf("

%d

",pSteps->GetMeter()) ); + if( pSteps && !pSteps->IsAutogen() ) + { + f.PutLine(""); + f.PutLine(""); + f.PutLine( ssprintf("",pSteps->GetMeter()) ); + HighScore hs = pProfile->GetStepsHighScoreList( pSteps ).GetTopScore(); + Grade grade = hs.grade; + if( grade != GRADE_NO_DATA ) + f.PutLine( ssprintf("",GradeToThemedString(grade).c_str()) ); + f.PutLine("

%d

%s

"); + f.PutLine(""); + } else + { f.PutLine( " " ); + } } f.Write( "" ); @@ -502,14 +671,16 @@ void PrintDifficultyTableForStepsType( RageFile &f, const Profile *pProfile, Ste f.PutLine( "\n" ); } PRINT_CLOSE(f); + + return true; } -void PrintDifficultyTable( RageFile &f, const Profile *pProfile, CString sTitle, vector &vpSongs, vector &vpAllSteps, vector &vStepsTypesToShow, map mapStepsToSong, vector vpCourses ) +void PrintGradeTable( RageFile &f, const Profile *pProfile, CString sTitle, vector &vpSongs, vector &vpAllSteps, vector &vStepsTypesToShow, map mapStepsToSong, vector vpCourses ) { - PrintStepsTypes( f, pProfile, sTitle, vStepsTypesToShow, PrintDifficultyTableForStepsType ); + PrintStepsTypes( f, pProfile, sTitle, vStepsTypesToShow, PrintGradeTableForStepsType ); } -void PrintInventoryForSong( RageFile &f, const Profile *pProfile, Song* pSong ) +bool PrintInventoryForSong( RageFile &f, const Profile *pProfile, Song* pSong ) { vector vpSteps = pSong->GetAllSteps(); @@ -525,6 +696,7 @@ void PrintInventoryForSong( RageFile &f, const Profile *pProfile, Song* pSong ) TABLE_LINE2( "Credit", pSong->m_sCredit ); TABLE_LINE2( "MusicLength", SecondsToTime(pSong->m_fMusicLengthSeconds) ); TABLE_LINE2( "Lyrics", !pSong->m_sLyricsFile.empty() ); + TABLE_LINE2( "NumTimesPlayed", pProfile->GetSongNumTimesPlayed(pSong) ); END_TABLE; // @@ -541,9 +713,10 @@ void PrintInventoryForSong( RageFile &f, const Profile *pProfile, Song* pSong ) DifficultyToString(pSteps->GetDifficulty()); PRINT_OPEN(f, s, true); // use poister value as the hash { - BEGIN_TABLE(2); - TABLE_LINE2( "Description", pSteps->GetDescription() ); + BEGIN_TABLE(3); TABLE_LINE2( "Meter", pSteps->GetMeter() ); + TABLE_LINE2( "Description", pSteps->GetDescription() ); + TABLE_LINE2( "NumTimesPlayed", pProfile->GetStepsNumTimesPlayed(pSteps) ); END_TABLE; BEGIN_TABLE(2); @@ -560,11 +733,13 @@ void PrintInventoryForSong( RageFile &f, const Profile *pProfile, Song* pSong ) } } PRINT_CLOSE(f); + + return true; } -void PrintInventoryForGroup( RageFile &f, const Profile *pProfile, CString sGroup ) +bool PrintInventoryForGroup( RageFile &f, const Profile *pProfile, CString sGroup ) { - PrintSongsInGroup( f, pProfile, sGroup, PrintInventoryForSong ); + return PrintSongsInGroup( f, pProfile, sGroup, PrintInventoryForSong ); } void PrintInventoryList( RageFile &f, const Profile *pProfile, CString sTitle, vector &vpSongs, vector &vpAllSteps, vector &vStepsTypesToShow, map mapStepsToSong, vector vpCourses ) @@ -683,9 +858,13 @@ void PrintScreenshots( RageFile &f, const Profile *pProfile, CString sTitle, CSt PRINT_CLOSE(f); } -enum SaveType { SAVE_TYPE_PLAYER, SAVE_TYPE_MACHINE }; -void SaveStatsWebPageToDir( CString sDir, SaveType saveType, const Profile *pProfile, const Profile *pProfileMachine ) +void SaveStatsWebPage( + CString sDir, + const Profile *pProfile, + const Profile *pMachineProfile, + HtmlType htmlType + ) { CString fn = sDir + STATS_HTML; @@ -790,47 +969,58 @@ function expandIt(whichEl)\n\ \n\ \n\ ", -TITLE.c_str(), STYLE_CSS ) ); +TITLE.c_str(), STYLE_CSS.c_str() ) ); } - CString sType; - switch( saveType ) - { - case SAVE_TYPE_PLAYER: sType = "Player: "; break; - case SAVE_TYPE_MACHINE: sType = "Machine: "; break; - } - - CString sName = - pProfile->m_sLastUsedHighScoreName.empty() ? - pProfile->m_sDisplayName : - pProfile->m_sLastUsedHighScoreName; + CString sName = pProfile->GetDisplayName(); + CString sMachineName = pMachineProfile->GetDisplayName(); time_t ltime = time( NULL ); CString sTime = ctime( <ime ); - f.Write( ssprintf( - "

%s

%s %s
%s
\n", - TITLE.c_str(), sType.c_str(), sName.c_str(), sTime.c_str() ) ); - - CString sPlayerName = pProfile->GetDisplayName(); - CString sMachineName = pProfileMachine->GetDisplayName(); - - switch( saveType ) + CString sNameCell; + switch( htmlType ) { - case SAVE_TYPE_PLAYER: - PrintStatistics( f, pProfile, sPlayerName+"'s Statistics", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintPopularity( f, pProfile, sPlayerName+"'s Popularity", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintHighScores( f, pProfile, sPlayerName+"'s High Scores", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintScreenshots( f, pProfile, sPlayerName+"'s Screenshots", sDir ); - PrintPopularity( f, pProfileMachine, sMachineName+"'s Popularity", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintHighScores( f, pProfileMachine, sMachineName+"'s High Scores", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + case HTML_TYPE_PLAYER: + sNameCell = ssprintf( + "Player Name: %s
\n" + "Last Machine: %s
\n" + "%s\n", + sName.c_str(), sMachineName.c_str(), sTime.c_str() ); break; - case SAVE_TYPE_MACHINE: - PrintStatistics( f, pProfile, "Statistics", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintPopularity( f, pProfile, "Popularity", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintHighScores( f, pProfile, "High Scores", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintDifficultyTable( f, pProfile, "Difficulty Table", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintInventoryList( f, pProfile, "Song Information", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); - PrintBookkeeping( f, pProfile, "Bookkeeping", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + case HTML_TYPE_MACHINE: + sNameCell = ssprintf( + "Machine: %s
\n" + "%s\n", + sName.c_str(), sTime.c_str() ); + break; + default: + ASSERT(0); + } + + f.Write( ssprintf( + "

%s

%s
\n", + TITLE.c_str(), sNameCell.c_str() ) ); + + switch( htmlType ) + { + case HTML_TYPE_PLAYER: + PrintInstructions( f, pProfile, "Instructions" ); + PrintStatistics( f, pProfile, "My Statistics", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintPopularity( f, pProfile, "My Popularity", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintHighScores( f, pProfile, "My High Scores", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintScreenshots( f, pProfile, "My Screenshots", sDir ); + PrintGradeTable( f, pProfile, "My Grade Table", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintPopularity( f, pProfile, "Last Machine Popularity", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintHighScores( f, pProfile, "Last Machine High Scores", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + break; + case HTML_TYPE_MACHINE: + PrintStatistics( f, pProfile, "Statistics", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintPopularity( f, pProfile, "Popularity", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintHighScores( f, pProfile, "High Scores", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintScreenshots( f, pProfile, "Screenshots", sDir ); + PrintGradeTable( f, pProfile, "Grade Table", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintInventoryList( f, pProfile, "Song Information", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); + PrintBookkeeping( f, pProfile, "Bookkeeping", vpSongs, vpAllSteps, vStepsTypesToShow, mapStepsToSong, vpCourses ); break; default: ASSERT(0); @@ -838,25 +1028,23 @@ TITLE.c_str(), STYLE_CSS ) ); f.PutLine( ssprintf("

%s

\n", FOOTER.c_str()) ); - f.PutLine( "" ); - f.PutLine( "" ); + f.PutLine( "\n" ); + f.PutLine( "\n" ); + f.Close(); + // + // Sign Stats.html + // + if( PREFSMAN->m_bSignProfileData ) + CryptManager::SignFileToFile(fn); + // // Copy CSS file from theme. If the copy fails, oh well... // CString sStyleFile = THEME->GetPathToO("ProfileManager style.css"); FileCopy( sStyleFile, sDir+STYLE_CSS ); LOG->Trace( "Done." ); -} -void SavePlayerHtmlToDir( CString sDir, const Profile* pProfilePlayer, const Profile* pProfileMachine ) -{ - SaveStatsWebPageToDir( sDir, SAVE_TYPE_PLAYER, pProfilePlayer, pProfileMachine ); -} - -void SaveMachineHtmlToDir( CString sDir, const Profile* pProfileMachine ) -{ - SaveStatsWebPageToDir( sDir, SAVE_TYPE_MACHINE, pProfileMachine, pProfileMachine ); } diff --git a/stepmania/src/ProfileHtml.h b/stepmania/src/ProfileHtml.h index 10bdf43386..8c1fd38052 100644 --- a/stepmania/src/ProfileHtml.h +++ b/stepmania/src/ProfileHtml.h @@ -13,8 +13,14 @@ #include "Profile.h" -void SavePlayerHtmlToDir( CString sDir, const Profile* pProfilePlayer, const Profile* pProfileMachine ); -void SaveMachineHtmlToDir( CString sDir, const Profile* pProfileMachine ); +enum HtmlType { HTML_TYPE_MACHINE, HTML_TYPE_PLAYER }; + +void SaveStatsWebPage( + CString sDir, + const Profile *pProfile, + const Profile *pMachineProfile, + HtmlType htmlType + ); #endif diff --git a/stepmania/src/ProfileManager.cpp b/stepmania/src/ProfileManager.cpp index f0173f3db7..9c1903c941 100644 --- a/stepmania/src/ProfileManager.cpp +++ b/stepmania/src/ProfileManager.cpp @@ -229,7 +229,7 @@ CString ProfileManager::GetPlayerName( PlayerNumber pn ) const { const Profile *prof = GetProfile( pn ); if( prof ) - return prof->m_sLastUsedHighScoreName; + return prof->GetDisplayName(); const char *names[NUM_PLAYERS] = { "PLAYER 1", "PLAYER 2" }; return names[pn]; diff --git a/stepmania/src/ScreenEvaluation.cpp b/stepmania/src/ScreenEvaluation.cpp index 249b1e3673..fae4323eb6 100644 --- a/stepmania/src/ScreenEvaluation.cpp +++ b/stepmania/src/ScreenEvaluation.cpp @@ -1310,7 +1310,7 @@ void ScreenEvaluation::Input( const DeviceInput& DeviceI, const InputEventType t Profile* pProfile = PROFILEMAN->GetProfile(pn); Profile::Screenshot screenshot; screenshot.sFileName = sFileName; - screenshot.sSignature = CRYPTMAN->GetFileSignature( sPath ); + screenshot.sMD5 = CRYPTMAN->GetMD5( sPath ); screenshot.highScore = m_HighScore[pn]; pProfile->AddScreenshot( screenshot ); } diff --git a/stepmania/src/StepMania.cpp b/stepmania/src/StepMania.cpp index 6088e90ac8..c2516ceb5c 100644 --- a/stepmania/src/StepMania.cpp +++ b/stepmania/src/StepMania.cpp @@ -1156,7 +1156,7 @@ CString SaveScreenshot( CString sDir, bool bSaveCompressed, bool bMakeSignature FlushDirCache(); if( bMakeSignature ) - CryptManager::SignFile( sPath ); + CryptManager::SignFileToFile( sPath ); return sFileName; }