linuxserver / docker-dokuwiki

GNU General Public License v3.0
110 stars 24 forks source link

[FEAT] Addition of php8.2-intl #67

Closed spacecake closed 9 months ago

spacecake commented 9 months ago

Is this a new feature request?

Wanted change

Add php8.2-intl to the docker base image.

Reason for change

Hello

The Dokuwiki plugin Combo gives errors when installed.

The critical error received: ` Error: Class "IntlDateFormatter" not found

An unforeseen error has occured. This is most likely a bug somewhere. It might be a problem in the combo plugin.

More info has been written to the DokuWiki error log. `

In my brief research I found several solution pointing to this change: stackoverflow, In the docker image a "find / -name php* 2>/dev/null" did not give me answers and did not find any in /usr/lib/php82/modules

Did I missed something?

Proposed code change

Kindly, include php8.2-intl extension in the image.

github-actions[bot] commented 9 months ago

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

Roxedus commented 9 months ago

This looks like a 3rd-party plugin for docuwiki, and I don't see why this extension is needed in the image. You can however use the Package Install mod to install it in your instance.

spacecake commented 9 months ago

Solution:

environment:

  • DOCKER_MODS=linuxserver/mods:universal-package-install
  • INSTALL_PACKAGES=php82-intl

Thank you Roxedus!