Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory size - v2 #14

Closed
szogi1910 opened this issue Sep 2, 2017 · 19 comments
Closed

Memory size - v2 #14

szogi1910 opened this issue Sep 2, 2017 · 19 comments
Assignees

Comments

@szogi1910
Copy link

szogi1910 commented Sep 2, 2017

The problem reappeared - #13
Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 3 bytes) in /home/pggames/domains/project-games.eu/public_html/statystyki/dd2mirage/control/queryFunctions.php on line 47

@kennyLtv
Copy link
Collaborator

kennyLtv commented Sep 3, 2017

Looking into this now.

@kennyLtv kennyLtv self-assigned this Sep 3, 2017
@kennyLtv
Copy link
Collaborator

kennyLtv commented Sep 3, 2017

So this is going to be clunky and slow but it should fix it till we can get a more permanent solution out. If you go to js/leaderboard/leaderboardFunctions.js and change on line 5 the URL for the ajax from leaderboardGrab.php to leaderboardGrabLarge.php I believe it will work temporarily. I am running that leaderboardGrab on a little over 17k entries on my test server. Please let me know if that doesn't fix it for now.

@szogi1910
Copy link
Author

Unfortunately, it didn't work for me

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

@APet74
Copy link
Collaborator

APet74 commented Sep 4, 2017 via email

@APet74
Copy link
Collaborator

APet74 commented Sep 5, 2017

Szogi, go download the latest release. I made a change to the leaderBoardGrab.php so that it uses server sided data tables. You will want to rerun the install.php in the /install/ directory as it adds some new information to the config.php which is required. Let me know if you have problems. As always, download the new files, and replace all current files you have.

Thanks.

@szogi1910
Copy link
Author

After uploading the new version of leaderboards.php returns us communique No data available in table

I suspect the error is in $ sql_details = array where I can not define the name of the table from which to retrieve server data

@APet74
Copy link
Collaborator

APet74 commented Sep 5, 2017 via email

@szogi1910
Copy link
Author

szogi1910 commented Sep 5, 2017

The data in the table was correctly downloaded before the update.
code below, which as you can see has data to connect to the database but when we have several tables it is probably unable to retrieve data using which leaderboards.php displays no data available in table

$sql_details = array(
			'user' => 'xxx_user',
			'pass' => 'xxx',
			'db' => 'xxx_rankme',
			'host' => 'localhost'
);

Yes I shoot maybe it should be

$sql_details = array(
			'user' => 'xxx_user',
			'pass' => 'xxx',
                        'table' => 'xxx',
			'db' => 'xxx_rankme',
			'host' => 'localhost'
);

@APet74
Copy link
Collaborator

APet74 commented Sep 5, 2017

Just pulled a fresh copy of the repo and it's giving an error. Let me address it real quick.

@APet74
Copy link
Collaborator

APet74 commented Sep 5, 2017

Ok so... I am not the smartest person alive, obviously. I misspelled the AJAX request... So it should work now. You can either redownload the new release OR go into js/leaderboard/leaderboardFunctions.js and at the bottom change "ajax": "control/leaderboard/leaderBoardGrab.php", to "ajax": "control/leaderboard/leaderboardGrab.php", (board isn't capitalized. Also in views/leaderboards/index.php change kills and deaths to K/D and ADR as that is what is actually being reported not kills and deaths (should be 2 kills and 2 deaths)

@szogi1910
Copy link
Author

Unfortunately, still does not show details
http://project-games.eu/statystyki/dd2mirage/leaderboards.php

@APet74
Copy link
Collaborator

APet74 commented Sep 5, 2017

As stated above, please change the following inside of `/js/leaderboard/leaderboardFunctions.js':

"ajax": "control/leaderboard/leaderBoardGrab.php"
to
"ajax": "control/leaderboard/leaderboardGrab.php"

Here is a screenshot that shows that it is still wrong.
http://petrzilkacoding.com/gitHubAPI/ssMemoryError.png
Here is the change in the master code directory: https://github.com/ToxicRevolution/CSGORankMeWeb/blob/master/js/leaderboard/leaderboardFunctions.js#L140

@APet74 APet74 self-assigned this Sep 5, 2017
@APet74
Copy link
Collaborator

APet74 commented Sep 6, 2017

I am still checking out your site and it seems to still be down but it looks as though it is down because you haven't placed the new javascript file into the folder. Let me know if this is intentional.

@szogi1910
Copy link
Author

szogi1910 commented Sep 6, 2017

I uploaded the files, which are new to the github community, and then I reinstalled the script and as you can see on the page the effect

Edit
By accident I closed the application

@APet74
Copy link
Collaborator

APet74 commented Sep 6, 2017

Is it broken? It seems to be working when I view it.

@APet74 APet74 reopened this Sep 6, 2017
@szogi1910
Copy link
Author

szogi1910 commented Sep 7, 2017

I do not quite understand what you mean by now

@APet74
Copy link
Collaborator

APet74 commented Sep 7, 2017

Is there still an issue? When I go to your site the table loads and all the data looks proper. Is there still an issue that needs addressing?

@szogi1910
Copy link
Author

works fine, forcing ctrl + f5 several times

@APet74
Copy link
Collaborator

APet74 commented Sep 7, 2017 via email

@kennyLtv kennyLtv closed this as completed Sep 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants