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

PHP 8 and Wordpress 6+ support #31

Merged
merged 6 commits into from
Oct 29, 2023
Merged

PHP 8 and Wordpress 6+ support #31

merged 6 commits into from
Oct 29, 2023

Conversation

mattbucci
Copy link
Collaborator

I don't expect this to get merged, but I just spent the last week getting this package working on a fresh WP install and I'd like to make it available for anyone who might want it.

This PR includes code from the following PRs:

This PR also includes

  • Docblock comments for each method
  • A refactor towards a class based polymorphic implementation of SQLRewriters which allows better groking of what's happening with all these regexes
  • 3 new methods for handling the following situations I ran into when testing
    • Ensure the columns used in the ORDER BY clause are also present in the SELECT clause.
    • Ensuring Group By is added if it needs to be due to aggregated columns
    • Converting MySQL LIMIT syntax to PostgreSQL LIMIT syntax more reliably
  • New Logic for handling the SHOW FULL COLUMNS query

This was referenced Oct 28, 2023
@kevinoid
Copy link
Collaborator

Thanks @mattbucci, your PRs are great and this one is particularly ambitious! Would you be interested in taking over as the maintainer of this project?

Unfortunately, I haven't had time to work on it for quite a while (partly because I don't have any sites which use it at the moment) and it has languished as a result. I think there are a lot of users, and potential users, who would benefit from more active maintenance. If you'd be interested, I'd be willing to transfer the project to you. Alternatively, if you'd rather co-maintain the project, I could transfer it to a GitHub organization, or just grant you permission to this repo. What do you think?

@mattbucci
Copy link
Collaborator Author

mattbucci commented Oct 29, 2023 via email

@kevinoid
Copy link
Collaborator

Great! I've sent you an invite to collaborate on this project. Once you are in, feel free to merge any of the PRs and start implementing the ideas you have in mind.

as this repo is often mentioned in various blogs I think it would be best to start with maintaining it in it's current location and then decide how to proceed from there.

Makes sense to me. For reference, my understanding is that when a repository is moved, GitHub will redirect from the previous project URL to the new one, so it shouldn't break those links. There are always risks and minor incompatibilities though, so I agree with moving slow. If you'd like to take ownership of the project or move it to an organization at any point in the future, just let me know.

@mattbucci mattbucci changed the base branch from wordpress4-compat to v2 October 29, 2023 20:02
@mattbucci mattbucci merged commit f035c8f into PostgreSQL-For-Wordpress:v2 Oct 29, 2023
@mattbucci mattbucci changed the title WIP: PHP 8 and Wordpress 6+ support PHP 8 and Wordpress 6+ support Oct 29, 2023
@mattbucci
Copy link
Collaborator Author

@kevinoid I sent you an invite to https://github.com/PostgreSQL-For-Wordpress which is an org where this repo can live moving forward whenever you want to migrate it.

@kevinoid
Copy link
Collaborator

Thanks @mattbucci! I love all the work you've been doing on this project; thanks so much! I've transferred it to the new org. Let me know if there's anything else I can do to help.

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