-
Notifications
You must be signed in to change notification settings - Fork 338
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 7 Issues #228
Comments
We are using this in production, for several thousand customers, with Composer, and MySQLi, in PHP 7.x. Works just fine. If you're having issues, please post the error messages you're referring to. We will have an update which doesn't reply on mcrypt available soon. |
@consolibyte What you do not like in my pull requests? |
It happens with me.
|
Updated to the latest version, and got another error from deprecated functions since PHP 7.1, which you mentioned you would have an update. |
public function oauthLoad($key, $app_username, $app_tenant)
I changed the driver.php myself, is this ok totally? |
I'm having the same issue
using PHP 7.2.2 |
Additional PHP 7 issues:
|
Please download latest code from git repo. If you download website then it will give error like as you saying |
@ajaygiri97 thanks, the consolibyte/quickbooks@3.6 does seem to be working in PHP 7.3 so far. Note - to use MySQLi or some other database driver, just specify the appropriate DSN (e.g., |
Am I going nuts guys? I am using the SDK from the 3.6 release and have been fine on php7.0. Recently upgraded to 7.3 (Debian Buster) and I am getting scores of warnings:
Any help appreciated. |
I'm not experiencing those warnings, but then again I'm not using the oauth functionality either. @consolibyte says earlier in this thread that they are working on an update to remove the mcrypt dependency:
|
The HTTP warnings are coming from each API call to QBO. I have checked and my SDK files are up to date with the current release. Any other tips appreciated as disabling error reporting does not seem to work and my logs are filled with garbage. |
Yeah, see, I'm using it to communicate with QuickBooks Desktop via the QB Web Connector. It's working fine. We've been wanting to switch to QB Online at some point, so good to know that this SDK is apparently not ready. My advice would be to fork, fix, and submit a pull request. Or, you could try reaching out to @consolibyte. Or, if you'd like to hire me to help you with this, contact me at https://compwright.com. |
The last pull request I followed was ignored. @consolibyte Keith do you have an ETA on this fix? PHP 7.3 is the default under Debian Buster. |
Has this been fixed? |
We are trying to update from OAuth 1.0 to OAuth 2.0 by the deadline of Dec 17 but this package is still using mcrypt_module_open and other deprecated PHP functions. Is there a timeline on when this will be fixed? |
@dahoovdog You can try to use my version of openssl usage https://github.com/ebogdanov/quickbooks-php/tree/encryption/mcrypt-to-openssl But in general I suggest you to consider migration for V3 SDK from Intuit. |
Seems the PHP kit is not ready for PHP7
Known issues
Mysql calls not using Mysqli
mcrypt deprecated
issues with Composer package incompatible with PHP 7
Any other items?
The text was updated successfully, but these errors were encountered: