Skip to content

[5.0] fix path build_incomplete#41562

Merged
HLeithner merged 2 commits intojoomla:5.0-devfrom
heelc29:5.0/build_incomplete
Sep 3, 2023
Merged

[5.0] fix path build_incomplete#41562
HLeithner merged 2 commits intojoomla:5.0-devfrom
heelc29:5.0/build_incomplete

Conversation

@heelc29
Copy link
Contributor

@heelc29 heelc29 commented Sep 3, 2023

Summary of Changes

change start directory from JPATH_BASE to JPATH_ROOT for build_incomplete.html

for reference see #40509 @dgrammatiko

Testing Instructions

checkout this repo (without install composer/npm)

  • visit [DOMAIN]/index.php
  • visit [DOMAIN]/installation/index.php
  • visit [DOMAIN]/administrator/index.php

Actual result BEFORE applying this Pull Request

when visit [DOMAIN]/administrator/index.php following warning appears:
Warning: file_get_contents(/administrator/templates/system/build_incomplete.html): Failed to open stream: No such file or directory in /administrator/includes/app.php on line 31

Expected result AFTER applying this Pull Request

always show error page build_incomplete
image

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@HLeithner HLeithner merged commit e28abb8 into joomla:5.0-dev Sep 3, 2023
@HLeithner
Copy link
Member

thanks

@heelc29 heelc29 deleted the 5.0/build_incomplete branch September 3, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants