Skip to content

fix(settings): Deprecate old things for real - #43581

Merged
nickvergessen merged 3 commits into
masterfrom
bugfix/noid/use-real-deprecation
Feb 14, 2024
Merged

fix(settings): Deprecate old things for real#43581
nickvergessen merged 3 commits into
masterfrom
bugfix/noid/use-real-deprecation

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Psalm output before:

ERROR: InvalidDocblock - lib/public/Settings/IManager.php:40:2 - Typo in @deprecated for const in OCP. (see https://psalm.dev/008)
/**
* @since 9.1.0
* @depreacted 29.0.0 Use {@see self::SETTINGS_ADMIN} instead
*/
public const KEY_ADMIN_SETTINGS = 'admin';
ERROR: InvalidDocblock - lib/public/Settings/IManager.php:46:2 - Typo in @deprecated for const in OCP. (see https://psalm.dev/008)
/**
* @since 9.1.0
* @depreacted 29.0.0 Use {@see self::SETTINGS_ADMIN} instead
*/
public const KEY_ADMIN_SECTION = 'admin-section';
ERROR: InvalidDocblock - lib/public/Settings/IManager.php:52:2 - Typo in @deprecated for const in OCP. (see https://psalm.dev/008)
/**
* @since 13.0.0
* @depreacted 29.0.0 Use {@see self::SETTINGS_PERSONAL} instead
*/
public const KEY_PERSONAL_SETTINGS = 'personal';
ERROR: InvalidDocblock - lib/public/Settings/IManager.php:58:2 - Typo in @deprecated for const in OCP. (see https://psalm.dev/008)
/**
* @since 13.0.0
* @depreacted 29.0.0 Use {@see self::SETTINGS_PERSONAL} instead
*/
public const KEY_PERSONAL_SECTION = 'personal-section';
------------------------------
4 errors found
------------------------------

Follow up

  • Also enforce the @since for constants in OCP => Follow up PR as there are 223 failures 😱

Checklist

@nickvergessennickvergessen added bug 3. to review Waiting for reviews labels Feb 14, 2024
@nickvergessennickvergessen added this to the Nextcloud 29 milestone Feb 14, 2024
@nickvergessennickvergessen self-assigned this Feb 14, 2024
@nickvergessennickvergessen changed the title Bugfix/noid/use real deprecationfix(settings): Deprecate old things for realFeb 14, 2024

@skjnldsvskjnldsv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@blizzzblizzz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure why commented out code was committed

Comment threadbuild/psalm/OcpSinceChecker.php Outdated
Comment threadbuild/psalm/OcpSinceChecker.php Outdated
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessen merged commit f9db9a7 into masterFeb 14, 2024
@nickvergessen
nickvergessen deleted the bugfix/noid/use-real-deprecation branch February 14, 2024 20:22
@blizzzblizzz mentioned this pull request Mar 5, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@nickvergessen@AndyScherzinger@st3iny@blizzz@bigcat88@skjnldsv