php-driver icon indicating copy to clipboard operation
php-driver copied to clipboard

PHP-239 - PHP 8

Open qkdreyer opened this issue 3 years ago • 12 comments

https://datastax-oss.atlassian.net/browse/PHP-239

qkdreyer avatar Jan 05 '22 16:01 qkdreyer

@mpenick @avalanche123

qkdreyer avatar Jan 10 '22 12:01 qkdreyer

@adutra

qkdreyer avatar Jan 10 '22 17:01 qkdreyer

@absurdfarce would be the right one to look into this.

adutra avatar Jan 17 '22 19:01 adutra

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.

CharlemagneLasse avatar Oct 20 '22 09:10 CharlemagneLasse

I can reproduce... sadly this was not unit tested 😞

qkdreyer avatar Oct 20 '22 14:10 qkdreyer

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 avatar Jan 04 '23 15:01 danielhe4rt

@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.

danielhe4rt avatar Jan 04 '23 15:01 danielhe4rt

Could you update pecl for php 8.x please?

Kabingeran avatar Mar 12 '23 12:03 Kabingeran

I think that its impossible for now since the holders of pecl is from DataStax.

danielhe4rt avatar Mar 12 '23 12:03 danielhe4rt

@DanielHe4rt Whose repository is this?

Kabingeran avatar Mar 12 '23 23:03 Kabingeran

Is was a fork from he4rt/ScyllaDB

danielhe4rt avatar Mar 13 '23 10:03 danielhe4rt

Ping me at discord or twitter!

danielhe4rt#0001 @danielhe4rtless

danielhe4rt avatar Mar 13 '23 10:03 danielhe4rt