search

dsavell / docker-grav

Docker Container for GRAV CMS
MIT License
40 stars 18 forks source link

Complains of 'mbstring' not loaded. #12

Closed aptonline closed 4 years ago

aptonline commented 4 years ago

Hi, I finally got around to testing the new images you made for the path issue I posted but upon using the new images I received an error saying that 'mbstring' was not loaded.

I fixed the issue by installing manually.

su
apt-get update
apt-get install php-mbstring

No sure if you need to check its installed as standard with the images.

dsavell commented 4 years ago

Hey Thanks again,

Are you able to provide the steps on how to repeat this?

aptonline commented 4 years ago

The mbstring error appeared as soon as I went to the front end url, also showed when I went to the admin login. I downloaded the admin image to replace the one I already had. After I installed mbstring using the above method it all worked as expected.

Miroka96 commented 4 years ago

I have had the same error using the dsavell/grav:admin image and was able to fix it by running 'apt install php7.3-mbstring' inside the container and restarting the container

Miroka96 commented 4 years ago

To prevent such errors in the future it might be an advantage to make the admin image inherit from the normal image

dsavell commented 4 years ago

Thank both, will take a look as soon as I can, I have raised and issue for inheritance that makes a lot of sense. https://github.com/dsavell/docker-grav/issues/14

Will get reviewing and rebuilding asap.

dsavell commented 4 years ago

Merged accepted, should now be resolved in latest tags, please repull