Skip to content

General backend type safety improvements and php code modernization #846

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

General backend type safety improvements and php code modernization #846

wants to merge 5 commits into from

Conversation

binsky08
Copy link
Collaborator

While trying to mitigate php8.4 deprecation warnings I ended in a comprehensive overhaul of our custom DB entity types and classes.
So we now have (almost everywhere) deterministic entity types from the controller to the entity class.

  1. I've added and applied a rector config for easier language migrations and php deprecation fixes
  2. Migrated implicit nullable parameters (deprecated since php8.4) to the explicit syntax introduced in php7.1; closes PHP 8.4 fix code #834
  3. Improved type safety in custom QBMapper classes and its callers by using a more strict entity typing
  4. Add .editorconfig to match Nextcloud's default file formatting when using an IDE like PhpStorm

@binsky08 binsky08 added this to the 2.5.0 milestone Sep 27, 2025
@binsky08 binsky08 requested a review from animalillo September 27, 2025 22:05
@binsky08 binsky08 mentioned this pull request Sep 28, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@animalillo animalillo Awaiting requested review from animalillo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.5.0
Development

Successfully merging this pull request may close these issues.

PHP 8.4 fix code
1 participant
@binsky08