Skip to content

Conversation

@KoenigMjr
Copy link
Contributor

Bedingt durch die modeunabhängige Verarbeitung sämtlicher Wildcards warnt das Programm, dass manche Felder nicht verfügbar sind, die in der derzeitigen Auswertung jedoch gar nicht vorhanden sein können (z.B. FMS Felder in ZVEI-Datensatz etc.).
Auf Loglevel "WARNING" oder darunter kommt es dann zu Fehlern wie: [WARNING ] field not found: fms bei allen Datensätzen, die eben nicht FMS entsprechen (und beinhalten).

Änderungen:

  • Wildcard-Ersetzung in replaceWildcards() nach Paketmodus (fms, pocsag, zvei, msg) aufgeteilt
  • Unnötige Wildcards werden nun abhängig vom Modus nicht mehr verarbeitet
  • {MSG} wird nun explizit für den Modus msg unterstützt
  • Kopfzeile mit aktuellem Änderungsdatum versehen

Sry für die vielen Pull Requests. Allerdings wenn ich schon Änderungen im Code mache, die mir für die breite Masse sinnig erscheinen, dann stelle ich sie eben auf diesem Wege der Allgemeinheit zur Verfügung.
Und ich denke themenbezogene PRs sind besser wie ein Großer mit Allen; macht in meinen Augen die Veränderungen leichter zum reviewen.

Schönes Wochenende!

…nabhängige Verarbeitung sämtlicher Wildcards warnt das Programm, dass manche Felder nicht verfügbar sind, die in der derzeitigen Auswertung jedoch gar nicht vorhanden sein können (z.B. FMS Felder in ZVEI-Datensatz etc.)

Änderungen:
- Wildcard-Ersetzung in `replaceWildcards()` nach Paketmodus (fms, pocsag, zvei, msg) aufgeteilt
- Unnötige Wildcards werden nun abhängig vom Modus nicht mehr verarbeitet
- `{MSG}` wird nun explizit für den Modus `msg` unterstützt
- Kopfzeile mit aktuellem Änderungsdatum versehen
@KoenigMjr KoenigMjr force-pushed the bugfix/fieldnotfound branch from e6900da to f53102d Compare August 8, 2025 10:01
@KoenigMjr KoenigMjr marked this pull request as ready for review August 8, 2025 10:03
@wattenre
Copy link

wattenre commented Aug 8, 2025

Ich finde dein Engagement top! Danke dir. Ich wünschte ich hätte momentan auch mehr Zeit um solch coolen Änderungen umzusetzen.

@Schrolli91
Copy link
Member

Schrolli91 commented Aug 8, 2025

Sry für die vielen Pull Requests. Allerdings wenn ich schon Änderungen im Code mache, die mir für die breite Masse sinnig erscheinen, dann stelle ich sie eben auf diesem Wege der Allgemeinheit zur Verfügung.

Genau so ist das auch gedacht - Ich danke Dir für deine Unterstützung.
Ich selbst habe leider aktuell auch nicht die Zeit das Projekt aktiv voran zu treiben.

Und ja - viele keine PRs lassen sich besser zwischen rein Reviewen, wenn man eigentlich eh keine Zeit hat ;-)

Copy link
Member

@Schrolli91 Schrolli91 left a comment

Choose a reason for hiding this comment

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

LGTM

@Schrolli91 Schrolli91 merged commit a1cb545 into BOSWatch:develop Aug 8, 2025
7 checks passed
@KoenigMjr KoenigMjr deleted the bugfix/fieldnotfound branch August 8, 2025 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants