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

Not working on php 8+ #165

Closed
Theo4 opened this issue Mar 3, 2022 · 3 comments
Closed

Not working on php 8+ #165

Theo4 opened this issue Mar 3, 2022 · 3 comments

Comments

@Theo4
Copy link

Theo4 commented Mar 3, 2022

Hello,

Trying to use the lib on a php8+ version and it doesn't work. A lot of deprecated notice and impossible to authenticate are main problems.

Is there any planned update ?

Thank you,

@plezan
Copy link

plezan commented Mar 4, 2022

Hello, we also have have this deprecations notice :

Return type of DocuSign\eSign\Model\*** should either be compatible with ArrayAccess::offsetSet(mixed $offset, mixed $value): void, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice

on every Docusign model using offsets :

  • offsetSet($offset, $value)
  • offsetUnset($offset)
  • offsetExists($offset)
  • offsetGet($offset)

Do you think it's possible to use the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice while you get this issue fixed ?

Thanks

Edit :

after some research it looks like a PHPSwaggerGen issue that has been fixed in the last release :
https://github.com/vanderlee/PHPSwaggerGen/blob/master/CHANGELOG.md#2321---2021-01-07

@dbbrahmbhatt
Copy link
Contributor

Thank you for filing the issue. We are currently working on the fix for this. Update will be posted here once the fix is shipped.

@dbbrahmbhatt
Copy link
Contributor

Issue has been fixed using #[\ReturnTypeWillChange] attribute in release 6.7.1

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

No branches or pull requests

3 participants