Skip to content

feat: Nextcloud 35 compatibility improvements - #32

Open
henmohr wants to merge 2 commits into
LibreSign:mainfrom
henmohr:nextcloud-35-compat
Open

henmohr wants to merge 2 commits into
LibreSign:mainfrom
henmohr:nextcloud-35-compat

Conversation

@henmohr

@henmohr henmohr commented Sep 9, 2026

Copy link
Copy Markdown
Contributor
  • Update PHP platform requirement from 8.1 to 8.3 in composer.json
  • Refactor InjectionMiddleware to use IAddress::matches() public API instead of direct Range::contains() calls
  • Update copyright year

The app already declared max-version=35 in info.xml. These changes improve code quality and ensure proper use of public OCP interfaces.

- Update PHP platform requirement from 8.1 to 8.3 in composer.json
- Refactor InjectionMiddleware to use IAddress::matches() public API
  instead of direct Range::contains() calls
- Update copyright year

The app already declared max-version=35 in info.xml. These changes
improve code quality and ensure proper use of public OCP interfaces.

Signed-off-by: henmohr <henmohr@users.noreply.github.com>
@henmohr
henmohr requested a review from vitormattos as a code owner September 9, 2026 16:15
@welcome

welcome Bot commented Sep 9, 2026

Copy link
Copy Markdown

Thanks for opening your first pull request in this repository! ✌️

- Add space after fn in arrow function (CS fixer requirement)
- Use $remoteAddress directly in log instead of casting Address to string
- Remove unused UndefinedClass baseline entry for OC_Helper

Signed-off-by: henmohr <henmohr@users.noreply.github.com>
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.

1 participant