ToxicRevolution / CSGORankMeWeb

Web interface for Rank Me CSGO Stat tracking plugin
https://Toxicrevolution.net
MIT License
30 stars 10 forks source link

Memory size #13

Closed szogi1910 closed 7 years ago

szogi1910 commented 7 years ago

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 8192 bytes) in /home/pggames/domains/project/public_html/statystyki/dd2mirage/control/queryFunctions.php on line 47

kennyLtv commented 7 years ago

How many players are in your database?

szogi1910 commented 7 years ago

12797 Total Users!

APet74 commented 7 years ago

Kenny or I can send you a fix. Or Kenny so you wanna give him the old version of queryfunctions and the leaderboard pages as well the players class. I can write a fix tomorrow.

szogi1910 commented 7 years ago

Ok, I'll wait for the fix

szogi1910 commented 7 years ago

And how will it with this?

APet74 commented 7 years ago

Sorry, been busy with school starting. I am looking into server-side pagination and that will hopefully speed things up AND fix your error. Sorry for the delay.

APet74 commented 7 years ago

@szogi1910 Please download a new ZIP of the project. I have added a new file in /control/leaderboard named leaderboardGrabLarge.php, please replace your leaderboardGrab.php with that file making sure to rename it to "leaderboardGrab.php". I have added the necessary functions to queryFunctions.php so that will need to be moved too. As always I suggest a full replacement of the file system followed by you deleting the install directory. Overwriting all your files will cause no harm. Please let me know if there is more.

Thanks.

szogi1910 commented 7 years ago

It now shows all players Only now is the problem of retrieving data for the score column Example player via www max pkt 1012 - http://imgur.com/a/NUW7x

Top players via www http://project-games.eu/statystyki/dd2mirage/leaderboards.php - http://i.imgur.com/aNnMxH0.png To players rekords pma - https://i.imgur.com/Ppkhro4.png

APet74 commented 7 years ago

So basically you're saying that the database shows a different number in the score column versus what the website shows. That's very interesting I don't know what would cause that because the script literally just pulls from the score column, I'll have to look into it and see if I can figure out what would be causing that. Do you have more than one rankme database, is there any other databases.

On Aug 25, 2017 5:19 PM, "szogi1910" notifications@github.com wrote:

It now shows all players Only now is the problem of retrieving data for the score column Example player via www max pkt 1012 - http://imgur.com/a/NUW7x

Top players via www http://project-games.eu/statystyki/dd2mirage/leaderboards.php - http://i.imgur.com/aNnMxH0.png To players rekords pma - https://i.imgur.com/Ppkhro4.png

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ToxicRevolution/CSGORankMeWeb/issues/13#issuecomment-325046688, or mute the thread https://github.com/notifications/unsubscribe-auth/AEgumWHOxoOruwQcVq4fWoIlkLQtPt4rks5sb0h_gaJpZM4O1uDM .

kennyLtv commented 7 years ago

Is it possible that you are pulling from an incorrect database now? https://puu.sh/xjy2p/58d0818185.png as it shows the same thing on the profile page.

APet74 commented 7 years ago

I just don't know how it is pulling wrong. All other people using it have not reporting any problems.

On Aug 25, 2017 5:39 PM, "Kenny Lindelof" notifications@github.com wrote:

Is it possible that you are pulling from an incorrect database now? https://puu.sh/xjy2p/58d0818185.png as it shows the same thing on the profile page.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ToxicRevolution/CSGORankMeWeb/issues/13#issuecomment-325049466, or mute the thread https://github.com/notifications/unsubscribe-auth/AEgumYHZt9uvrlERrOx-dcNFAmfe2PLdks5sb00hgaJpZM4O1uDM .

szogi1910 commented 7 years ago

@APet74 There, I have several rankme tables each have a separate prefix @kennyLtv I checked twice and the data I have in config and are correct

Here you can see the version that works without the last fix and gets the data correctly http://project-games.eu/statystyki/dd2/leaderboards.php

APet74 commented 7 years ago

I have made a change that should resolve it. For our own records, the issue was within the SQL statement of queryFunctions.php, the statement had WHERE kills = 0 AND deaths = 0 instead of WHERE kills <> 0 AND deaths <> 0 Please check now and let me know.

On Sat, Aug 26, 2017 at 4:07 AM, szogi1910 notifications@github.com wrote:

@APet74 https://github.com/apet74 There, I have several rankme tables each have a separate prefix @kennyLtv https://github.com/kennyltv I checked twice and the data I have in config and are correct

Here you can see the version that works without the last fix and gets the data correctly http://project-games.eu/statystyki/dd2/leaderboards.php

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ToxicRevolution/CSGORankMeWeb/issues/13#issuecomment-325105740, or mute the thread https://github.com/notifications/unsubscribe-auth/AEgumai0TIok75mGO-kiJq_TcZ6_eqoMks5sb-BggaJpZM4O1uDM .

szogi1910 commented 7 years ago

Gets data correctly. Thanks for taking the time and adding a fix.