[17.37] Players crashing server on join - new game crashing with import string

[Zacharum]

Standard (no mods) PVP scenario with custom world settings to get off the ground a little faster. Works fine for players already connected.

Noticed this when a new player joined my server. Server would crash with an output error.

The scenario level caused a non-recoverable error.
Please report this error to the scenario author.

Error while running event level::on_gui_click (ID 1)
Unknown entity name: player
stack traceback:
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:1388: in function 'set_player'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:902: in function '?'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3574: in function <.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3567>
stack traceback:
[C]: in function 'find_non_colliding_position'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:1388: in function 'set_player'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:902: in function '?'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3574: in function <.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3567>

Also, trying to start a new scenario with the exported string from the server crashes on attempting to start with a different error. I noticed that the seed option goes away when importing that string.

eyJnYW1lX2NvbmZpZyI6eyJ0aW1lX2xpbWl0IjowLCJhbGxvd19zcGVjdGF0b3JzIjp0cnVlLCJub19ydXNoX3RpbWUiOjAsImJhc2VfZXhjbHVzaW9uX3RpbWUiOjAsInJldmVhbF90ZWFtX3Bvc2l0aW9ucyI6dHJ1ZSwicmV2ZWFsX21hcF9jZW50ZXIiOnRydWUsInRlYW1fd2FsbHMiOnRydWUsInRlYW1fdHVycmV0cyI6dHJ1ZSwidHVycmV0X2FtbXVuaXRpb24iOnsib3B0aW9ucyI6WyJmaXJlYXJtLW1hZ2F6aW5lIiwicGllcmNpbmctcm91bmRzLW1hZ2F6aW5lIiwidXJhbml1bS1yb3VuZHMtbWFnYXppbmUiXSwic2VsZWN0ZWQiOiJmaXJlYXJtLW1hZ2F6aW5lIn0sInRlYW1fYXJ0aWxsZXJ5IjpmYWxzZSwiZ2l2ZV9hcnRpbGxlcnlfcmVtb3RlIjpmYWxzZSwiYXV0b19uZXdfcm91bmRfdGltZSI6MCwicHJvdGVjdF9lbXB0eV90ZWFtcyI6ZmFsc2UsImVuZW15X2J1aWxkaW5nX3Jlc3RyaWN0aW9uIjpmYWxzZSwibmV1dHJhbF9jaGVzdHMiOmZhbHNlfSwidGVhbV9jb25maWciOnsiZnJpZW5kbHlfZmlyZSI6dHJ1ZSwidW5sb2NrX2NvbWJhdF9yZXNlYXJjaCI6ZmFsc2UsImRlZmNvbl9tb2RlIjpmYWxzZSwibWF4X3BsYXllcnMiOjAsImRlZmNvbl90aW1lciI6NSwic3RhcnRpbmdfY2hlc3RfbXVsdGlwbGllciI6NSwicmVzZWFyY2hfbGV2ZWwiOnsib3B0aW9ucyI6WyJub25lIiwiYXV0b21hdGlvbi1zY2llbmNlLXBhY2siLCJsb2dpc3RpYy1zY2llbmNlLXBhY2siLCJtaWxpdGFyeS1zY2llbmNlLXBhY2siLCJjaGVtaWNhbC1zY2llbmNlLXBhY2siLCJwcm9kdWN0aW9uLXNjaWVuY2UtcGFjayIsInV0aWxpdHktc2NpZW5jZS1wYWNrIiwic3BhY2Utc2NpZW5jZS1wYWNrIl0sInNlbGVjdGVkIjoibm9uZSJ9LCJhdmVyYWdlX3RlYW1fZGlzcGxhY2VtZW50IjozMDAwLCJhbHdheXNfZGF5IjpmYWxzZSwiZXZvbHV0aW9uX2ZhY3RvciI6MCwiZHVwbGljYXRlX3N0YXJ0aW5nX2FyZWFfZW50aXRpZXMiOmZhbHNlLCJ0ZWNobm9sb2d5X3ByaWNlX211bHRpcGxpZXIiOjAuNX0sIm1vZGlmaWVyX2xpc3QiOnsiY2hhcmFjdGVyX21vZGlmaWVycyI6eyJjaGFyYWN0ZXJfcnVubmluZ19zcGVlZF9tb2RpZmllciI6MCwiY2hhcmFjdGVyX2hlYWx0aF9ib251cyI6MCwiY2hhcmFjdGVyX2NyYWZ0aW5nX3NwZWVkX21vZGlmaWVyIjowLCJjaGFyYWN0ZXJfbWluaW5nX3NwZWVkX21vZGlmaWVyIjowLCJjaGFyYWN0ZXJfYnVpbGRfZGlzdGFuY2VfYm9udXMiOjAsImNoYXJhY3Rlcl9yZWFjaF9kaXN0YW5jZV9ib251cyI6MH0sImZvcmNlX21vZGlmaWVycyI6eyJ3b3JrZXJfcm9ib3RzX3NwZWVkX21vZGlmaWVyIjowLCJ3b3JrZXJfcm9ib3RzX3N0b3JhZ2VfYm9udXMiOjAsIndvcmtlcl9yb2JvdHNfYmF0dGVyeV9tb2RpZmllciI6MCwibWluaW5nX2RyaWxsX3Byb2R1Y3Rpdml0eV9ib251cyI6MCwiaW5zZXJ0ZXJfc3RhY2tfc2l6ZV9ib251cyI6MCwic3RhY2tfaW5zZXJ0ZXJfY2FwYWNpdHlfYm9udXMiOjAsImxhYm9yYXRvcnlfc3BlZWRfbW9kaWZpZXIiOjAsImxhYm9yYXRvcnlfcHJvZHVjdGl2aXR5X2JvbnVzIjowLCJmb2xsb3dpbmdfcm9ib3RzX2xpZmV0aW1lX21vZGlmaWVyIjowLCJtYXhpbXVtX2ZvbGxvd2luZ19yb2JvdF9jb3VudCI6MCwidHJhaW5fYnJha2luZ19mb3JjZV9ib251cyI6MH0sInR1cnJldF9hdHRhY2tfbW9kaWZpZXIiOnsiZ3VuLXR1cnJldCI6MCwibGFzZXItdHVycmV0IjowLCJmbGFtZXRocm93ZXItdHVycmV0IjowLCJhcnRpbGxlcnktdHVycmV0IjowLCJzbWFsbC13b3JtLXR1cnJldCI6MCwibWVkaXVtLXdvcm0tdHVycmV0IjowLCJiaWctd29ybS10dXJyZXQiOjAsImJlaGVtb3RoLXdvcm0tdHVycmV0IjowfSwiYW1tb19kYW1hZ2VfbW9kaWZpZXIiOnsiYXJ0aWxsZXJ5LXNoZWxsIjowLCJiaW9sb2dpY2FsIjowLCJidWxsZXQiOjAsImNhbm5vbi1zaGVsbCI6MCwiY2Fwc3VsZSI6MCwiY29tYmF0LXJvYm90LWJlYW0iOjAsImNvbWJhdC1yb2JvdC1sYXNlciI6MCwiZWxlY3RyaWMiOjAsImZsYW1ldGhyb3dlciI6MCwiZ3JlbmFkZSI6MCwibGFuZG1pbmUiOjAsImxhc2VyLXR1cnJldCI6MCwibWVsZWUiOjAsInJhaWxndW4iOjAsInJvY2tldCI6MCwic2hvdGd1bi1zaGVsbCI6N30sImd1bl9zcGVlZF9tb2RpZmllciI6eyJhcnRpbGxlcnktc2hlbGwiOjAsImJpb2xvZ2ljYWwiOjAsImJ1bGxldCI6MCwiY2Fubm9uLXNoZWxsIjowLCJjYXBzdWxlIjowLCJjb21iYXQtcm9ib3QtYmVhbSI6MCwiY29tYmF0LXJvYm90LWxhc2VyIjowLCJlbGVjdHJpYyI6MCwiZmxhbWV0aHJvd2VyIjowLCJncmVuYWRlIjowLCJsYW5kbWluZSI6MCwibGFzZXItdHVycmV0IjowLCJtZWxlZSI6MCwicmFpbGd1biI6MCwicm9ja2V0IjowLCJzaG90Z3VuLXNoZWxsIjowfX0sInRlYW1zIjpbeyJuYW1lIjoiQWxwaGEiLCJjb2xvciI6InJlZCIsInRlYW0iOjEsIm1lbWJlcnMiOnt9fSx7Im5hbWUiOiJCcmF2byIsImNvbG9yIjoicHVycGxlIiwidGVhbSI6MSwibWVtYmVycyI6e319LHsibmFtZSI6IkNoYXJsaWUiLCJjb2xvciI6ImFjaWQiLCJtZW1iZXJzIjp7fSwidGVhbSI6MX0seyJuYW1lIjoiRGVsdGEiLCJjb2xvciI6InBpbmsiLCJtZW1iZXJzIjp7fSwidGVhbSI6MX1dLCJkaXNhYmxlZF9pdGVtcyI6e30sImludmVudG9yeV9saXN0Ijp7Im5vbmUiOnt9LCJzbWFsbCI6eyJpcm9uLXBsYXRlIjoyMDAsInBpcGUiOjEwMCwicGlwZS10by1ncm91bmQiOjIwLCJjb3BwZXItcGxhdGUiOjIwMCwiaXJvbi1nZWFyLXdoZWVsIjoyMDAsImVsZWN0cm9uaWMtY2lyY3VpdCI6MjAwLCJ0cmFuc3BvcnQtYmVsdCI6NDAwLCJyZXBhaXItcGFjayI6MjAsImluc2VydGVyIjoxMDAsInNtYWxsLWVsZWN0cmljLXBvbGUiOjUwLCJidXJuZXItbWluaW5nLWRyaWxsIjo1MCwic3RvbmUtZnVybmFjZSI6NTAsImJ1cm5lci1pbnNlcnRlciI6MTAwLCJhc3NlbWJsaW5nLW1hY2hpbmUtMSI6MjAsImVsZWN0cmljLW1pbmluZy1kcmlsbCI6MjAsImJvaWxlciI6NSwic3RlYW0tZW5naW5lIjoxMCwib2Zmc2hvcmUtcHVtcCI6Miwid29vZCI6NTB9LCJtZWRpdW0iOnsiaXJvbi1wbGF0ZSI6MjAwLCJwaXBlIjoxMDAsInBpcGUtdG8tZ3JvdW5kIjoyMCwiaXJvbi1nZWFyLXdoZWVsIjoxMDAsImNvcHBlci1wbGF0ZSI6MTAwLCJzdGVlbC1wbGF0ZSI6MTAwLCJlbGVjdHJvbmljLWNpcmN1aXQiOjQwMCwidHJhbnNwb3J0LWJlbHQiOjQwMCwidW5kZXJncm91bmQtYmVsdCI6MjAsInNwbGl0dGVyIjoyMCwicmVwYWlyLXBhY2siOjIwLCJpbnNlcnRlciI6MTUwLCJzbWFsbC1lbGVjdHJpYy1wb2xlIjoxMDAsIm1lZGl1bS1lbGVjdHJpYy1wb2xlIjo1MCwiZmFzdC1pbnNlcnRlciI6NTAsImxvbmctaGFuZGVkLWluc2VydGVyIjo1MCwiYnVybmVyLWluc2VydGVyIjoxMDAsImJ1cm5lci1taW5pbmctZHJpbGwiOjUwLCJlbGVjdHJpYy1taW5pbmctZHJpbGwiOjQwLCJzdG9uZS1mdXJuYWNlIjoxMDAsInN0ZWVsLWZ1cm5hY2UiOjMwLCJhc3NlbWJsaW5nLW1hY2hpbmUtMSI6NDAsImFzc2VtYmxpbmctbWFjaGluZS0yIjoyMCwiYm9pbGVyIjoxMCwic3RlYW0tZW5naW5lIjoyMCwiY2hlbWljYWwtcGxhbnQiOjIwLCJvaWwtcmVmaW5lcnkiOjUsInB1bXBqYWNrIjo4LCJvZmZzaG9yZS1wdW1wIjoyLCJ3b29kIjo1MH0sImxhcmdlIjp7Imlyb24tcGxhdGUiOjIwMCwicGlwZSI6MTAwLCJwaXBlLXRvLWdyb3VuZCI6MjAsImNvcHBlci1wbGF0ZSI6MjAwLCJzdGVlbC1wbGF0ZSI6MjAwLCJlbGVjdHJvbmljLWNpcmN1aXQiOjQwMCwiaXJvbi1nZWFyLXdoZWVsIjoyNTAsInRyYW5zcG9ydC1iZWx0Ijo0MDAsInVuZGVyZ3JvdW5kLWJlbHQiOjQwLCJzcGxpdHRlciI6NDAsInJlcGFpci1wYWNrIjoyMCwiaW5zZXJ0ZXIiOjIwMCwiYnVybmVyLWluc2VydGVyIjo1MCwic21hbGwtZWxlY3RyaWMtcG9sZSI6NTAsImJ1cm5lci1taW5pbmctZHJpbGwiOjUwLCJlbGVjdHJpYy1taW5pbmctZHJpbGwiOjUwLCJzdG9uZS1mdXJuYWNlIjoxMDAsInN0ZWVsLWZ1cm5hY2UiOjUwLCJlbGVjdHJpYy1mdXJuYWNlIjoyMCwiYXNzZW1ibGluZy1tYWNoaW5lLTEiOjUwLCJhc3NlbWJsaW5nLW1hY2hpbmUtMiI6NDAsImFzc2VtYmxpbmctbWFjaGluZS0zIjoyMCwiZmFzdC1pbnNlcnRlciI6MTAwLCJsb25nLWhhbmRlZC1pbnNlcnRlciI6MTAwLCJtZWRpdW0tZWxlY3RyaWMtcG9sZSI6NTAsInN1YnN0YXRpb24iOjEwLCJiaWctZWxlY3RyaWMtcG9sZSI6MTAsImJvaWxlciI6MTAsInN0ZWFtLWVuZ2luZSI6MjAsImNoZW1pY2FsLXBsYW50IjoyMCwib2lsLXJlZmluZXJ5Ijo1LCJwdW1wamFjayI6MTAsIm9mZnNob3JlLXB1bXAiOjIsIndvb2QiOjUwfX0sImVxdWlwbWVudF9saXN0Ijp7Im5vbmUiOnsiY29tYmF0LXNob3RndW4iOjEsInBpZXJjaW5nLXNob3RndW4tc2hlbGwiOjIwMDAsInBvd2VyLWFybW9yIjoxLCJzb2xhci1wYW5lbC1lcXVpcG1lbnQiOjM3LCJiYXR0ZXJ5LW1rMi1lcXVpcG1lbnQiOjIsInBlcnNvbmFsLXJvYm9wb3J0LW1rMi1lcXVpcG1lbnQiOjIsImNvbnN0cnVjdGlvbi1yb2JvdCI6NTAsInBpc3RvbCI6MSwiZmlyZWFybS1tYWdhemluZSI6MTB9LCJzbWFsbCI6eyJzdWJtYWNoaW5lLWd1biI6MSwiZmlyZWFybS1tYWdhemluZSI6MzAsInNob3RndW4iOjEsInNob3RndW4tc2hlbGwiOjIwLCJpcm9uLWF4ZSI6MSwiaGVhdnktYXJtb3IiOjF9LCJtZWRpdW0iOnsic3RlZWwtYXhlIjozLCJzdWJtYWNoaW5lLWd1biI6MSwiZmlyZWFybS1tYWdhemluZSI6NDAsInNob3RndW4iOjEsInNob3RndW4tc2hlbGwiOjIwLCJjYXIiOjEsIm1vZHVsYXItYXJtb3IiOjF9LCJsYXJnZSI6eyJzdGVlbC1heGUiOjMsInN1Ym1hY2hpbmUtZ3VuIjoxLCJwaWVyY2luZy1yb3VuZHMtbWFnYXppbmUiOjQwLCJjb21iYXQtc2hvdGd1biI6MSwicGllcmNpbmctc2hvdGd1bi1zaGVsbCI6MjAsInJvY2tldC1sYXVuY2hlciI6MSwicm9ja2V0Ijo4MCwiY29uc3RydWN0aW9uLXJvYm90IjoyNSwiY2FyIjoxLCJwb3dlci1hcm1vciI6MSwiZnVzaW9uLXJlYWN0b3ItZXF1aXBtZW50IjoxLCJleG9za2VsZXRvbi1lcXVpcG1lbnQiOjEsImVuZXJneS1zaGllbGQtZXF1aXBtZW50IjoyLCJwZXJzb25hbC1yb2JvcG9ydC1tazItZXF1aXBtZW50IjoxfX0sInZpY3RvcnkiOnsibGFzdF9zaWxvX3N0YW5kaW5nIjp0cnVlLCJzcGFjZV9yYWNlIjp0cnVlLCJyZXF1aXJlZF9yb2NrZXRzX3NlbnQiOjEsInByb2R1Y3Rpb25fc2NvcmUiOmZhbHNlLCJyZXF1aXJlZF9wcm9kdWN0aW9uX3Njb3JlIjo1MDAwMDAwMCwib2lsX2hhcnZlc3QiOmZhbHNlLCJyZXF1aXJlZF9vaWwiOjEwMDAwMDAwLCJraWxsX3Njb3JlIjpmYWxzZSwicmVxdWlyZWRfa2lsbF9zY29yZSI6MTAwMDAwMH0sInN0YXJ0aW5nX2VxdWlwbWVudCI6eyJvcHRpb25zIjpbIm5vbmUiLCJzbWFsbCIsIm1lZGl1bSIsImxhcmdlIl0sInNlbGVjdGVkIjoic21hbGwifSwic3RhcnRpbmdfY2hlc3QiOnsib3B0aW9ucyI6WyJub25lIiwic21hbGwiLCJtZWRpdW0iLCJsYXJnZSJdLCJzZWxlY3RlZCI6Im5vbmUifX0=

[Klonan]

Thanks for the report,
It should be fixed for the next release

Can you provide a save game? I can double check the fix, and also fix your save game so you can play right away

[Zacharum]

Oh, I thought I had posted the save in here. Here's a link to it as it doesn't seem to want to upload here..

https://drive.google.com/open?id=1_zKEx ... LzxlBlaCy8

Thank you so much!!

[Klonan]

Confirmed that its fixed in the next version

Here is a fixed version of your save:

https://drive.google.com/open?id=1R0iWZ ... DTAkKa1hUw

[Zacharum]

You are a god. Worked flawlessly!

[cuixun819]

First of all, the following text comes from the translator.

This problem still exists in 0.17.52. When I want to switch teams in the game, the game will collapse.

Can you tell me how to modify it? Thank you.



====================================

Error while running event level::on_gui_click (ID 1)
Character can only be connected to one player at a time.
stack traceback:
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:1405: in function 'set_player'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:905: in function '?'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3555: in function <.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3548>
stack traceback:
[C]: in function 'set_controller'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:1405: in function 'set_player'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:905: in function '?'
.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3555: in function <.../AppData/Roaming/Factorio/temp/currently-playing/pvp.lua:3548>

[Klonan]

First of all, the following text comes from the translator.

This problem still exists in 0.17.52. When I want to switch teams in the game, the game will collapse.

Can you tell me how to modify it? Thank you.

Seems it was broken by a recent change,
It have fixed the script in the next version