Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PHP warning on index.php with Docker install #160

Closed
PeterHero opened this issue Dec 15, 2023 · 2 comments
Closed

PHP warning on index.php with Docker install #160

PeterHero opened this issue Dec 15, 2023 · 2 comments
Labels
bug Something isn't working ui Any problem related to the User Interface

Comments

@PeterHero
Copy link

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

  • LWT version: [e.g. 2.8.0]

Desktop (if necessary):

  • OS: Arch linux
  • Browser firefox
  • Version 120

Additional context
My lwt is installed with docker.
@HugoFara HugoFara added bug Something isn't working ui Any problem related to the User Interface labels Dec 27, 2023
@ProgramComputer
Copy link

@HugoFara All .md files are ignored in

lwt/.dockerignore

Line 16 in 403b2b4

*.md
.
Related to docker help #146 .

@HugoFara
Copy link
Owner

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.

@HugoFara HugoFara linked a pull request Dec 31, 2023 that will close this issue
[BUG] Help not found in Docker #146
Merged
@HugoFara HugoFara removed a link to a pull request Dec 31, 2023
[BUG] Help not found in Docker #146
Merged
HugoFara added a commit that referenced this issue Jan 3, 2024
@HugoFara
Merge all features from dev branch.
0bc776f 
New databse migration strategy.
Fixes feeds (#168).
Adds missing documentation to Docker (#146, #160).
Changes in PHP and JS globals.
Fixes reading position was not set.
Read text through API (#153, #155).
Fixes word was not saved/deleted.
Fixes #170 and #69.
Updates API (#175).
Adds dependency to php-xml (#178, #181).
Updates makefile (#179).
Adds MeCab support on Mac (#135).
Adds the option to hide/show word romanization (#119).
Raises URL size limit to 2048 (#144).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working ui Any problem related to the User Interface
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ProgramComputer @HugoFara @PeterHero