Fix various issues with subscript errors (#1345)

* Fix various issues with subscript errors

* Missed PoEAPI change

Author: Wires77

src/Classes/ImportTab.lua

@@ -160,11 +160,11 @@ local ImportTabClass = newClass("ImportTab", "ControlHost", "Control", function(
 	self.controls.exportFrom:SelByValue(self.exportWebsiteSelected or main.lastExportWebsite or "Pastebin", "id")
 	self.controls.generateCodeByLink = new("ButtonControl", { "LEFT", self.controls.exportFrom, "RIGHT"}, {8, 0, 100, 20}, "Share", function()
 		local exportWebsite = exportWebsitesList[self.controls.exportFrom.selIndex]
-		local response = buildSites.UploadBuild(self.controls.generateCodeOut.buf, exportWebsite)
-		if response then
+		local subScriptId = buildSites.UploadBuild(self.controls.generateCodeOut.buf, exportWebsite)
+		if subScriptId then
 			self.controls.generateCodeOut:SetText("")
 			self.controls.generateCodeByLink.label = "Creating link..."
-			launch:RegisterSubScript(response, function(pasteLink, errMsg)
+			launch:RegisterSubScript(subScriptId, function(pasteLink, errMsg)
 				self.controls.generateCodeByLink.label = "Share"
 				if errMsg then
 					main:OpenMessagePopup(exportWebsite.id, "Error creating link:\n"..errMsg)

src/Classes/PoEAPI.lua

@@ -80,9 +80,10 @@ function PoEAPIClass:FetchAuthToken(callback)
 	if id then
 		launch.subScripts[id] = {
 			type = "DOWNLOAD",
-			callback = function(code, state, port)
+			callback = function(response, errMsg)
+				local code, state, port = unpack(response or {})
 				if not code then
-					ConPrintf("Failed to get code from server")
+					ConPrintf("Failed to get code from server: %s", errMsg)
 					self.authToken = nil
 					self.refreshToken = nil
 					self.tokenExpiry = nil

src/Launch.lua

@@ -299,14 +299,18 @@ function launch:DownloadPage(url, callback, params)
 			errMsg = "No data returned"
 		end
 		ConPrintf("Download complete. Status: %s", errMsg or "OK")
-		return responseHeader, responseBody, errMsg
+		return {responseHeader, responseBody}, errMsg
 	]]
 	local id = LaunchSubScript(script, "", "ConPrintf", url, params.header, params.body, self.connectionProtocol, self.proxyURL)
 	if id then
 		self.subScripts[id] = {
 			type = "DOWNLOAD",
-			callback = function(responseHeader, responseBody, errMsg)
-				callback({header=responseHeader, body=responseBody}, errMsg)
+			callback = function(response, errMsg)
+				if response then
+					callback({header=response[1], body=response[2]}, errMsg)
+				else
+					callback({}, errMsg)
+				end
 			end
 		}
 	end

src/LaunchServer.lua

@@ -197,4 +197,4 @@ while (os.time() < stopAt) and shouldRetry do
 	attempt = attempt + 1
 end
 server:close()
-return code, state, port
+return {code, state, port}