PHP-MySQLi-Database-Class icon indicating copy to clipboard operation
PHP-MySQLi-Database-Class copied to clipboard

Is this project still alive?

Open iahim opened this issue 2 years ago • 12 comments

I don't see too much movement in the last 2 years...

iahim avatar Aug 28 '22 09:08 iahim

I still use it religiously.

peterlaws avatar Sep 11 '22 13:09 peterlaws

I still use it religiously.

I have tried to use it lately but i have a warning ... using php 7.4.30. PHP Notice: Undefined offset: 0 in vendor/thingengineer/mysqli-database-class/MysqliDb.php on line 559

mihaiflorio avatar Sep 11 '22 18:09 mihaiflorio

That is just a notice - but it works despite that??...... I use it with 8.0, no issues...

But, I agree, would like some input from the author, @ThingEngineer

peterlaws avatar Sep 11 '22 18:09 peterlaws

That is just a notice - but it works despite that??...... I use it with 8.0, no issues...

But, I agree, would like some input from the author, @ThingEngineer

Yeah it does work, is just annoying :)

mihaiflorio avatar Sep 11 '22 18:09 mihaiflorio

Sorry guys, I'll work this notice next week. Apart from that everything is working fine in the library?

I'll release next version with this fix.

On Sun, Sep 11, 2022, 2:34 PM mihaiflorio @.***> wrote:

I still use it religiously.

I have tried to use it lately but i have a warning ... using php 7.4.30. PHP Notice: Undefined offset: 0 in vendor/thingengineer/mysqli-database-class/MysqliDb.php on line 559

— Reply to this email directly, view it on GitHub https://github.com/ThingEngineer/PHP-MySQLi-Database-Class/issues/995#issuecomment-1243018901, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZCL4XADZVMYQKNAA7SYQDV5YQ5BANCNFSM572YMTFA . You are receiving this because you are subscribed to this thread.Message ID: @.*** com>

avbdr avatar Sep 11 '22 19:09 avbdr

@avbdr on php 7.4.30 seems to work just fine (apart of the notice). Thanks a lot for fixing it.

mihaiflorio avatar Sep 11 '22 21:09 mihaiflorio

Hi guys, I'm using this package since 2016 and it worked always very well. In the next months I'll need to migrate a project to php 8.x, and I would like to keep using this library if possible. It's not 100% clear if it's compatible with php 8, as there are some open issues about it, while some people says that it's working. I can give some support if needed...

Just let me know, in case. Bye!

funder7 avatar Sep 10 '23 22:09 funder7

@funder7 it does work, i use it with php8. there is indeed that weird notice (but it does not break the code). I was thinking about forking it ... if the owner does not update it anymore.

iahim avatar Sep 11 '23 09:09 iahim

I dont know what else to add to the library, I think it is good as is and further "improvements" less likely will bring anything major anymore. I have to sit to fix php8 related deprecation notices with a backwards compatibility, this is so far the only thing in my todo

avbdr avatar Sep 11 '23 11:09 avbdr

@avbdr totally agree, there is no need for further improvements, just the fix for php8. I do not think is too much work, I can pay you to fix it (my OCD always is getting crazy when I see the NOTICE from the library) :)))))) I am serious about it - let me know how much it costs, to have it fixed sooner than later.

iahim avatar Sep 11 '23 17:09 iahim

(my OCD always is getting crazy when I see the NOTICE from the library) :))))))

You should only have notices on for debugging....... So you'd pay for something to be fixed when you have the option to turn off........ Hell, I'd fix it for £100.........

peterlaws avatar Sep 11 '23 22:09 peterlaws

I dont know what else to add to the library, I think it is good as is and further "improvements" less likely will bring anything major anymore. I have to sit to fix php8 related deprecation notices with a backwards compatibility, this is so far the only thing in my todo

Sure, I was referring to fixing any warning to make it fully compatible with php 8.x, no need to add other functionalities IMHO.

funder7 avatar Sep 16 '23 13:09 funder7