feat(trackmania): MapTable module

[SobakaPirat]

Summary

Сomplete rework of how the maps are shown and how they are stored for Trackmania wiki
The problems this module solves #8120

LPDB

• All maps are saved to the global tournament_maps variable so they are not overwritten when the module is reinvoked
• The tournament_maps are stored in the maps field of tournament_<tournament_name> (the wiki variable set by the infobox, the page title is used as fallback)

Tournament information -> Maps

[
  {
    "map": "You shall not pass",
    "author1": {
      "flag": "United Kingdom",
      "page": "Pac",
      "displayName": "Pac"
    },
    "author2": {
      "flag": "Germany",
      "page": "Piranha_(German)",
      "displayName": "Piranha"
    },
    "author3": {
      "flag": "Canada",
      "page": "Carl_Jr.",
      "displayName": "CarlJr"
    },
    "links": {
      "trackmania-io": "https://trackmania.io/#/leaderboard/123"
      "trackmania-exchange": "https://trackmania.exchange/mapshow/321"
    }
  },
]

How did you test this change?

https://liquipedia.net/trackmania/User:SobakaPirat/MapTable

[hjpalpha]

fwiw Module:MapPoolTable is a module that already exists on the git
maybe it could be extended to be usable by trackmania too (instead of creating a new module)

[steve020607]

Why use local function for all of it?

[SobakaPirat]

fwiw Module:MapPoolTable is a module that already exists on the git maybe it could be extended to be usable by trackmania too (instead of creating a new module)

	Trackmania	MapPoolTable
Links	external	internal
Mappers	multiple	1
LPDB	yes	no
Images	external	internal

Plus, MapPoolTable looks bad without map images, because of the horizontal scrolling

[SobakaPirat]

@steve020607

Why use local function for all of it?

I dropped the class related stuff as Hjpa recommended, also it was necessary for splitting the module
#8132 (comment)
#8132 (comment)

[steve020607]

Probably for that it should follow under Features/

[hjpalpha]

just an fyi: means we need bot runs on trackmania with merging this

[hjpalpha]

@steve020607

Why use local function for all of it?

I dropped the class related stuff as Hjpa recommended, also it was necessary for splitting the module #8132 (comment) #8132 (comment)

technically could move most into a local table
but imo local functions are fine
as long as it is no global functions we are good :)

[mbergen]

IIRC the LPDB "maps" field is limited in length, and with this amount of info stored in it you'll run into issues pretty fast.

[hjpalpha]

IIRC the LPDB "maps" field is limited in length, and with this amount of info stored in it you'll run into issues pretty fast.

it stores into lpdb_datapoint with most of the data going into extradata

[hjpalpha]

seems okay (on phone)