Dialogs compilation bug

Ashlebede

New Member
Reaction score
43
In my first attempt to use vJASS, I got a compilation error when I used globals of type [ljass]dialog[/ljass].

JASS:
library publics initializer init
    globals
        //////////
        //DIALOG//
        //////////
        dialog MainDialog
        
        button array Main_btn [3]
        
        button array btn_Mode [2]
        button array btn_Settings [5]
        button array btn_Teams [4]
        button array btn_Repicks [2]
        button array btn_Shrines[2]
        button array btn_SH [4]
        
        /////////
        //MODES//
        /////////
        boolean AP = true
        boolean shrinesAllowed = true
        boolean repicksAllowed = true
        
        integer SH = 0
        integer Teams = 0
        
        ////////
        //MISC//
        ////////
        force array Team [2]
    endglobals
    
    private function init takes nothing returns nothing
        set MainDialog = DialogCreate()
    endfunction
endlibrary

function Trig_Dialog_Main_Actions takes nothing returns nothing
    call DialogSetMessage(MainDialog,"Main Menu")
    set Main_btn[0]=DialogAddButton(MainDialog,"Te|cffFFFFFFa|rms",65)
    set Main_btn[1]=DialogAddButton(MainDialog,"S|cffFFFFFFe|rttings",69)
    set Main_btn[2]=DialogAddButton(MainDialog,"|cffFFFFFFS|rtart Game",83)
    
    call DialogDisplay(Player(0),MainDialog,true)
endfunction

//===========================================================================
function InitTrig_Dialog_Main takes nothing returns nothing
    local trigger trg = CreateTrigger(  )
    
    call TriggerRegisterTimerEventSingle( trg, 0.10 )
    call TriggerAddAction( trg, function Trig_Dialog_Main_Actions )
    
    set trg = null
endfunction


Now, for the line which reads [ljass]call DialogSetMessage(MainDialog,"Main Menu")[/ljass] and every line after that which references to my [ljass]MainDialog[/ljass] variable, I get this error :

Line X : Cannot convert integer to dialog

Pretty logical, since the code now looks like this :

JASS:
    call DialogSetMessage((1), "Main Menu")


Now, is there something wrong with my World Editor or am I just doing it wrong? I had a fully working trigger when I didn't use vJASS, but I kind of figured I should use it if I was going to transfer the script from a map to another (which I did). Now, that took me hours to code and I kind of hate that CTRL+H find & replace... the popup can't be dragged, and all... (I didn't post the whole code)
 
Right, like I'm coding vJASS without using JassHelper... >_>

See, the thing is, it reads like this :

JASS:
call DialogSetMessage(MainDialog,"Main Menu")


Proper syntax, as far as I know. Maindialog is of type [ljass]dialog[/ljass], in theory. However, when I save my map, it gives me the error I told you. It says it's an integer now. And I can see that the line, in the error popup, looks like this :


JASS:
call DialogSetMessage((1), "Main Menu")


Now, my other globals work well, so that's not what's wrong. Also, every time I use the variable MainDialog outside its library, it becomes [ljass](1)[/ljass] after compilation... my other variables work just fine.

I also tried making MainDialog an udg, which works (though I have to do find & replace). For now I'll keep it like that, but I'd just like to know why a [ljass]dialog[/ljass] variable can become [ljass](1)[/ljass]... everywhere in the code...
 
Do you have a function or struct somewhere else in your code called MainDialog?

Also, he said latest JassHelper, not JassHelper that came with Newgen Pack.
 
Also, put this into your map and try saving.

JASS:

//! zinc
  library DoIHaveZINC{
  }
//! endzinc


If you get an error, then 1) You don't have ZINC support, and 2) Your JassHelper is probably the NewGen one and you should really download the latest JassHelper and update the [LJASS]common.j[/LJASS] and [LJASS]blizzard.j[/LJASS] in your JassHelper folder.
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • The Helper The Helper:
    News portal has been retired. Main page of site goes to Headline News forum now
  • The Helper The Helper:
    I am working on getting access to the old news portal under a different URL for those that would rather use that for news before we get a different news view.
  • Ghan Ghan:
    Easily done
    +1
  • The Helper The Helper:
    https://www.thehelper.net/pages/news/ is a link to the old news portal - i will integrate it into the interface somewhere when i figure it out
  • Ghan Ghan:
    Need to try something
  • Ghan Ghan:
    Hopefully this won't cause problems.
  • Ghan Ghan:
    Hmm
  • Ghan Ghan:
    I have converted the Headline News forum to an Article type forum. It will now show the top 20 threads with more detail of each thread.
  • Ghan Ghan:
    See how we like that.
  • The Helper The Helper:
    I do not see a way to go past the 1st page of posts on the forum though
  • The Helper The Helper:
    It is OK though for the main page to open up on the forum in the view it was before. As long as the portal has its own URL so it can be viewed that way I do want to try it as a regular forum view for a while
  • Ghan Ghan:
    Yeah I'm not sure what the deal is with the pagination.
  • Ghan Ghan:
    It SHOULD be there so I think it might just be an artifact of having an older style.
  • Ghan Ghan:
    I switched it to a "Standard" article forum. This will show the thread list like normal, but the threads themselves will have the first post set up above the rest of the "comments"
  • The Helper The Helper:
    I don't really get that article forum but I think it is because I have never really seen it used on a multi post thread
  • Ghan Ghan:
    RpNation makes more use of it right now as an example: https://www.rpnation.com/news/
  • The Helper The Helper:
  • The Helper The Helper:
    What do you think Tom?
  • tom_mai78101 tom_mai78101:
    I will have to get used to this.
  • tom_mai78101 tom_mai78101:
    The latest news feed looks good
  • The Helper The Helper:
    I would like to see it again like Ghan had it the first time with pagination though - without the pagination that view will not work but with pagination it just might...
  • The Helper The Helper:
    This drink recipe I have had more than a few times back in the day! Mind Eraser https://www.thehelper.net/threads/cocktail-mind-eraser.194720/

      The Helper Discord

      Staff online

      Members online

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top