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

Fixes for php 7.4, MySQL 8, and image servers #120

Open
wants to merge 13 commits into
base: 4.4
Choose a base branch
from

Conversation

Frug
Copy link

@Frug Frug commented Mar 7, 2023

With a few tweaks I've been able to run the killboard on PHP 7.4 and MySQL 8 (without changing any of the strictness settings in MySQL). All my inventory images were borked so I included a fix for that, too.
Maybe someone still running a killboard will find these changes useful. The commit history describes the changes, but I realize this should probably be several PRs rather than one.

A few fixes of note:

  • Removed a broken API call to ESI that used to fetch the remote alliance ID. That needs to be rewritten to fix it. All I did was remove the broken call. Alliance IDs still won't get updated properly.
  • Updated to use CCP's newer image server, and updated the paths to the images so it actually works
  • Fixed a few deprecation warnings.
  • Fixed several GROUP BY clauses that were missing non-aggregate fields
  • Did a bunch of work on the kill list (class.killlist.php) after fixing the GROUP BY clause led me down a rabbit hole to get the right columns listed.

NOTE: When running the updates, the migrations crashed because some default timestamps were set to 0000-00-00 00:00:00. I assume MySQL used to let you use that as a date, but these days it complains if you try to alter a table with a default like that. I manually set a proper default of a random date, but didn't actually write a migration to fix that for you. It only comes up if you try to run the updates from before 032 on MySQL 8.

@Frug Frug changed the title Remove outdated api call, fix member count Fixes for php 7.4, MySQL 8, and image servers Mar 7, 2023
@tonytonyistony
Copy link

@Frug Id like to speak with you about this implementation. Im having some issues trying to get this working on my server and would like to brainstorm. How can I reach you?

@Frug
Copy link
Author

Frug commented May 13, 2024

@Frug Id like to speak with you about this implementation. Im having some issues trying to get this working on my server and would like to brainstorm. How can I reach you?

Sorry I haven't been very active here. What kind of issues are you having?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants