Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

pls improve map validation tool (upload vs server pathing)

Offen
  • Stimmt etwas nicht?
#364 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
32/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
java
Bereich
api, backend

Rechercherichtung

Beginne am Endpunkt /maps/validate und vergleiche die Validierungsergebnisse für die Pfade map-name und scenario.lua bei hochgeladenen und serverseitig erstellten Versionen. Bestätige die erwartete Ordnerkonvention anhand des Upload-Ablaufs und lege anschließend fest, wie beide Modi ausgewählt werden sollen und was konsistente Validierungsergebnisse und ein konsistentes Upload-Verhalten bedeuten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

First of all, I love our new validation tool: https://api.faforever.com/maps/validate
But I found some Issues that should be improved...

Validate Map Name
The map name validation for Adaptive Map says adaptive_map.v0001 for the folder.

  • That folder name and path is created while map upload and in FAF Editor

  • server created version = /maps/adaptive_map.v0001/adaptive_map.scmap

Validate scenario.lua
But in the validation of the obviously wrong path in the scenario content, it gives you a folder path without .v0001 /maps/adaptive_map/adaptive_map.scmap

  • As far as I know, that path is created in GPG Editor and needed to be applied for map upload (without .v0001 in folder and file path)? Please correct me if I'm wrong.

  • upload version = /maps/adaptive_map/adaptive_map.scmap

#0

Conclusion
FAF Editor version = server created version = /maps/adaptive_map.v0001/adaptive_map.scmap
GPG Editor version = upload version = /maps/adaptive_map/adaptive_map.scmap

That's why I think, the tool is confusing how it is now and leads to maps not being able to load. It would be nice to clarify if an upload version or an server created version should be validated.
It would be nice to give both options to a map maker. And validate consistently over name and scenario validation, like in picture 2#. A checkbox could be a solution to enable the map maker to get the right information. Because I really like to prevent issues like this while uploading:
foldername - scenario_error
foldername - script_error

Bad example
1# difference in validated name and scenario folder naming
#1

Good example
2# example for constant folder naming with .v0001 like in the server created version
#2

One additional question
Isn't it possible to also allow an upload with the version number in the folder?
That would be awesome. This way there is no difference between GPG Editor and FAF Editor maps and both can simply be uploaded. No broken files due to upload process ;)

Vorherrschende Sprache
Java
Sterne
31
Forks
30
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

  • Enthält ein Dockerfile oder eine Docker-Compose-Datei
  • Keine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus FAForever/faf-java-api

Alle Issues in FAForever/faf-java-api

Ähnliche Issues

Weitere Issues zu Java

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.