add php 8.1 to test

[Rotzbua]

now released https://www.php.net/releases/8.1/en.php

[mdestafadilah]

'mbstring' extension is not loaded. This is required for Grav to run correctly.

grav use 'ext-mbstring'.

[Rotzbua]

That is not correct, even if the doc claims it:
https://learn.getgrav.org/17/basics/requirements#php-requirements

The truth is that the extension mbstring is just optional:

grav/composer.json

Lines 78 to 79 in 2c252c4

	"suggest": {
	"ext-mbstring": "Recommended for better performance",

Because of the polyfill:

grav/composer.json

Line 22 in 2c252c4

"symfony/polyfill-mbstring": "~1.23",

If this is really a problem there must be at least one single test failing.

[mdestafadilah]

@Rotzbua hi, i see too.. but i'm testing it in heroku.

any update here?

[Rotzbua]

@mdestafadilah I am not familiar with your setup nor your use case. The best is to create a detailed issue or provide a proper solution as pull request.

[mdestafadilah]

@mdestafadilah I am not familiar with your setup nor your use case. The best is to create a detailed issue or provide a proper solution as pull request.

hems. may your self test it with fresh install to heroku. in my own server still using PHP 8.0 for production but at heroku using PHP 8.1 for development.

just wait others to test it.

[mahagr]

@mdestafadilah Error should be fixed now, but I recommend installing the PHP extension as it is much faster.

[Rotzbua]

@mdestafadilah Stop pointing at me. Stop trying to make your problems to my problems. If you are too lazy to create a proper issue it is your fault.