lazymofo / datagrid

PHP MySQL CRUD Datagrid
MIT License
141 stars 57 forks source link

Usage with PHP 8.1 #82

Closed fluxxus closed 2 years ago

fluxxus commented 2 years ago

I am getting lots of errors/warnings when using with PHP version 8.1. It seems that passing null to non-nullable parameters of built-in functions is not acceptable anymore.

Just a few examples: Deprecated: mb_strlen(): Passing null to parameter #1 ($string) of type string is deprecated in... Deprecated: mb_detect_encoding(): Passing null to parameter #1 ($string) of type string is deprecated in... Deprecated: mb_convert_encoding(): Passing null to parameter #1 ($string) of type array|string is deprecated in...

Is there an easy fix for these or? Thx

lazymofo commented 2 years ago

Hi!

Thanks for pointing this out! Newer versions of php are becoming more strict on their input. I'll install php 8 and see if I can address these warnings. For the time being you can disable deprecations messages if desired.

-Ian

On Mon, Jan 10, 2022 at 12:20 PM M @.***> wrote:

I am getting lots of errors/warnings when using with PHP version 8.1. It seems that passing null to non-nullable parameters of built-in functions is not acceptable anymore.

Just a few examples: Deprecated: mb_strlen(): Passing null to parameter #1 ($string) of type string is deprecated in... Deprecated: mb_detect_encoding(): Passing null to parameter #1 ($string) of type string is deprecated in... Deprecated: mb_convert_encoding(): Passing null to parameter #1 ($string) of type array|string is deprecated in...

Is there an easy fix for these or? Thx

— Reply to this email directly, view it on GitHub https://github.com/lazymofo/datagrid/issues/82, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRM2WD4YNBRSXVTR7K2EBLUVM5SVANCNFSM5LUR7PIA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you are subscribed to this thread.Message ID: @.***>

fluxxus commented 2 years ago

Ok, thanks for the tip! I am quite a noob when it comes to PHP, but your library is a life saver for me :)

lazymofo commented 2 years ago

I believe all these warnings have been fixed for php 8.1 compatibility. Let me know if you see any others.

Thanks! -Ian

On Tue, Jan 11, 2022 at 11:32 AM ian sokolowski @.***> wrote:

Hi!

Thanks for pointing this out! Newer versions of php are becoming more strict on their input. I'll install php 8 and see if I can address these warnings. For the time being you can disable deprecations messages if desired.

-Ian

On Mon, Jan 10, 2022 at 12:20 PM M @.***> wrote:

I am getting lots of errors/warnings when using with PHP version 8.1. It seems that passing null to non-nullable parameters of built-in functions is not acceptable anymore.

Just a few examples: Deprecated: mb_strlen(): Passing null to parameter #1 ($string) of type string is deprecated in... Deprecated: mb_detect_encoding(): Passing null to parameter #1 ($string) of type string is deprecated in... Deprecated: mb_convert_encoding(): Passing null to parameter #1 ($string) of type array|string is deprecated in...

Is there an easy fix for these or? Thx

— Reply to this email directly, view it on GitHub https://github.com/lazymofo/datagrid/issues/82, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRM2WD4YNBRSXVTR7K2EBLUVM5SVANCNFSM5LUR7PIA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you are subscribed to this thread.Message ID: @.***>