Skip to content

Conversation

brianteeman
Copy link
Contributor

Summary of Changes

Improve error message with helpful information

Testing Instructions

When Joomla is not installed (there is no configuration.php) and you try to run the cli commands php cli/joomla.php you get a message that you need to install Joomla to use the CLI

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Signed-off-by: BrianTeeman <[email protected]>
@richard67
Copy link
Member

I have tested this item ✅ successfully on 2687326


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46259.

@richard67 richard67 added the bug label Oct 10, 2025
@richard67
Copy link
Member

Setting the "bug" label as it is not really a new feature.

@tecpromotion
Copy link
Contributor

I have tested this item ✅ successfully on 2687326


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46259.

@tecpromotion
Copy link
Contributor

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46259.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 10, 2025
@richard67 richard67 added this to the Joomla! 5.4.1 milestone Oct 10, 2025
@muhme
Copy link
Contributor

muhme commented Oct 17, 2025

✅ Final test before merge with unzipped nightly build: One-line message seen before

  • Test with PR full package unzipped: Three-line messages with command (command works with copy&paste)

@muhme muhme merged commit eb688b3 into joomla:5.4-dev Oct 17, 2025
40 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Oct 17, 2025
@muhme
Copy link
Contributor

muhme commented Oct 17, 2025

Thank you @brianteeman for your contribution. Thank you @tecpromotion and @richard67 for testing.

@brianteeman brianteeman deleted the cli branch October 17, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants