getLine is not recognizing the last line

[Subpixel]

This was working last night, but it decided to stop working for some reason. The problem is inside the for loop, when it echoes  the page I see the last line perfectly fine.

Code: [Select]

function ZapkCRPG_HTTP::onDisconnect(%this)
{
%page = %this.page;
echo(%page);
for(%a = 1; %a < getLineCount(%page); %a++)
{
%line = getLine(%page, %a);
if(firstWord(%line) $= "<latestVersion>")
{
$ZapkCRPG_HTTP_latestVersion = restWords(%line);
//echo("\c4Downloaded Version:" SPC $ZapkCRPG_Version);
//echo("\c4Latest Version:" SPC $ZapkCRPG_HTTP_latestVersion);
}
if(firstWord(%line) $= "<releaseDate>")
{
$ZapkCRPG_HTTP_releaseDate = restWords(%line);
//echo("\c4Update Released:" SPC $ZapkCRPG_HTTP_releaseDate);
}
if(firstWord(%line) $= "<downloadLink>")
{
$ZapkCRPG_HTTP_downloadLink = restWords(%line);
//echo("\c4Download Link:" SPC $ZapkCRPG_HTTP_downloadLink);
}
if(firstWord(%line) $= "<rtbID>")
{
$ZapkCRPG_HTTP_rtbID = restWords(%line);
//echo("\c4RTB ID:" SPC $ZapkCRPG_HTTP_rtbID);
}
}
if($ZapkCRPG_Version != $ZapkCRPG_HTTP_latestVersion)
{
%mcText = "A new version (" @ $ZapkCRPG_HTTP_latestVersion @ ") was released on" SPC $ZapkCRPG_HTTP_releaseDate @ ".";
clientCmdZapkCRPG_mc("Update Available!", %mcText);
if(isObject(RTBMM_TransferQueue) && $ZapkCRPG_HTTP_rtbID !$= "")
{
//Thanks Greek2Me for the notification code <3
if(isObject(RTBCC_NotificationManager))
{
RTBCC_NotificationManager.push("Zapk's City RPG", "Downloading client updates.", "new", "", 5000);
}
RTB_ModManager.setVisible(1);
RTBMM_TransferView_Init();
RTBMM_TransferQueue.addItem($ZapkCRPG_HTTP_rtbID);
}
}
else
{
echo("\c4Client up-to-date.");
}
}When I try to echo $ZapkCRPG_HTTP_rtbID, it doesn't return anything. Obviously my for loop isn't recognizing the last line. When I force it to go higher, it says Index out of rande (0-3, 4), when there is obviously a fifth line there.
Oh, and here's the page it's reading off of.
http://pastebin.com/raw.php?i=jmvwzy58
It isn't recognizing the <rtbID> line inside the for loop.

[Ad Bot]

[@trinick]

Are you intentionally skipping the first line (Line 0)? Otherwise, just check if %a is less than or equal to (<=) getLineCount(%page) to read another line.

[Subpixel]

Are you intentionally skipping the first line (Line 0)? Otherwise, just check if %a is less than or equal to (<=) getLineCount(%page) to read another line.

Yeah, line 0 is always blank.

[@trinick]

I don't have a copy of Blockland so I can't actually crack it open and test this for you, but what you're saying doesn't make sense. If echo shows 5 lines and getLineCount() returns 4 and refuses to read a line over 3, then you've discovered a bug and you'll have to find a new way to do it. Personally I think you should just parse it onLine(%this, %line) rather than parsing it after it disconnects. Then you don't have to deal with getLine, or storing the data, at all.

[Subpixel]

I don't have a copy of Blockland so I can't actually crack it open and test this for you, but what you're saying doesn't make sense. If echo shows 5 lines and getLineCount() returns 4 and refuses to read a line over 3, then you've discovered a bug and you'll have to find a new way to do it. Personally I think you should just parse it onLine(%this, %line) rather than parsing it after it disconnects. Then you don't have to deal with getLine, or storing the data, at all.

Hm...
Will try this.