Skip to content
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

New Component - Add safe start #160

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

New Component - Add safe start #160

wants to merge 6 commits into from

Conversation

Andx667
Copy link
Contributor

@Andx667 Andx667 commented Jan 14, 2025

PULL REQUEST

When merged this pull request will:

  • title
  • senkt die Waffe bei Missionsbeginn
  • optionale Client Einstellung: sichert die Waffe des Spielers zu Missionsbeginn

von https://github.com/ArmaForces/Mods/tree/master/addons/safestart übernommen

IMPORTANT

  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Remove {changes}.

@Andx667
Copy link
Contributor Author

Andx667 commented Jan 14, 2025

wollen wir das? würde MB etwas entlasten da die Waffe nicht von MB gesichert werden muss sondern würde das in die Hand der spieler legen.

@Andx667 Andx667 self-assigned this Jan 14, 2025
@Andx667 Andx667 added this to the 3.2.0 milestone Jan 14, 2025
@@ -2,6 +2,21 @@
#include "\x\cba\addons\xeh\script_xeh.hpp"
#include "\z\ace\addons\main\script_macros.hpp"

// AFM macros

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that the same "afm" then the function tag? You replaced the function tag with "ttt" but not this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFM ist die community von der ich das habe. Wollte es hier als kleine Referenz auf die Quelle drin lassen. Aber dann sollte der kommentar am besten etwas ergänzt werden damit das klar wird.


if (is3DEN) exitWith {};

[QGVAR(enableSafety), player] call CBA_fnc_localEvent;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I understand correctly, the safety event only gets called once here in the client postInit, that is called after the normal init, no?
This works in most cases, but to be 100% sure wouldn't it be better to call the event on player respawn? With that we also eliminate the possibilities of accidents in missions with respawns or a training.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ja das ist eine gute Idee, das ganze (zusätzlich) beim Player Respawn auszuführen.

@Hexomanya
Copy link

Ich würde es wenn da bevorzugen, wenn der Missionsbauer die Möglichkeit hätte, dass den Spielern aufzuzwingen.
Sonst macht das wieder kaum einer, außer es passiert einem mal was und dann ist eh zu spät. Und ich sehe keinen validen Grund warum jemand was dagegen haben könnte, seine Waffe zu Beginn der Mission gesichert zu bekommen.

@Andx667
Copy link
Contributor Author

Andx667 commented Jan 15, 2025

Ich würde es wenn da bevorzugen, wenn der Missionsbauer die Möglichkeit hätte, dass den Spielern aufzuzwingen. Sonst macht das wieder kaum einer, außer es passiert einem mal was und dann ist eh zu spät. Und ich sehe keinen validen Grund warum jemand was dagegen haben könnte, seine Waffe zu Beginn der Mission gesichert zu bekommen.

das sollte sich machen lassen.

Wir hatten mal eine Diskussion darüber ob Waffen zu Beginn gesichert sein sollen. Da kam das Argument, dass ja in Echt, auch jeder für seine eigene Waffe zuständig ist. Aber für den Spielspaß und den Spielfluss wäre es besser das einheitlich aufzuzwingen.

@Andx667
Copy link
Contributor Author

Andx667 commented Jan 15, 2025

Unerwartetes Problem:
unsere Art der Loadoutzuweisung (über die onPlayerRespawn) funktioniert mit dieser Art nicht. Vermutlich wird der EH gerufen wenn die neue Waffe noch nicht zugewiesen ist.

@Andx667 Andx667 removed this from the 3.2.0 milestone Jan 17, 2025
@Andx667 Andx667 removed their assignment Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants