search

HugoFara / lwt

Learn languages by reading! A language learning app stemmed from Learning with Texts (LWT).
https://hugofara.github.io/lwt/
The Unlicense
164 stars 19 forks source link

PHP warning on index.php with Docker install #160

Closed PeterHero closed 6 months ago

PeterHero commented 7 months ago

Describe the bug A docker installed lwt gets php warning on index.php:

To Reproduce Steps to reproduce the behavior:

  1. Start Docker by docker compose up
  2. Go to 'http://localhost:8010/index.php'
  3. See warning PHP Warning: file_get_contents(UNLICENSE.md): Failed to open stream: No such file or directory in /var/www/html/inc/session_utility.php on line 5238

Expected behavior See no warning

Server (please complete) Data Value LWT version v002008001 Web Server Apache/2.4.38 (Debian) Server Software Apache/2.4.38 Server Location localhost:8010 PHP Version 8.1.20

Desktop (if necessary):

Additional context My lwt is installed with docker.

ProgramComputer commented 7 months ago

@HugoFara All .md files are ignored in https://github.com/HugoFara/lwt/blob/403b2b484ea846baa8114ca28478d8c89ca90f4c/.dockerignore#L16 . Related to docker help #146 .

HugoFara commented 7 months ago

I removed the line from .dockerignore. On the next push on master the Docker image will contain the necessary files.

Reason: I don't see a real reason to ignore README.md and UNLICENSE.md.