Lua-Modules icon indicating copy to clipboard operation
Lua-Modules copied to clipboard

feat(match2): create team fortress support

Open Hesketh2 opened this issue 1 year ago • 7 comments

Summary

Match2 for Team Fortress wiki, they are one of wikis that hasnt got one but also pretty similar to existing setups we have.

Base

  • The first revision is a paste from World of Tanks (which Tanks itself is then a cleaner/updated version based on COD)
  • a Match of TF behaves similar to those, Have Maps and Scores

Custom

  • Due to Team Fortress has quite a lot of custom links that needs to be added back, there are more than usual here
  • logstf and logstfgold are suggested to be added as Per Map instead of a single link. Similar to stats in CS Match2, because of this both logstf does get stored in Extradatas.
  • Vetos/MVP may be removed later, I'm still waiting response from the editors if they want it

How did you test this change?

LIVE on Several pages : https://liquipedia.net/teamfortress/ETF2L/Season_31/Highlander/Premiership/Preseason_Playoffs

Sidenote

Because there are the {{match}} vs {{match/old}} thing that needs to be converted, the Copypaste Generator is not included here, will make a separate PR when the conversion for that is done

Hesketh2 avatar Jun 11 '24 13:06 Hesketh2

There are issues I need help with the current revision here https://liquipedia.net/teamfortress/User:Hesketh2 image

  • Matches always ended up having the left side being the winner, this include even if per Map sets the winner to 2, the matchlist sets left side team as winner
  • for Maps that used |finished=true over winner=X, regardless of score the leftside also always won

had no idea which part cause this

Hesketh2 avatar Jun 12 '24 13:06 Hesketh2

The most recent revision seems to fix the issue I have, I had the placementSortFunction as local function in the module but I made the call going to the commons MatchGroup/Input which doesnt seem to have it

idk if this is the actual fix it was too simple for me to assume I did it correctly but at least right now its fixed. Now the setPlacement thing is left because I cant delete anything in that function locally at all otherwise the match just throws error

I could not find other wikis that had converted setPlacement to commons as well that I could copy over

Hesketh2 avatar Jun 12 '24 18:06 Hesketh2

wanted to revisit on this one, I'm still need help on how to solve the unexpected winner issue with the setplacement thing

Hesketh2 avatar Jul 08 '24 14:07 Hesketh2

wanted to revisit on this one, I'm still need help on how to solve the unexpected winner issue with the setplacement thing

what "unexpected winner" issue?

hjpalpha avatar Jul 10 '24 15:07 hjpalpha

wanted to revisit on this one, I'm still need help on how to solve the unexpected winner issue with the setplacement thing

what "unexpected winner" issue?

So right now if I removed the SetPlacement function thats locally in the process module (in hopes of using the function from commons instead) https://liquipedia.net/teamfortress/index.php?title=Module%3AMatchGroup%2FInput%2FCustom%2Fdev&type=revision&diff=127718&oldid=127717

The unexpected winner error would show image

setPlacement do get called in three other places, my assumption is the input in the parentheses needs to be changed? image

Hesketh2 avatar Jul 11 '24 11:07 Hesketh2

https://liquipedia.net/teamfortress/index.php?title=Module%3AMatchGroup%2FInput%2FCustom%2Fdev&type=revision&diff=127721&oldid=127718

hjpalpha avatar Jul 11 '24 12:07 hjpalpha

Revisiting this I believe all the requested changes are there, not sure if I missed any

Hesketh2 avatar Aug 06 '24 11:08 Hesketh2

Missing Match/Legacy and Copy Paste Gen

Rathoz avatar Aug 23 '24 06:08 Rathoz

I assume this error will go away after 4634 merged image

Hesketh2 avatar Aug 23 '24 15:08 Hesketh2

you removed too much

Rathoz avatar Aug 23 '24 16:08 Rathoz

lgtm if tested

did you compare old match1 data to new match1 data?

How to test it again? I recall theres one when wildrift was worked on but I cant find the module for the test it

but from comparing a match of both bracket version on first glance it looks normal to me: https://liquipedia.net/teamfortress/Special:LiquipediaDB/User:Hesketh2#match

Hesketh2 avatar Sep 03 '24 11:09 Hesketh2

lgtm if tested did you compare old match1 data to new match1 data?

How to test it again? I recall theres one when wildrift was worked on but I cant find the module for the test it

but from comparing a match of both bracket version on first glance it looks normal to me: https://liquipedia.net/teamfortress/Special:LiquipediaDB/User:Hesketh2#match

if the data is identical that means it is fine :)

hjpalpha avatar Sep 03 '24 12:09 hjpalpha

#4650 adjustment needed

Rathoz avatar Sep 09 '24 15:09 Rathoz

I also rename the MatchGroupInput to MatchGroupInputUtil for more consistency vs other wikis too

Hesketh2 avatar Sep 09 '24 16:09 Hesketh2

pinging if theres anything I need to fix more

Hesketh2 avatar Sep 19 '24 08:09 Hesketh2