[Snippet] PreventSave

[TriggerHappy]

PreventSave - Disable Game Saving

library PreventSave initializer onInit
/***************************************************************
*
*   v1.0.1 by TriggerHappy
*   ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯
*   This library allows you to enable or disable game saving. It works by showing
*   a dialog instantly before a game is saved, causing the save to interrupt. There
*   are no known side effects.
*   _________________________________________________________________________
*   1. Installation
*   ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯
*   Copy the script to your map and save it (requires JassHelper *or* JNGP)
*   _________________________________________________________________________
*   2. API
*   ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯
*       function SaveGameEnable takes boolean flag returns nothing
*       function IsSaveEnabled takes nothing returns boolean
*       function PreventSave takes player p, boolean flag returns nothing
*
***************************************************************/
  
    globals
        boolean GameAllowSave = false
    endglobals
  
    //====================================
    // Do not edit below this line
    //====================================
  
    globals
        private dialog Dialog = DialogCreate()
        private timer  Timer  = CreateTimer()
        private player LocalPlayer
    endglobals
  
    function SaveGameEnable takes boolean flag returns nothing
        set GameAllowSave = flag
    endfunction
  
    function IsSaveEnabled takes nothing returns boolean
        return GameAllowSave
    endfunction
  
    function PreventSave takes player p, boolean flag returns nothing
        if (p == LocalPlayer) then
            call SaveGameEnable(not flag)
        endif
    endfunction
  
    private function Exit takes nothing returns nothing
        call DialogDisplay(LocalPlayer, Dialog, false)
    endfunction
  
    private function StopSave takes nothing returns boolean
        if not IsSaveEnabled() then
            call DialogDisplay(LocalPlayer, Dialog, true)
        endif
        call TimerStart(Timer, 0.00, false, function Exit)
        return false
    endfunction
  
    private function onInit takes nothing returns nothing
        local trigger t = CreateTrigger()
        set LocalPlayer = GetLocalPlayer()
      
        call TriggerRegisterGameEvent(t, EVENT_GAME_SAVE)
        call TriggerAddCondition(t, Filter(function StopSave))
    endfunction

endlibrary

 

[azlier]

Efficient, and well tested. I myself can verify that it works on both single player and online.

Approved.

 

[Mooglefrooglian]

Every map should have an option to use this, to prevent laming.

Thank you!

Edit: Wait... why are you setting a global variable to GetLocalPlayer()? I'm pretty sure that desyncs.

library PreventSave initializer onInit

    globals
        boolean GameAllowSave = false
    endglobals

//====================================
// Do not edit below this line
//====================================

    globals
        private dialog D = DialogCreate()
        private timer T = CreateTimer()
    endglobals

    private function Exit takes nothing returns nothing
        call DialogDisplay(GetLocalPlayer(), D, false)
    endfunction

    private function StopSave takes nothing returns nothing
        if not GameAllowSave then
            call DialogDisplay(GetLocalPlayer(), D, true)
            call TimerStart(T, 0.00, false, function Exit)
        endif
    endfunction

    private function onInit takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerRegisterGameEvent(t, EVENT_GAME_SAVE)
        call TriggerAddAction(t, function StopSave)
    endfunction

endlibrary

Shouldnt it be like that?

 

[azlier]

No. Setting a global to the local player works just fine.

 

[Mooglefrooglian]

But dont globals sync, and having them different cause a desync?

Is it just when you try to use that data?

 

[azlier]

Globals are not synced.

 

[kennyman94]

I have no clue about what you guys are talking about but what version of jass is this for?

 

[TriggerHappy]

Do you mean vJass? Blizzard doesn't give Jass patch versions it's just Jass2. If you do mean vJass, just get the newest Jasshelper or download the GUI one in the spells section (check my profile).

 

[kennyman94]

I mean like i just got JNGP and i dont know if different editor versions use different forms of jass.

 

[cookies4you]

finally! i've been looking for something to prevent this! finally! a way to stop seekers from saving and exploiting moving props!

 

[Fanny.Shaver]

Taken from TheHelper and not even given credits...

 

[dead-man-walking]

Er.... Proof?

 

[Weep]

Taken from TheHelper and not even given credits...

Wrong - this was posted by its author.

Why is there no facepalm emote?

 

[Fanny.Shaver]

Er.... Proof?

Proof, is that I have it on my map for about six months...

@up

Well, I knew someone did that on thehelper, didn't knew who did. Strange that long later in here. Honors back.

 

[Troll-Brain]

I would suggest this :

//==================================== // PreventSave
//====================================
//
//   This library allows you to enable or disable game saving. 
//   It works by showing a dialog instantly before a game is saved and this causes the dialog
//   to close. It does nothing visually to the game as far as I've noticed.
//
//   You can toggle allowing or disallowing saving by setting
//   GameAllowSave to true, or false.
//
//====================================
// Import Instructions
//====================================
//
//      Decide whether you want GUI or vJASS, then copy the trigger(s) over to your map.
//
//      This requires JassHelper which is included
//      in JassNewGenPack.
//
//====================================

library PreventSave initializer onInit
    
    globals
        boolean GameAllowSave = false
    endglobals
    
//====================================
// Do not edit below this line
//====================================
    
    globals
        private dialog D = DialogCreate()
        private timer T  = CreateTimer()
        private player p
    endglobals
    
    private function Exit takes nothing returns nothing
        call DialogDisplay(p, D, false)
    endfunction
    
    private function StopSave takes nothing returns nothing
        if not GameAllowSave then
            call DialogDisplay(p, D, true)
        endif
        call TimerStart(T, 0.00, false, function Exit)
    endfunction
    
    private function onInit takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerRegisterGameEvent(t, EVENT_GAME_SAVE)
        call TriggerAddAction(t, function StopSave)
        set p = GetLocalPlayer()
    endfunction

endlibrary

Yes it will be "less efficient" but without breaking backward compatibility you will allow to enable or not saving game for players.
Indeed, we will be able to play with GetLocalPlayer and the boolean.
If you have no clue of what i'm talking about, check this :
http://www.hiveworkshop.com/forums/jass-resources-412/snippet-localhelper-215108/

Btw, what happens if we have already a dialog displayed ?

​

 

[TriggerHappy]

I would suggest this :
Yes it will be "less efficient" but without breaking backward compatibility you will allow to enable or not saving game for players.
Indeed, we will be able to play with GetLocalPlayer and the boolean.
If you have no clue of what i'm talking about, check this :
http://www.hiveworkshop.com/forums/jass-resources-412/snippet-localhelper-215108/

Disabling/enabling saving for just one player still causes the game to be saved for everybody. At least it did in my (LAN) tests.
It also displays the message "You were disconnected" to the other players.

scope example initializer onInit
    private function onInit takes nothing returns nothing
        if (GetLocalPlayer() == Player(0)) then
            set GameAllowSave = true
        endif
    endfunction
endscope

Btw, what happens if we have already a dialog displayed ?

The script still blocks the save.

 

[Troll-Brain]

I mean allow or not the player X to try to save.
It shouldn't cause a desync with my edit of your code, or it does ?
The timer starts for everybody, and displaying a dialog in a local block should be fine.

 

[TriggerHappy]

I mean allow or not the player X to try to save.
It shouldn't cause a desync with my edit of your code, or it does ?
The timer starts for everybody, and displaying a dialog in a local block should be fine.

Yeah, with your edit it doesn't desync.
I was also confused because I saw the load file on both games, so I assumed it created the save file for all players but obviously it's just on my hard drive

I'll update the script in a bit, thanks.

 

[TriggerHappy]

Updated.

• Using a boolexpr instead of trigger action.
• PreventSave function added (read doc).
• Added support for disabling for specific players.

 

[chobibo]

call TriggerAddCondition(t, function StopSave)
call TriggerAddCondition(t, Filter(function StopSave) )

 

[Troll-Brain]

Filter() is optional in this case IIRC, it's a vJass feature : implicit boolexpr
It will add it automatically for the resulted jass code.

So now just to be sure,
if you use PreventSave(true) with player red, then when he will try to save the game, it will not be saved on each computer, regardless if you use PreventSave(false) for some other players ?

 

[chobibo]

Oh, I see.

 

[TriggerHappy]

call TriggerAddCondition(t, function StopSave)
call TriggerAddCondition(t, Filter(function StopSave) )

Filter is not needed because the function is private, meaning it will only be passed to through Filter() once.

The point of creating a boolexpr like that is that it gets cached for re-use and this function will never be re-used.

So in this case it's just a wasted function call.

So now just to be sure,
if you use PreventSave(true) with player red, then when he will try to save the game, it will not be saved on each computer, regardless if you use PreventSave(false) for some other players ?

I can probably test this later, though I will need two computers because afaik this can't be tested locally, even on LAN since the save file is created on the HDD.

 

[chobibo]

I didn't know it doesn't need it, I thought it would produce an error, sorry about that.

 

[TriggerHappy]

I didn't know it doesn't need it, I thought it would produce an error, sorry about that.

Actually, I didn't know implicit boolexpr was a vJASS feature.
So yeah, it would throw an error normally.

In that case forget my last post

 

[Troll-Brain]

If it doesn't work as intended, instead of editing the boolean in a local block you could use an other array one and GetTriggerPlayer() or something like that.
If a such native response event doesn't exit, then maybe we have a specific player event for that ?

 

[Troll-Brain]

Bump for possible issues since the last year ^^
So, how about the tests ?

 

[edo494]

regarding the TriggerAddCondition, I think you should put the function into Filter block even if it is done implicitly, because if some newbie reads it, he may try to do the same thing later on, and be surprised why it wont work(if it wont get Conditioned implicitly), also Filter > Condition

 

[TriggerHappy]

If somebody could test over multiple computers that would help.

/* 
If Player(1) saves, then it shows the "Saving..." dialog 
and a save file is created. No dialog pops up for Player(0).
*/

call PreventSave(Player(1), false)
call PreventSave(Player(0), true)

So it should be working however I'm testing on a single computer and the save file will always show up.

 

[TriggerHappy]

Updated, v1.0.1

• Wrapped StopSave in Filter.
• Updated API with two functions, SaveGameEnable and IsSaveEnabled.
• Updated documentation.