search

linuxserver / docker-grocy

A container for grocy - the ERP application for your kitchen https://grocy.info
GNU General Public License v3.0
333 stars 44 forks source link

Version 3.0.0 missing php7-ctype package #26

Closed osuka closed 3 years ago

osuka commented 3 years ago

linuxserver.io

The latest version needs a dependency that is not present (php7-ctype). Otherwise errors pop up during normal operations.

Expected Behavior

Should work with no errors displayed.

Current Behavior

Complains with a red error popup on screen when browsing. Logs on the browser js console too.

Steps to Reproduce

  1. This is a straightforward error. Just using the application normally will trigger the error in /stockoverview

The fix is to simply add php7-ctype to the Dockerfiles It can be fixed in a running container by opening a terminal session inside an running apk update; apk add php7-ctype.

Environment

OS: Linux CPU architecture: x86_64 How docker service was installed: Synology launch

Command used to create docker container (run/create/compose/screenshot)

Standard docker run command from latest.

Docker logs

Nothing shown on docker logs regarding this error.

github-actions[bot] commented 3 years ago

Thanks for opening your first issue here! Be sure to follow the issue template!

Forceu commented 3 years ago

Temporary fix until the PR is merged:

Execute docker exec -it YOUR_CONTAINER_NAME apk add php7-ctype and restart the container

aptalca commented 3 years ago

fixed in #27