Skip to content

fix a php8.2 deprecation #[\AllowDynamicProperties] #197

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

Closed
wants to merge 1 commit into from

Conversation

peterdd
Copy link

@peterdd peterdd commented Mar 29, 2023

https://www.php.net/releases/8.2/en.php

I just tried to install the current phpldapadmin 1.2.6.5 on a fresh current Debian 12 "Testing" with apache2 and php 8.2, and a

cp config.php.example config.php

https://www.php.net/releases/8.2/en.php

I just tried to install the current phpldapadmin 1.2.6.5 on a fresh current Debian 12 "Testing" with apache2 and php 8.2, and a

```bash
cp config.php.example config.php
```
@williamdes
Copy link

williamdes commented Mar 31, 2023

Edit: PR replaced by #202

Moved to: #203

@williamdes
Copy link

I just tried to install the current phpldapadmin 1.2.6.5 on a fresh current Debian 12 "Testing" with apache2 and php 8.2, and a

You should try to apt install phpldapadmin instead :)

@peterdd
Copy link
Author

peterdd commented Mar 31, 2023

It was just a single quick drive by fix only for the login page, never used phpldapadmin before.

As you handle it much deeper I better close this minimal quickshot.

@peterdd peterdd closed this Mar 31, 2023
@williamdes
Copy link

williamdes commented Mar 31, 2023

It was just a single quick drive by fix only for the login page, never used phpldapadmin before.

As you handle it much deeper I better close this minimal quickshot.

Oh okay, feel free to give a shot to the Debian package in case I missed something

I will do the PR stuff for the patches, thank you for reminding me I have to do it ;p

Edit: I did do this PR in #202

@williamdes
Copy link

williamdes commented Mar 31, 2023

/cc @leenooks
I will be opening PR's for each patch, feel free to git apply them and add me as an author if it makes you gain some time VS PR management.

See list on: #203

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.

2 participants