Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

pls improve map validation tool (upload vs server pathing)

Aberta
  • Algo errado?
#364 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
32/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
java
Domínio
api, backend

Direção de pesquisa

Comece pelo endpoint /maps/validate e compare os resultados da validação dos caminhos map-name e scenario.lua para as versões enviadas e criadas pelo servidor. Confirme a convenção de pastas esperada com o fluxo de envio e, em seguida, defina como os dois modos devem ser selecionados e o que significam resultados de validação e comportamento de envio consistentes.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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 ;)

Linguagem predominante
Java
Estrelas
31
Forks
30
Nenhum PR com merge em 30d

Preparar o ambiente

  • Inclui um Dockerfile ou arquivo Docker Compose
  • Sem modelo de pull request
  • Sem guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de FAForever/faf-java-api

Todas as issues de FAForever/faf-java-api

Issues semelhantes

Mais issues de Java

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.