Persist smileys/local and interwiki image folders outside of container

[MachX428]

Fixes custom smiley and interwiki icon support

---

• I have read the contributing guideline and understand that I have made the correct modifications

---

Description:

Following the official DokuWiki documentation for working with custom smileys (https://www.dokuwiki.org/smileys) and interwiki icons (https://www.dokuwiki.org/interwiki) does not result in the custom icons being displayed. This is because the folders the user is supposed to put the icons into are not properly persisted outside of the docker container. This PR modifies the 50-config file to make these locations persistent.

Benefits of this PR and context:

This fixes an outstanding bug and closes #40.

How Has This Been Tested?

The change has been tested by:

• Install a new container from the remote docker-dokuwiki
• Run install.php and restart the container
• Create a smileys/local folder as described in the official documentation: https://www.dokuwiki.org/smileys
  • config/dokuwiki/lib/images/smileys/local
• Copy one of the user provided smileys from https://www.dokuwiki.org/_media/wiki:usersmileys:button-danger-yellow.png
• Rename it test.png and put it in the config/dokuwiki/lib/images/smileys/local folder
• Create a config/dokuwiki/conf/smileys.local.conf file

# Testing Smileys
:SMILEYTEST:                local/test.png

• Create/edit a page in DokuWiki and use the text ":SMILEYTEST:"

====== Test ======

===== Smiley =====

:SMILEYTEST:

• Verify that the text ":SMILEYTEST:" and a generic icon appears
  • If it was working correctly, the smiley image you saved would appear instead
• Create an interwiki folder as described in the official documentation: https://www.dokuwiki.org/interwiki
  • config/dokuwiki/lib/images/interwiki
• Use your favorite paint program to create a 16x16 pixel red square and save it as ubuntu.png
  • Move it to the interwiki folder you created
• Create a config/dokuwiki/conf/interwiki.local.conf file

# Testing Interwiki's
ubuntu https://wiki.ubuntu.com/{NAME}

• Edit your DokuWiki test page and add the text "[[ubuntu>teams|Ubuntu Teams]]"

====== Test ======

===== Smiley =====

:SMILEYTEST:  

===== Interwiki =====

[[ubuntu>teams|Ubuntu Teams]]

• Verify that a generic circle icon appears next to the interwiki link you created
  • If it was working correctly, the red square icon you created would appear instead
• Build new (local) image from this PR.
• Shut down and remove the existing container (but don't delete your persistent folders)
• Clear the image cache:

rm -rf config/dokuwiki/data/cache/*/*.{gif,png,css}

• Create and start up new container using the local image you just created
• Go back to your DokuWiki test page
  • You may have to force refresh
• The correct smiley image and red interwiki icon should now be visible

Source / References:

See issue #40.

[github-actions]

Thanks for opening this pull request! Be sure to follow the pull request template!

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2020-07-29-pkg-5739847a-pr-41/index.html
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2020-07-29-pkg-5739847a-pr-41/shellcheck-result.xml

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[MachX428]

Is there anything I need to do in order to get this approved? It would be very useful to get this fixed in the repo.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[MachX428]

I'm just commenting to indicate I still have interest in this. Is there anything that I can do to help get this approved?

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[thelamer]

This doesn't work for me without:

  [[ ! -d /config/dokuwiki/lib/images/smileys ]]
    mkdir -p /config/dokuwiki/lib/images/smileys

Did you test this ?

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2020-07-29-pkg-c3609974-pr-41/index.html
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2020-07-29-pkg-c3609974-pr-41/shellcheck-result.xml

[MachX428]

Thank you. I did indeed check it, but in retrospect my test process was flawed. I started with the non-modified repo, went through the test steps to create the smileys/interwiki's and verified they DIDN'T work. I then rebuilt the repo with the changes, and verified that the previous non-functioning pages now worked. However, since my steps had added the /config/dokuwiki/lib/images folder, it masked the fact I needed to check for that in the 50-config file as well.

I have added the following to 50-config:

[[ ! -d /config/dokuwiki/lib/images/ ]]
mkdir -p /config/dokuwiki/lib/images/

and merged/rebased with the latest. I then started completely from scratch, with no existing folders, rebuilt the container with the changes, ran it, and verified everything works as expected. I then updated my PR branch with the changes. Thanks again for pointing this out.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2020-07-29-pkg-cd7de6ad-pr-41/index.html
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2020-07-29-pkg-cd7de6ad-pr-41/shellcheck-result.xml

[nemchik]

@MachX428 this looks fine to me, but it needs to be rebased at this point. Can you rebase and then ping me back?

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/index.html
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/shellcheck-result.xml

[MachX428]

@nemchik - I rebased as per requested, and validated all works as expected.

[nemchik]

LGTM. I made two commits to adjust things a bit (the new mkdir logic you added is combined with the one above it, and nested one more level, and the conditionals are switched to using full if syntax).

Once our CI builds the image if you wouldn't mind testing (I'll specifically mention the tag) i'll approve and merge this.

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/index.html
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/shellcheck-result.xml

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/index.html
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/shellcheck-result.xml

[nemchik]

lspipepr/dokuwiki:2022-07-31a-pkg-5b04a879-pr-41 would be the tag to test.

[MachX428]

@nemchik - I tested that tag and all works well.

That being said, I'm thinking one more slight change might make this just a little more robust and have less possibility for confusion.

When I originally did the change, I just created the base lib/images directory. If you then follow the official smiley instructions here https://www.dokuwiki.org/smileys and create the /smileys/local directory under /images, and put your smileys there, it works. I figured anybody creating the images/smileys directory would obviously be reading the instructions and also know to create the /local directory under it. But now with your changes, it creates actually the /images/smileys directory for you. And at this point it almost makes it seem like you can just put the custom smileys there since that directory exists. Of course if you do that, it won't work because the symlink is to /images/smileys/local as per the official instructions. So my suggestion is we just bite the bullet and create the whole /images/smileys/local path at this point to just avoid any confusion. It is more intuitive and probably what I should have just done in the first place. I am happy to make the change, test it, and push it up for you if you agree.

[MachX428]

I went ahead and made the change I described, and retested. All looks well on my end if you want to add that in.

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/index.html
https://ci-tests.linuxserver.io/lspipepr/dokuwiki/2022-07-31a-pkg-5b04a879-pr-41/shellcheck-result.xml