Skip to content

feat: Ensure Nextcloud code use strict comparaison - #105

Merged
come-nc merged 4 commits into
mainfrom
carl/strict-comparaison
Oct 14, 2026
Merged

come-nc merged 4 commits into
mainfrom
carl/strict-comparaison

Conversation

@CarlSchwan

@CarlSchwan CarlSchwan commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

Description

Add strict: true to in_array when possible

Use === and !== instead of == and !=

Motivation and context

Security and correctness

How has this been tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

PR checklist

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan CarlSchwan self-assigned this Sep 3, 2026
@CarlSchwan
CarlSchwan force-pushed the carl/strict-comparaison branch from da15883 to 63c145d Compare September 24, 2026 12:26
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the carl/strict-comparaison branch from 63c145d to fe01cc6 Compare September 24, 2026 13:34
Comment thread src/ClassNameImport/NextcloudNamespaceSkipVoter.php
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the carl/strict-comparaison branch from d3ae134 to fcdc994 Compare October 14, 2026 14:37
@come-nc
come-nc merged commit 5c992da into main Oct 14, 2026
22 checks passed
@come-nc
come-nc deleted the carl/strict-comparaison branch October 14, 2026 14:42
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.

3 participants