Super Mario Bros. X Forums

Hoeloe wrote:The "corrupted" thing clearly comes from the download stopping early, meaning you only have half the file (hence the "unexpected end of file" error). Why it's doing that I'm not sure, but since you are the only person to have this issue, I'm pretty sure it's not the file that is the problem.

Phazon1111 wrote:I'm trying to make it that when NPC 178 (The SMB1 axe) is collected, 30 seconds is added to the level's timer in similar fashion to the speed run levels from SMG2/SM3DL/SM3DW
via leveltimer.lua.

Code: Select all

timerApi = loadAPI("leveltimer");

function onLoad()  
	--But now in the onLoad override, we set a few values
	timerApi.setSecondsLeft(30);
 --For one, we need to set the timer value. No greater than 999
	timerApi.setTimerState(true);
 --Then, we need to make sure we enable it. It's not enabled by default
end


function onNPCKill(eventObj,npcID,killReason)
	if npcID == 178 then
		getSecondsLeft()
		timerApi.setSecondsLeft(timerApi.secondsleft + 10)
	end
end

No error messages pop up upon starting the level or collect the axe.

Intuition wrote:Is that all the code there is? There is no "getSecondsLeft" function anywhere in there. We also don't know what is it that you get. Does it not increase? Does the increased amount not match what you want (which judging by the code you posted seems to be the case, it's increasing 10 and not 30)?

Enjl wrote:

Phazon1111 wrote:I'm trying to make it that when NPC 178 (The SMB1 axe) is collected, 30 seconds is added to the level's timer in similar fashion to the speed run levels from SMG2/SM3DL/SM3DW
via leveltimer.lua.

Code: Select all

timerApi = loadAPI("leveltimer");

function onLoad()  
	--But now in the onLoad override, we set a few values
	timerApi.setSecondsLeft(30);
 --For one, we need to set the timer value. No greater than 999
	timerApi.setTimerState(true);
 --Then, we need to make sure we enable it. It's not enabled by default
end


function onNPCKill(eventObj,npcID,killReason)
	if npcID == 178 then
		getSecondsLeft()
		timerApi.setSecondsLeft(timerApi.secondsleft + 10)
	end
end

No error messages pop up upon starting the level or collect the axe.

Many things wrong:
-getSecondsLeft is a function from leveltimer so you have to prefix it with timerApi. like you did for the others
-npcID is a misnomer because the variable contains the killed npc. I prefer to call it killedNPC. You can cet the ID with killedNPC.id (or right now npcID.id which is silly)
-you should seriously use onStart in favour of onLoop. The latter has been deprecated for over a year.
-you sould make the timerApi variable local so only this file can access it.
-loadAPI has been deprecated for over a year and replaced with API.load. Use that.

MosaicMario wrote:

Code: Select all

function onMessageBox(eventObj, msgStr)
   if msgStr = "CutsceneTalk1" then

function onMessageBox(eventObj, msgStr)
   if msgStr = "CutsceneTalk2" then
   
function onMessageBox(eventObj, msgStr)
	if msgStr = "CutsceneTalk3" then

function onMessageBox(eventObj, msgStr)
   if msgStr = "CutsceneTalk4" then	

Hoeloe wrote:

MosaicMario wrote:

Code: Select all

function onMessageBox(eventObj, msgStr)
   if msgStr = "CutsceneTalk1" then

function onMessageBox(eventObj, msgStr)
   if msgStr = "CutsceneTalk2" then
   
function onMessageBox(eventObj, msgStr)
	if msgStr = "CutsceneTalk3" then

function onMessageBox(eventObj, msgStr)
   if msgStr = "CutsceneTalk4" then	

Quick question:

When you display a message box, how do you think the code determines which of these it has to run? It can't be that if statement, because that only occurs after it's already started to run the function.

Basically, you can't have multiple definitions of the same function. You should be using elseif statements rather than trying to redefine the function.

function onMessageBox(eventObj, msgStr)
   if msgStr == "CutsceneTalk1" then
      eventObj.cancelled === true
      Text.showMessageBox("MosaicMario: Great job guys! We made it through the fortress!")
      Text.showMessageBox("Rosy: I wonder why there were so many ninjis in there?")
      elseif msgStr == "CutsceneTalk2" then
      eventObj.cancelled === true
      Text.showMessageBox("Hoops: Hey guys, look at this bridge! Where could it lead?")
      Text.showMessageBox("Berry: Yoshi!(I don't know, but i'm curious too!)")
      Text.showMessageBox("MosaicMario: Hoops, I think that maybe you should slow dow-")
      Text.showMessageBox("Hoops: Last one to the other side's a rotton mushroom!")
      elseif msgStr == "CutsceneTalk3" then
      eventObj.cancelled === true
      Text.showMessageBox("MosaicMario: Hoops! Wait up!")
      elseif msgStr == "CutsceneTalk4" then
      eventObj.cancelled === true
      Text.showMessageBox("Rosy:...Well, what do you say Berry? Should we catch up with them?")
      Text.showMessageBox("Berry: Yoshi!(Yeah, sure!)")
      Text.showMessageBox("Rosy: Let's go catch them i guess.")
   end
end