1
0

Chat history limit on the WebAdmin chat

Did some refactoring on the JavaScript side of the WebAdmin chat
Got rid of cRoot::sRootState
Fixed a crash on the WebAdmin when a page is loaded before a world exists

git-svn-id: http://mc-server.googlecode.com/svn/trunk@775 0a769ca7-a7f5-676a-18bf-c427514a06d6
This commit is contained in:
faketruth 2012-08-22 11:22:26 +00:00
parent 492cbe87ee
commit e086b931ea
4 changed files with 104 additions and 99 deletions

View File

@ -1,84 +1,91 @@
local CHAT_HISTORY = 50
local JavaScript = [[ local JavaScript = [[
<script type="text/javascript"> <script type="text/javascript">
function createXHR() function createXHR()
{ {
var request = false; var request = false;
try { try {
request = new ActiveXObject('Msxml2.XMLHTTP'); request = new ActiveXObject('Msxml2.XMLHTTP');
} }
catch (err2) { catch (err2) {
try { try {
request = new ActiveXObject('Microsoft.XMLHTTP'); request = new ActiveXObject('Microsoft.XMLHTTP');
} }
catch (err3) { catch (err3) {
try { try {
request = new XMLHttpRequest(); request = new XMLHttpRequest();
} }
catch (err1) catch (err1) {
{
request = false; request = false;
} }
} }
}
return request;
} }
return request;
}
function loadWholePage( url ) function OpenPage( url, callback )
{
var xhr = createXHR();
xhr.onreadystatechange=function()
{
if (xhr.readyState == 4)
{
callback( xhr )
}
};
xhr.open("GET", url , true);
xhr.send(null);
}
function LoadPageInto( url, storage )
{
OpenPage( url, function( xhr )
{ {
var storage = document.getElementById('ChatDiv'); var ScrollBottom = storage.scrollTop + storage.offsetHeight;
var xhr = createXHR(); var bAutoScroll = (ScrollBottom >= storage.scrollHeight); // Detect whether we scrolled to the bottom of the div
xhr.onreadystatechange=function()
{
if(xhr.readyState == 4)
{
//alert( xhr.status + " " + xhr.statusText );
//if(xhr.status == 200)
{
storage.innerHTML = xhr.responseText;//getBody(xhr.responseText);
storage.scrollTop = storage.scrollHeight;
}
}
};
xhr.open("GET", url , true);
xhr.send(null);
return false; storage.innerHTML = xhr.responseText;
}
function SendChatMessage()
{
var MessageContainer = document.getElementById('ChatMessage');
if( MessageContainer.value == "" ) return;
var xhr = createXHR(); if( bAutoScroll == true )
xhr.onreadystatechange=function() {
{ storage.scrollTop = storage.scrollHeight;
if(xhr.readyState == 4) }
{ } );
//alert( xhr.status + " " + xhr.statusText );
RefreshChat();
}
};
xhr.open("GET", "/~webadmin/Core/Chat/?ChatMessage=" + MessageContainer.value, true);
xhr.send(null);
MessageContainer.value = "";
}
function RefreshChat() return false;
}
function SendChatMessage()
{
var MessageContainer = document.getElementById('ChatMessage');
if( MessageContainer.value == "" ) return;
OpenPage( "/~webadmin/Core/Chat/?ChatMessage=" + MessageContainer.value, function( xhr )
{ {
loadWholePage('/~webadmin/Core/Chat/?JustChat=true'); RefreshChat();
} } );
MessageContainer.value = "";
setInterval(RefreshChat, 1000); }
window.onload = RefreshChat;
function RefreshChat()
</script> {
LoadPageInto('/~webadmin/Core/Chat/?JustChat=true', document.getElementById('ChatDiv'));
}
setInterval(RefreshChat, 1000);
window.onload = RefreshChat;
</script>
]] ]]
local ChatLogMessages = {} local ChatLogMessages = {}
function AddMessage( PlayerName, Message ) function AddMessage( PlayerName, Message )
table.insert( ChatLogMessages, { name = PlayerName, message = Message } ) table.insert( ChatLogMessages, { name = PlayerName, message = Message } )
while( #ChatLogMessages > CHAT_HISTORY ) do
table.remove( ChatLogMessages, 1 )
end
end end
function OnChat( Player, Message ) function OnChat( Player, Message )
@ -95,6 +102,7 @@ function HandleRequest_Chat( Request )
end end
if( Request.Params["ChatMessage"] ~= nil ) then if( Request.Params["ChatMessage"] ~= nil ) then
LOG("Chat msg: " .. Request.Params["ChatMessage"] )
local Message = "[WebAdmin]: " .. Request.Params["ChatMessage"] local Message = "[WebAdmin]: " .. Request.Params["ChatMessage"]
cRoot:Get():GetServer():SendMessage( Message ) cRoot:Get():GetServer():SendMessage( Message )
AddMessage("WebAdmin", Request.Params["ChatMessage"] ) AddMessage("WebAdmin", Request.Params["ChatMessage"] )

View File

@ -33,30 +33,23 @@
cRoot* cRoot::s_Root = 0; cRoot* cRoot::s_Root = 0;
typedef std::map< AString, cWorld* > WorldMap;
struct cRoot::sRootState
{
cWorld* pDefaultWorld;
WorldMap WorldsByName;
};
cRoot::cRoot() cRoot::cRoot()
: m_Server( 0 ) : m_Server( NULL )
, m_MonsterConfig( 0 ) , m_MonsterConfig( NULL )
, m_GroupManager( 0 ) , m_GroupManager( NULL )
, m_CraftingRecipes(NULL) , m_CraftingRecipes(NULL)
, m_FurnaceRecipe( 0 ) , m_FurnaceRecipe( NULL )
, m_WebAdmin( 0 ) , m_WebAdmin( NULL )
, m_PluginManager( 0 ) , m_PluginManager( NULL )
, m_Log( 0 ) , m_Log( NULL )
, m_bStop( false ) , m_bStop( false )
, m_bRestart( false ) , m_bRestart( false )
, m_InputThread( 0 ) , m_InputThread( NULL )
, m_pState( new sRootState ) , m_pDefaultWorld( NULL )
{ {
s_Root = this; s_Root = this;
} }
@ -68,7 +61,6 @@ cRoot::cRoot()
cRoot::~cRoot() cRoot::~cRoot()
{ {
s_Root = 0; s_Root = 0;
delete m_pState;
} }
@ -232,8 +224,8 @@ void cRoot::LoadWorlds(void)
// First get the default world // First get the default world
AString DefaultWorldName = IniFile.GetValueSet("Worlds", "DefaultWorld", "world"); AString DefaultWorldName = IniFile.GetValueSet("Worlds", "DefaultWorld", "world");
m_pState->pDefaultWorld = new cWorld( DefaultWorldName.c_str() ); m_pDefaultWorld = new cWorld( DefaultWorldName.c_str() );
m_pState->WorldsByName[ DefaultWorldName ] = m_pState->pDefaultWorld; m_WorldsByName[ DefaultWorldName ] = m_pDefaultWorld;
// Then load the other worlds // Then load the other worlds
unsigned int KeyNum = IniFile.FindKey("Worlds"); unsigned int KeyNum = IniFile.FindKey("Worlds");
@ -256,7 +248,7 @@ void cRoot::LoadWorlds(void)
continue; continue;
} }
cWorld* NewWorld = new cWorld( WorldName.c_str() ); cWorld* NewWorld = new cWorld( WorldName.c_str() );
m_pState->WorldsByName[ WorldName ] = NewWorld; m_WorldsByName[ WorldName ] = NewWorld;
} // for i - Worlds } // for i - Worlds
} }
@ -266,7 +258,7 @@ void cRoot::LoadWorlds(void)
void cRoot::StartWorlds(void) void cRoot::StartWorlds(void)
{ {
for( WorldMap::iterator itr = m_pState->WorldsByName.begin(); itr != m_pState->WorldsByName.end(); ++itr ) for( WorldMap::iterator itr = m_WorldsByName.begin(); itr != m_WorldsByName.end(); ++itr )
{ {
itr->second->InitializeSpawn(); itr->second->InitializeSpawn();
} }
@ -278,7 +270,7 @@ void cRoot::StartWorlds(void)
void cRoot::StopWorlds(void) void cRoot::StopWorlds(void)
{ {
for( WorldMap::iterator itr = m_pState->WorldsByName.begin(); itr != m_pState->WorldsByName.end(); ++itr ) for( WorldMap::iterator itr = m_WorldsByName.begin(); itr != m_WorldsByName.end(); ++itr )
{ {
itr->second->StopThreads(); itr->second->StopThreads();
} }
@ -290,11 +282,12 @@ void cRoot::StopWorlds(void)
void cRoot::UnloadWorlds(void) void cRoot::UnloadWorlds(void)
{ {
for( WorldMap::iterator itr = m_pState->WorldsByName.begin(); itr != m_pState->WorldsByName.end(); ++itr ) m_pDefaultWorld = NULL;
for( WorldMap::iterator itr = m_WorldsByName.begin(); itr != m_WorldsByName.end(); ++itr )
{ {
delete itr->second; delete itr->second;
} }
m_pState->WorldsByName.clear(); m_WorldsByName.clear();
} }
@ -303,7 +296,7 @@ void cRoot::UnloadWorlds(void)
cWorld* cRoot::GetDefaultWorld() cWorld* cRoot::GetDefaultWorld()
{ {
return m_pState->pDefaultWorld; return m_pDefaultWorld;
} }
@ -312,8 +305,8 @@ cWorld* cRoot::GetDefaultWorld()
cWorld* cRoot::GetWorld( const AString & a_WorldName ) cWorld* cRoot::GetWorld( const AString & a_WorldName )
{ {
WorldMap::iterator itr = m_pState->WorldsByName.find( a_WorldName ); WorldMap::iterator itr = m_WorldsByName.find( a_WorldName );
if( itr != m_pState->WorldsByName.end() ) if( itr != m_WorldsByName.end() )
return itr->second; return itr->second;
return 0; return 0;
} }
@ -324,7 +317,7 @@ cWorld* cRoot::GetWorld( const AString & a_WorldName )
bool cRoot::ForEachWorld(cWorldListCallback & a_Callback) bool cRoot::ForEachWorld(cWorldListCallback & a_Callback)
{ {
for (WorldMap::iterator itr = m_pState->WorldsByName.begin(), itr2 = itr; itr != m_pState->WorldsByName.end(); itr = itr2) for (WorldMap::iterator itr = m_WorldsByName.begin(), itr2 = itr; itr != m_WorldsByName.end(); itr = itr2)
{ {
++itr2; ++itr2;
if (a_Callback.Item(itr->second)) if (a_Callback.Item(itr->second))
@ -341,7 +334,7 @@ bool cRoot::ForEachWorld(cWorldListCallback & a_Callback)
void cRoot::TickWorlds( float a_Dt ) void cRoot::TickWorlds( float a_Dt )
{ {
for( WorldMap::iterator itr = m_pState->WorldsByName.begin(); itr != m_pState->WorldsByName.end(); ++itr ) for( WorldMap::iterator itr = m_WorldsByName.begin(); itr != m_WorldsByName.end(); ++itr )
{ {
itr->second->Tick( a_Dt ); itr->second->Tick( a_Dt );
} }
@ -390,7 +383,7 @@ void cRoot::AuthenticateUser(int a_ClientID)
int cRoot::GetTotalChunkCount(void) int cRoot::GetTotalChunkCount(void)
{ {
int res = 0; int res = 0;
for ( WorldMap::iterator itr = m_pState->WorldsByName.begin(); itr != m_pState->WorldsByName.end(); ++itr ) for ( WorldMap::iterator itr = m_WorldsByName.begin(); itr != m_WorldsByName.end(); ++itr )
{ {
res += itr->second->GetNumChunks(); res += itr->second->GetNumChunks();
} }
@ -403,7 +396,7 @@ int cRoot::GetTotalChunkCount(void)
void cRoot::SaveAllChunks(void) void cRoot::SaveAllChunks(void)
{ {
for (WorldMap::iterator itr = m_pState->WorldsByName.begin(); itr != m_pState->WorldsByName.end(); ++itr) for (WorldMap::iterator itr = m_WorldsByName.begin(); itr != m_WorldsByName.end(); ++itr)
{ {
itr->second->SaveAllChunks(); itr->second->SaveAllChunks();
} }
@ -415,7 +408,7 @@ void cRoot::SaveAllChunks(void)
bool cRoot::ForEachPlayer(cPlayerListCallback & a_Callback) bool cRoot::ForEachPlayer(cPlayerListCallback & a_Callback)
{ {
for (WorldMap::iterator itr = m_pState->WorldsByName.begin(), itr2 = itr; itr != m_pState->WorldsByName.end(); itr = itr2) for (WorldMap::iterator itr = m_WorldsByName.begin(), itr2 = itr; itr != m_WorldsByName.end(); itr = itr2)
{ {
++itr2; ++itr2;
if (!itr->second->ForEachPlayer(a_Callback)) if (!itr->second->ForEachPlayer(a_Callback))
@ -437,7 +430,7 @@ void cRoot::LogChunkStats(void)
int SumNumInLighting = 0; int SumNumInLighting = 0;
int SumNumInGenerator = 0; int SumNumInGenerator = 0;
int SumMem = 0; int SumMem = 0;
for (WorldMap::iterator itr = m_pState->WorldsByName.begin(), end = m_pState->WorldsByName.end(); itr != end; ++itr) for (WorldMap::iterator itr = m_WorldsByName.begin(), end = m_WorldsByName.end(); itr != end; ++itr)
{ {
cWorld * World = itr->second; cWorld * World = itr->second;
int NumInGenerator = World->GetGeneratorQueueLength(); int NumInGenerator = World->GetGeneratorQueueLength();

View File

@ -102,8 +102,9 @@ private:
bool m_bStop; bool m_bStop;
bool m_bRestart; bool m_bRestart;
struct sRootState; typedef std::map< AString, cWorld* > WorldMap;
sRootState* m_pState; cWorld* m_pDefaultWorld;
WorldMap m_WorldsByName;
cThread* m_InputThread; cThread* m_InputThread;
static void InputThread(void* a_Params); static void InputThread(void* a_Params);

View File

@ -237,8 +237,11 @@ void cWebAdmin::Request_Handler(webserver::http_request* r)
cPlayerAccum PlayerAccum; cPlayerAccum PlayerAccum;
cWorld * World = cRoot::Get()->GetDefaultWorld(); // TODO - Create a list of worlds and players cWorld * World = cRoot::Get()->GetDefaultWorld(); // TODO - Create a list of worlds and players
World->ForEachPlayer(PlayerAccum); if( World != NULL )
Content.append(PlayerAccum.m_Contents); {
World->ForEachPlayer(PlayerAccum);
Content.append(PlayerAccum.m_Contents);
}
Content += "</ul><br>"; Content += "</ul><br>";
} }