logical error - death and paycheck

[ghost]

Expected behaviour

when dead not getting a paycheck but instead the text "you missed a paycheck"

Actual behaviour

still getting a paycheck as by how "death" is implemented and checked

Steps to reproduce the behaviour

should be obvious - but k

1. die / get killed
2. wait up to (paycheck time here) before respawn
3. see you still get your paycheck even when dead

RPT and/or extDB3 logs (if applicable)

not applicable

Mission version: 6.0.0

Launcher version: 1.5.146373

Game version: 1.98.146373

Branch: main / beta branch not specified

Ok - so a bit more although I guess all active devs know the issue:
The way "death" is implement is to copy the player as a corpse and respawn at the spawn island. The issue: "isAlive player" returns true as the player isn't dead but only a fixed camera is set on the corpse.
Possible solution: Set a flag on the player (isn't such already done?) and check it instead of isAlive to check if the player currently is considered "dead" or "alive".

[Edward-Bakker]

I'll make a PR first thing when I wake up. Good spot 👍

[Edward-Bakker]

Is this also a thing in v5?

[ghost]

Is this also a thing in v5?

tbh I neither know nor care for anything older than master branch v6 (as my goal is to get a clean v6 release - and quite a lot has to be fixed before release - which I highly doubt will - when I compare my branch with this repo github already says "diff too long - wont load") - and aside from paycheck was changed between v5 and v6 I would give it a wild guess in the blue: most likely - but have to be checked/confirmed for v5 - if anyone is still into it and if there's a v5.x release planed (otherwise I don'T know why v5 is kept alive (and default) when there's a lot of work done on v6)

[imthatguyhere]

why v5 is kept alive (and default)

Because people pull from the repo and leaving a dev branch as default caused issues previously with early adoption. Dealing with supporting something that is rapidly changing isn't great fun.

[imthatguyhere]

Is this also a thing in v5?

It seems to be:

Framework/Altis_Life.Altis/core/fsm/client.fsm

Line 60 in 7569490

action=/*%FSM<ACTION""">*/"if (!alive player) then {" \n

[ghost]

why v5 is kept alive (and default)

Because people pull from the repo and leaving a dev branch as default caused issues previously with early adoption. Dealing with supporting something that is rapidly changing isn't great fun.

just for the sake of try to keep it within limits: can you tell me any difference making 5.x.x not a dev branch?

[Jason2605]

5.x.x is the branch for the v5 release of AL. The only work that should be going onto that branch are bug fixes. As explained above, we have had issues in the past when leaving a branch that is not ready for release, and is rapidly changing (v6 in this case), being the default branch on GitHub as people begin to use this as their base. This is not what we want, and hence why it has been setup this way.

Getting back on topic of this issue - if there is a bug report created, it would be nice to confirm if this is an issue affecting previous versions as well (v5) so that when we get around to creating a patch for the issue, it can also be applied to the v5 branch if required.

[ghost]

Da die meisten hier nach ihren profilen eh Deutsche sind bring ichs einfach mal auf den Punkt: Das Problem warum hier nichts voran kommt ist eure Engstirnigkeit. Ich hab mich letztes Wochenende (oder wars das davor?) und jede einzelne zeile code durchgetestet - jede Funktion einzeln ausprobiert - und bin dabei teils auf Code von 2013 gestoßen. Auf meinen PR hab ich nur "joar, toll, schön für dich, interessiert uns aber nich" bekommen. Warum? Weils vermutlich auch einfach keinen interessiert. Nicht nur, dass das Projekt Altis Life ziemlich tot ist (liegt halt daran dass nur noch wenige überhaupt ArmA3 zocken) gibt es auch das große Problem, dass viele "Admins" (oder besser: diejenigen die sich hier das Geraffel ziehen und n Server aufsetzen) lieber irgendwelche fixes in ihre mission basteln nach dem Motto "hauptsache mein Server läuft" anstatt hier mal ordentlich Meldung zu machen das ALLE was von haben.
Und die paar Hansel die sich dann doch mal hier her verirren und mal melden, was alles grundlegend so teils überhaupt nicht funzt, die sollen sich dann noch die Mühe machen und v5 auch noch testen obwohl die Code-Basis die gleiche ist?

Ach kommt - hört doch auf. Entweder ihr wollt das hier am Leben halten (naja, oder besser: wiederbeleben?) - dann setzt euch auch gott verdammt noch mal hin wie ich mit n paar Freunden oder zumindest nem zweiten Rechner und zweiten Account und testet eure Bastelein BEVOR ihr es public pusht - oder lasst es einfach und gebt eure "eigentlich haben wir da ja nich mehr so wirklich bock drauf" Einstellung zu und gebt es an jemanden ab der den Willen hat dass pro-aktiv durchzuziehen.

Soll jetzt zwar nich heißen, dass ich der Kandidat dafür wäre - aber ich mach mir im Gegensatz zu euch wenigstens die Mühe und probier das aus was ich rumgepfuscht hab bevor ich n commit public pushe. So die letzten paar PRs die mehr kaputt gemacht statt gefixt haben zeigen ja, dass offenbar nur wild drauf los ge-auto-refactored wurde ohne sich Gedanken drüber zu machen - oder es danach wenigstens mal lokal auszuprobieren - dann wäre es aufgefallen das in der Wanted-Liste plötzlich STR_Crime_187 statt Manslaughter - wiel eben an genau der Stelle das "localize" nun mal weiterhin erforderlich ist.
Ich hab echt gedacht ich könnte mal den Stein hier etwas weiterschieben und was gegen diese offenbar vorherschende Problematik, dass jeder Admin nur an seiner eigenen Mission was rumbastelt, machen. Ende vom Lied: Ich bin auch zu so einem mutiert - weil hier stoß ich ja eh nur auf Gegenwehr bei Vorschlägen einige grundlegende und tiefsitzende Probleme mal in Angriff zu nehmen ... Warum bin ich eigentlich vier Mal wieder zurück gekommen? Ich weiß es nich ... aber es wäre trotzdem zumindest HÖFLICH gewesen, wenn ich meine issues schon aus trotz dicht mache - die auch bitte closed zu lassen und nicht wie anders leider auch täglich üblich mit der necromancing-Keule rumzuschwingen - aber das ist eher ein Problem der Platform GitHub selbst die halt Repo-Maintainern mehr Rechte über fremde Inhalte einräumt ... will wissen was der Typ der den Ausfall zu verantworten hat geraucht hat - muss derb gutes Zeug sein ...