-
Notifications
You must be signed in to change notification settings - Fork 310
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
logical error - death and paycheck #720
Comments
I'll make a PR first thing when I wake up. Good spot 👍 |
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) |
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. |
It seems to be:
|
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? |
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. |
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. 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. |
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
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".
The text was updated successfully, but these errors were encountered: