Skip to content

[5.4] fix installer disk error - #48166

Open
Raheem54 wants to merge 10 commits into
joomla:5.4-devfrom
Raheem54:update-fix-installer-disk-error
Open

[5.4] fix installer disk error#48166
Raheem54 wants to merge 10 commits into
joomla:5.4-devfrom
Raheem54:update-fix-installer-disk-error

Conversation

@Raheem54

@Raheem54 Raheem54 commented Jul 30, 2026

Copy link
Copy Markdown

Pull Request resolves #48156.

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

update misleading installer error when disk space/quota exceeded by check free space before error

closes #48165

Testing Instructions

try to install any extension on system with full space and observe error message

Actual result BEFORE applying this Pull Request

displays: "install path does not exist"

Expected result AFTER applying this Pull Request

displays: "The filesystem is full or the user's disk quota has been exceeded."

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@brianteeman

Copy link
Copy Markdown
Contributor

this doesn't really solve the problem it just delays it. Perhaps a more sensible check would be against the size of the file being written to temp and also ensuring that there is enough space to extract it in the temp and then copy it to the correct location

@Raheem54

Copy link
Copy Markdown
Author

do you mean to compare size source to free size in extension root in package adapter ?

@brianteeman

Copy link
Copy Markdown
Contributor

what i mean is that your current code won't solve the reported problem.

@Raheem54

Raheem54 commented Jul 30, 2026

Copy link
Copy Markdown
Author

is this edit will solve it ?

@tecpromotion tecpromotion changed the title fix installer disk error [5.4] fix installer disk error Aug 8, 2026
Comment thread installation/language/en-GB/joomla.ini
Comment thread libraries/src/Installer/Installer.php
Comment thread libraries/src/Installer/Installer.php
@Raheem54

Raheem54 commented Aug 12, 2026

Copy link
Copy Markdown
Author

I made all the changes and test failed I don't know why I'm just change the style, I think it's related to update the branch but there is no any conflicts I don't know why it failed

@Raheem54
Raheem54 requested a review from QuyTon August 12, 2026 19:16
@richard67

Copy link
Copy Markdown
Member

I made all the changes and test failed I don't know why I'm just change the style, I think it's related to update the branch but there is no any confects I don't know why it failed

@Raheem54 Confects ... :-) I think you meant conflicts. System tests failed due to a timeout when testing extension installation from URL. That can happen sometimes. Not related to your PR. I've just restarted the failed CI jobs. Let's see if they pass this time.

@Raheem54

Copy link
Copy Markdown
Author

@Raheem54 Confects ... :-) I think you meant conflicts. System tests failed due to a timeout when testing extension installation from URL. That can happen sometimes. Not related to your PR. I've just restarted the failed CI jobs. Let's see if they pass this time.

It failed again and I don't know how It was passing all tests before the update

@Raheem54
Raheem54 requested a review from richard67 August 12, 2026 19:37
@richard67
richard67 removed their request for review August 12, 2026 19:39
@Raheem54
Raheem54 requested a review from richard67 August 13, 2026 14:41
Comment thread libraries/src/Installer/Installer.php Outdated
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators PR-5.4-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Installer reports misleading "Install path does not exist" when disk quota is exceeded

5 participants