linuxserver / docker-dokuwiki

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

Added symlinks from /app/dokuwiki to /config/dokuwiki for core dokuwiki directories #18

Closed avks closed 4 years ago

avks commented 4 years ago

Description:

Certain plugins cannot find the path to core dokuwiki directories/files. This is due to the fact that the existing image separates core files from user files, which is handled by 50-config.

Benefits of this PR and context:

This will fix problems caused due to calls by certain plugins, to incorrect/non-existent path. More details can be found at #9

How Has This Been Tested?

After making changes to 50-config, I built the image:

docker build -t avks/docker-dokuwiki .

Next I created a container with the same image:

docker run --name avks-dokuwiki -e PUID=1000 -e PGID=1000 -e TZ=Asia/Kolkata -e APP_URL=/dokutest -p 80:80 -v $(pwd)/dokuconfig:/config --restart unless-stopped -d avks/docker-dokuwiki

After initial dokuwiki setup, I logged into the admin panel and installed searchindex plugin, with which I initially had issues. The plugin worked fine without throwing any error.

My testing environment was:

Notes

I really admire how linuxserver team has separated the core files from user files. This PR will keep it that way, since the mounted volume (on host system) does not expose the core files.

Source / References:

Issue #9

LinuxServer-CI commented 4 years ago

I am a bot, here are the test results for this PR: https://lsio-ci.ams3.digitaloceanspaces.com/lspipepr/dokuwiki/2020-07-29-pkg-06bf329e-pr-18/index.html https://lsio-ci.ams3.digitaloceanspaces.com/lspipepr/dokuwiki/2020-07-29-pkg-06bf329e-pr-18/shellcheck-result.xml