Skip to content
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.

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

PHP 8.2 Deprecation #3546

Closed
dleffler opened this issue Dec 20, 2022 · 2 comments
Closed

PHP 8.2 Deprecation #3546

dleffler opened this issue Dec 20, 2022 · 2 comments
Assignees

Comments

@dleffler
Copy link

Describe the bug
Use of 'self::' in some functions is dprecated in PHP v8.2

To Reproduce
Steps to reproduce the behavior:

  1. Open elFinder under PHP v8.2
  2. Note the number of warning messages referencing elFinder.class.php line # 795 calling 'self'

Expected behavior
Should not throw messages

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: Windows
  • Browser Chrome
  • Version 108

Fix
Change elFinder.class.php line # 795 to:

$doRegist = ($_reqCmd && in_array($_reqCmd, array_map('elFinder::getCmdOfBind', explode(' ', $cmd))));
'''
instead of currently
'''
$doRegist = ($_reqCmd && in_array($_reqCmd, array_map('self::getCmdOfBind', explode(' ', $cmd))));
@github-actions
Copy link

github-actions bot commented Jun 3, 2023

This issue is stale because it has been open for 50 days with no activity.

@github-actions github-actions bot added the stale label Jun 3, 2023
@nao-pon
Copy link
Member

nao-pon commented Jun 3, 2023

I'll fix it. Thanks!

@nao-pon nao-pon self-assigned this Jun 3, 2023
@github-actions github-actions bot removed the stale label Jun 4, 2023
Bit-Code-Developer pushed a commit to Bit-Apps-Pro/elFinder that referenced this issue Oct 22, 2023
…elf::getCmdOfBind which is deprecated in PHP v8.2
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

No branches or pull requests

2 participants