Ticket #41572

Move action actions and enabler to separate file

Eröffnet am: 2021-02-14 10:14 Letztes Update: 2023-01-23 13:14

Auswertung:
Verantwortlicher:
Typ:
Status:
Geschlossen
Komponente:
Meilenstein:
Priorität:
5 - Mittel
Schweregrad:
5 - Mittel
Lösung:
Gefixt
Datei:
Keine

Details

in the bundled rulesets and include the file. The action stuff is starting to totally dominate game.ruleset

Consider doing this for rulesaving too

---
Individual tickets to implement:

- #45037 (done)
- #45039 (closed)
- #45064 (done)
- #45065 (done)
- #46427 (done)
- #45066 (done)

Ticket-Verlauf (3/13 Historien)

2021-02-14 10:14 Aktualisiert von: kvilhaugsvik
  • New Ticket "Move action actions and enabler to separate file" created
2021-09-26 11:24 Aktualisiert von: cazfi
  • Meilenstein Update from (Keine) to 3.2.0
  • Komponente Update from Rulesets to General
Kommentar

Reply To kvilhaugsvik

in the bundled rulesets and include the file.

My plan would be to introduce new enablers.ruleset, not just split game.ruleset.

2022-07-07 00:05 Aktualisiert von: dark-ether
Kommentar

this seems necessary for custom actions so i am planning on doing this ticket, just to be sure so that i don't forget anything, this includes moving action related configs and actions enablers to a actions.ruleset , and updating the packaged rulesets to work with this change and then going to the wiki and documenting this change or is updating the doc directory sufficient? also just to be sure but right now no one is currently working on this right? it doesn't show an owner for this ticket.

2022-07-07 01:30 Aktualisiert von: cazfi
Kommentar

Great!

This is a bit too big thing for a single ticket. I'll split this to some steps -> start by #45037

2022-07-07 03:12 Aktualisiert von: cazfi
  • Details Updated
2022-07-07 04:14 Aktualisiert von: dark-ether
Kommentar

Reply To cazfi

Great! This is a bit too big thing for a single ticket. I'll split this to some steps -> start by #45037

hmm,so the thing is, i am kinda impatient so i started work on this as soon as i commented, and as it is just copy and pasting code, so i have done a lot of progress, around 12000 lines of change(almost all by copy pasting from the game.ruleset to the new file actions.ruleset file for each ruleset so 6000 "deleted' and 6000 "added") , and have added a new function on the ruleset.c file, but i haven't touched any part about build or installation,and haven't thought at all about compatibility, i have no idea what do next as i have kinda broken the logical sequence of steps. can i send the part i already did here, then you can tell me what i should do next?

2022-07-07 04:27 Aktualisiert von: cazfi
Kommentar

Reply To dark-ether

(almost all by copy pasting from the game.ruleset to the new file actions.ruleset file for each ruleset so 6000 "deleted' and 6000 "added")

That copy+paste doesn't sound like a big task to redo when it's the correct time to do it.

I'd recommend that you keep your current patch around as a reference, and something where you can copy parts from, but that you start now from scratch in the order that things can actually be merged (we can't break the things even during the intermediate steps)

2022-07-08 16:37 Aktualisiert von: cazfi
  • Details Updated
2022-12-05 05:04 Aktualisiert von: cazfi
  • Details Updated
2022-12-24 22:51 Aktualisiert von: cazfi
  • Details Updated
2022-12-24 22:51 Aktualisiert von: cazfi
  • Details Updated
2023-01-03 13:08 Aktualisiert von: cazfi
  • Details Updated
2023-01-23 13:14 Aktualisiert von: cazfi
  • Status Update from Offen to Geschlossen
  • Verantwortlicher Update from (Keine) to cazfi
  • Lösung Update from Keine to Gefixt
  • Details Updated

Dateianhangliste

Keine Anhänge

Bearbeiten

Please login to add comment to this ticket » Anmelden