-
Notifications
You must be signed in to change notification settings - Fork 155
PHP-239 - PHP 8 #147
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-239 - PHP 8 #147
Conversation
Signed-off-by: Dusan <dmalusev@nanointeractive.com>
Signed-off-by: Dusan Malusev <malusevd99@gmail.com>
44f4db3
to
c84ddd3
Compare
b11167c
to
48ee3a9
Compare
@absurdfarce would be the right one to look into this.
With these changes, I get following error when trying to access a row on 8.1:
Uncaught Error: Cannot use object of type Cassandra\\Rows as array
The code is as simple as:
$res = $session->execute("select * from foobar where key='value'");
if (!$res->valid())
return null;
$row = $res[0]; // error happens here
Worked fine with PHP 7.4 and the original datastax cassandra driver. So ArrayAccess seems to be broken here.
I can reproduce... sadly this was not unit tested 😞
abd5ff0
to
d9bb109
Compare
danielhe4rt
commented
Jan 4, 2023
What is missing exactly? I'm studying about Cassandra and ScyllaDB, since my main lang is PHP it was about to help me a lot on my studies...
danielhe4rt
commented
Jan 4, 2023
@qkdreyer how can I get in touch with you? I saw your PHP 8.1 implementations and I'm wondering if there's any chance to use it for a demo.
Kabingeran
commented
Mar 12, 2023
Could you update pecl for php 8.x please?
danielhe4rt
commented
Mar 12, 2023
I think that its impossible for now since the holders of pecl is from DataStax.
@danielhe4rt Whose repository is this?
danielhe4rt
commented
Mar 13, 2023
Is was a fork from he4rt/ScyllaDB
danielhe4rt
commented
Mar 13, 2023
Ping me at discord or twitter!
danielhe4rt#0001
@danielhe4rtless
Uh oh!
There was an error while loading. Please reload this page.
https://datastax-oss.atlassian.net/browse/PHP-239