Skip to content

[7.0] Remove deprecated strength parameter from AES class#48002

Open
SniperSister wants to merge 3 commits into
joomla:7.0-devfrom
SniperSister:7.x-remove-encrypt-deprecation
Open

[7.0] Remove deprecated strength parameter from AES class#48002
SniperSister wants to merge 3 commits into
joomla:7.0-devfrom
SniperSister:7.x-remove-encrypt-deprecation

Conversation

@SniperSister

@SniperSister SniperSister commented Jun 22, 2026

Copy link
Copy Markdown
Contributor
  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

Deprecated parameter removed

Testing Instructions

Code review

Actual result BEFORE applying this Pull Request

  • Parameter present

Expected result AFTER applying this Pull Request

  • Parameter gone

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@laoneo laoneo added the Architecture Pull requests which improve the architecture. label Jun 22, 2026
@laoneo

laoneo commented Jun 22, 2026

Copy link
Copy Markdown
Member

Do we not call that in core where we have to adapt the function call?

@SniperSister

Copy link
Copy Markdown
Contributor Author

Done

@laoneo

laoneo commented Jun 22, 2026

Copy link
Copy Markdown
Member

Thanks. Then we need to document that here as well.

@laoneo laoneo changed the title [7.x| Remove deprecated strengt parameter from AES class [7.0] Remove deprecated strengt parameter from AES class Jun 22, 2026
@QuyTon QuyTon changed the title [7.0] Remove deprecated strengt parameter from AES class [7.0] Remove deprecated strength parameter from AES class Jun 23, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@SniperSister Do the already registered passkeys continue to work?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Architecture Pull requests which improve the architecture. Documentation Required PR-7.0-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants