Fix demos.txt

Started by P.V.A.B.-jello
17/04/2014 14:56
Page 1 of 1
P.V.A.B.-jello17/04/2014 14:56

The old structure was 10000% better and easier to parse. While I can see the point of reordering the "Player" field (to make it easier to parse players with spaces in their name), any competent person should have already written a RegEx to parse the old string properly.

Fun fact: we don't need a" Map info" row if it's just the map name with a string prefix.

tl;dr: revert demos.txt change.

dEMolite mapper17/04/2014 14:59

demos.txt is just a feature from the past for the ircbot. they might have changed the bot with a new message, thats why the layout has been changed

P.V.A.B.-jello17/04/2014 16:00

But why after years of using the same format and every other community following suit to follow the same standard? Sometimes it's best to leave things the way they are instead of trying to reinvent the wheel.

PsYxOpAt mapper17/04/2014 16:24

i will bring the old format later and will keep the current one as well

P.V.A.B.-jello17/04/2014 16:53(edited 17/04/2014 16:54)

Okay, thanks for the quick response. Maybe call the new format something like "demos_full.txt" or "demos_ext.txt".

On a different note it would be nice if you had a service like an API for WRs. Doesn't have to be restful but it could be really nice. :) Especially if an authorized user (such as an admin) wanted to manually update or add a wr. Or simply calling a get request for a particular map (that anyone could do). I kind of did something similar here: http://kz-endo.com/wrs/kz_xj_3ways for example but it's makeshift because it pulls from a database which is populated/updated through demos text files.

PsYxOpAt mapper17/04/2014 19:07

I don't have that much free time. If you want send me a pm with a detailed explanation of your idea and I will see if I can do it.

PsYxOpAt mapper17/04/2014 19:25(edited 17/04/2014 19:26)

demos.txt fixed. Atm inside you can find mapname, record time (by request I can convert it into seconds not in minutes: 03:35.32). jumper country and name and dl link for the demo. Let me know if anyone needs anything else. The ext file will come tomorrow.

dEMolite mapper17/04/2014 19:50

he could had used demos_bot.txt or demos_tcl.txt or not?

P.V.A.B.-jello17/04/2014 19:58(edited 17/04/2014 20:01)

I think the old way of full seconds is best so that someone trying to read the file isn't forced to use that exact time format (00:00.00). Although it is the best format in my opinion, it offers greater flexibility to provide a float value.

I love the addition of the demo id (although the title line is bugged), but simply providing the id number rather than a full url should be sufficient. Also, appending it at the end kind of defeats the purpose of providing the country code before the name (which I believe was to make it easier to parse player names with spaces). I would maybe put the id first if you really want to change everything up or, better yet, reverse the country code and name to where they were before so that old WR plugins can still parse correctly without having to edit the source and recompile. Example below of what I think would be best:

www.Xtreme-Jumps.eu
8b1_brickngrass 231.68 Nukk ru 642


Edit: I was unaware those alternates existed demolite.

PsYxOpAt mapper17/04/2014 20:19

done.

P.V.A.B.-jello17/04/2014 20:53

Any reason for the preceeding space character on each line?

remake17/04/2014 21:21(edited 17/04/2014 21:23)

Wouldn't it be better to revert back to the old system and simply provide a new file for all the additional information? The previous format has been hard-coded into many plugins and sites over the years, thus I feel it's unnecessary to update the format for no solid reason.

dEMolite mapper17/04/2014 23:45

just check these and edit your plugins or php fetch scripts
http://xtreme-jumps.eu/demos_tcl.txt
http://xtreme-jumps.eu/demos_bot.txt

remake17/04/2014 23:54(edited 17/04/2014 23:55)

I already did. It's just that not all developers are still around to actively update their code.

PsYxOpAt mapper18/04/2014 10:30

Updated with the old layout http://xtreme-jumps.eu/demos.txt. added country tld code and demo id for download. if anyone has an idea how to remove the 1st line let me know.

P.V.A.B.-jello18/04/2014 16:24(edited 18/04/2014 16:30)

demolite, those were also reformatted. not many people know the pawn scripting language or how to edit amxx source code so the different time format, order, etc. are taken into account. these changes affect countless servers by updating the original text file.

Page 1 of 1

Please log in to post replies.