evilru / quake3-vsp-stats

VSP Stats Game Log Processor - updated and working version of vsp-0.45-xp-1.2-full.zip from https://vsp.goquake.com/ as docker image: https://hub.docker.com/r/evilru/quake3-vsp-stats.
MIT License
11 stars 10 forks source link

compatibility with MariaDB 10.2 and PHP 7.1 #1

Closed jf3218 closed 6 years ago

jf3218 commented 6 years ago

Possible to have a fix for Quake3-vsp-stats with MariaDB and PHP 7.1 ?
thx!

evilru commented 6 years ago

Hello,

As vsp-stats is pretty old, it used a very old version of adodb (3.50 19 May 2003) I just updated the adodb lib to 5.20.10 (latest) and changed the corresponding config to make use of the now needed mysqli driver. I don't have any game.log for testing right now - at least i was able to call it from the browser and it did not throw any PHP error...

Could you please give it a try and tell me if it works? https://github.com/evilru/quake3-vsp-stats/tree/50a82222b150b92f5f6bbb625b9b2bdc6847ffa7

Cheers, Martin

jf3218 commented 6 years ago

Hello thanks you for the update, I did the test, I'm now able to call it from the browser but noting are showed, http://146.71.78.147/vsp/pub/themes/bismarck/index.php?config=cfg-default.php.

When I go see my database ffa_vsp with phpmyadmin , there noting have been created inside ... So yeah, I think the Stats dosen't read the game.log

Best regards, jf.

evilru commented 6 years ago

Hm.. did you run the games.log import as described under point 3.c in the readme?

jf3218 commented 6 years ago

Yes I did via Power Shell http://prntscr.com/im9nv0 and via the web http://prntscr.com/im9par http://prntscr.com/im9pgy

1- There noting get created into my database ffa_vsp
2- You can see my database setting in cfg-default.php - http://prntscr.com/im9swx 3- Here the game settings set to 'q3a' - http://prntscr.com/im9ujh

I dont know why noting get create in the database, maybe need to enable a option in the php.ini for adodb?

evilru commented 6 years ago

while switching to mysqli, i also needed to rename the db setup file. you can go with the latest from the master branch, it works now for me. can you confirm this?

cheers

jf3218 commented 6 years ago

Good job this working 👍 http://146.71.78.147/vsp-stats/pub/themes/bismarck/index.php

Small question: how to show the adwards list ?

evilru commented 6 years ago

Hm.. this worked for me. Did you use the master branch?

I have the feeling it might relate to this change: https://github.com/evilru/quake3-vsp-stats/commit/692b27261afe2437ee986c30d0250152b6def0bc

jf3218 commented 6 years ago

This was a mistake in my setting.. This working just fine now except for the flags... Probally a other setting not set correctly .