Index: server/DataAccess.cpp
===================================================================
--- server/DataAccess.cpp	(revision fb894b4932b2f5899183842a73fff58d358fcd15)
+++ server/DataAccess.cpp	(revision 84754c0e0363b46a820920d4cb87977c9c9ad439)
@@ -72,12 +72,11 @@
 }
 
-// this is no longer used anywhere
-int DataAccess::updatePlayer(string username, string password)
+int DataAccess::updatePlayer(Player* p)
 {
    ostringstream values, where;
 
-   values << "password='" << password << "'";
+   values << "level=" << p->level << ", experience=" << p->experience << ", honor=" << p->honor << ", wins=" << p->wins << ", losses=" << p->losses << "";
    
-   where << "name='" << username << "'";
+   where << "id=" << p->getId() << "";
 
    return update("users", values.str(), where.str());
@@ -114,13 +113,15 @@
          cout << "Class from db: " << atoi(row[3]) << endl;
       }
+      p->level = atoi(row[4]);
+      p->experience = atoi(row[5]);
+      p->honor = atoi(row[6]);
+      p->wins = atoi(row[7]);
+      p->losses = atoi(row[8]);
       cout << "Player class: " << p->playerClass << endl;
-      if (row[7] == NULL)
-          cout << "wins: NULL" << endl;
-      else
-          cout << "wins: " << atoi(row[7]) << endl;
-      if (row[8] == NULL)
-          cout << "losses: NULL" << endl;
-      else
-          cout << "losses: " << atoi(row[8]) << endl;
+      cout << "level: " << p->level << endl;
+      cout << "experience: " << p->experience << endl;
+      cout << "honor: " << p->honor << endl;
+      cout << "wins: " << p->wins << endl;
+      cout << "losses: " << p->losses << endl;
       cout << "Loaded player from db" << endl;
    }else {
@@ -140,4 +141,5 @@
 list<Player*>* DataAccess::getPlayers()
 {
+   // This method doesn't seem to ever get used. Decide whether it's actually needed
    MYSQL_RES *result;
    MYSQL_ROW row;
@@ -154,4 +156,6 @@
       cout << row[0] << ", " << row[1] << ", " << row[2] << endl;
       lstPlayers->push_back(new Player(row[1], row[2]));
+
+      // need to assign all the other db values to the player
    }
 
@@ -310,4 +314,5 @@
    query_state = mysql_query(connection, oss.str().c_str());
 
+
    if (query_state != 0) {
       cout << mysql_error(connection) << endl;
Index: server/DataAccess.h
===================================================================
--- server/DataAccess.h	(revision fb894b4932b2f5899183842a73fff58d358fcd15)
+++ server/DataAccess.h	(revision 84754c0e0363b46a820920d4cb87977c9c9ad439)
@@ -20,8 +20,5 @@
    bool verifyPassword(string encrypted, string password);
    int insertPlayer(string username, string password, Player::PlayerClass playerClass);
-   // this method needs to be more rebust. maybe pass in a player object amd
-   // the method could use the player id to find the player and update any
-   // attributes that changed
-   int updatePlayer(string username, string password);
+   int updatePlayer(Player* player);
 
    int* getPlayerRecord(int playerId);