minj / foxtrick

FoxTrick is a browser extension for the Hattrick online football manager game, currently available for Firefox, Google Chrome, as well as for Opera.
https://www.foxtrick.org
GNU General Public License v3.0
71 stars 49 forks source link

Vlaams broken due to inconsistent ordering (age vs. TSI parsing issue) #1141

Closed minj closed 9 years ago

minj commented 9 years ago

Original issue 1141 created by CatzHoek on 2013-08-29T13:53:40.000Z:

Vlaams LAs fucked up, players are represented like this:

TSI = 3 540 30 jaar en 77 dagen, loon = 670 €/week. Is matig in vorm, zijn conditie is redelijk. Zijn ervaring is zwak, zijn leiderschap slecht. Zijn clubliefde is goddelijk.

and foxtrick expects this kind of formatting (age before TSI basicly):

30 years and 77 days TSI = 3 540, wage = 670 €/week In inadequate form, passable stamina. Has weak experience and poor leadership. Has divine loyalty.

minj commented 9 years ago

Comment #1 originally posted by CatzHoek on 2013-08-29T14:02:59.000Z:

<empty>

Labels: -Priority-Medium, Priority-High

minj commented 9 years ago

Comment #2 originally posted by CatzHoek on 2013-08-29T14:03:46.000Z:

contacted LAs, hopefully no Foxtrick action required

minj commented 9 years ago

Comment #3 originally posted by CatzHoek on 2013-08-29T14:21:27.000Z:

16:00 Von dir Hi, in the vlaams language, the ordering of TSI and player age is not consistent with the rest of hattrick. Also it breaks Foxtrick by beeing different to all other languages.

Is there any special reason you don't order the information as any other language does? Would be nice if you could change it to the "default" way if there is no arguement against it.

Link to our FT ticket: (https://code.google.com/p/foxtrick/issues/detail?id=1141)

It would be awesome if you could adopt the default way to order TSI and age.

Thanks a lot.

Catz

minj commented 9 years ago

Comment #4 originally posted by CatzHoek on 2013-09-01T13:03:30.000Z:

fixed in stage, should be fine on production soonish, tomorrow as the LA said.

minj commented 9 years ago

Comment #5 originally posted by CatzHoek on 2013-09-03T12:26:19.000Z:

LAs fixed the issue

Status: Invalid