Foohy / jazztronauts

Jazztronauts is a cooperative theft-em-up for Garry's Mod, also known as prop hunt 2.
122 stars 35 forks source link

Add multiplayer conditional for dialogue #167

Closed Commenter25 closed 1 year ago

Commenter25 commented 1 year ago

Closes #98

I'm not sure if there's any other situations where the text specifically addresses multiple players. I think everything else in the intro is pretty neutral. But if anyone knows anything, I'll change it as well.

SageJFox commented 1 year ago

Maybe a bit of a "I bought this waffle press so I'm gonna get my money's worth out of it" scenario, but, in the intro, the Bartender has the line

You'll want to fasten yourself securely in your seat, and brace yourself.

Could add a multiplayer check and add an alternate of

You'll want to fasten yourselves securely in your seats, and brace yourselves.

Commenter25 commented 1 year ago

Can't test it at the moment like I normally would, but this should be good! I also want to mention: yes, I absolutely fucking hate the whole added function just for an if check; but it seems Lua doesn't have a real ternary operator to make it compact, so I felt like squishing it into one line just kinda looks awful:

multiplayer = function() if player.GetCount() > 1 then return true else return false end end
SageJFox commented 1 year ago

Couldn't you do this?

multiplayer = function() return player.GetCount() > 1 end

Or is it too early for me to be thinking just yet. x'd

Commenter25 commented 1 year ago

god why didnt i think of that, you're probably right. I remember trying return player.GetCount() - 1, and then I realized Lua doesn't consider 0 as false and I'm just JS-pilled xd

I'm still like, trying to hyperfocus on studying, but I can probably squeeze in testing that real quick later. Out of curiosity, do you know where the game save is stored, so I can back that up?

SageJFox commented 1 year ago

Y'know, your original attempt would technically be more accurate (since it'd make it "0 players" rather than "0 player", not that that should ever show up) You could just wrap your logic in an isbool(), that converts 0 to false and any other number to true.

As for the save data, that's stored in SQL, which will be sv.db and cl.db in your root folder, but keep in mind that those are shared with every addon. Why are you concerned with the save data for this?

Foohy commented 1 year ago

I think the other prologue PR #161 touched some of the same files as this and caused some conflicts, but I reviewed the change on its own and looks all good to me, very useful. I wouldn't want to make assumptions but fixing the conflicts, but I'll merge as soon as those are all ready 2 rumble

Commenter25 commented 1 year ago

oh christ what did it do trying to fix the conflicts

i think im just going to reconstruct this in a new branch