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.

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

[5][cli] maintenance database update db structure #42568

Merged
merged 6 commits into from
Mar 4, 2024

Conversation

alikon
Copy link
Contributor

@alikon alikon commented Dec 26, 2023

Pull Request for Issue #30477

Summary of Changes

added cli command like CMS "Maintenance: Database - Update Structure"

Testing Instructions

from cli run
php cli/joomla.php maintenance:database
or
php cli/joomla.php maintenance:database --fix

Actual result BEFORE applying this Pull Request

N/A

Expected result AFTER applying this Pull Request

same as "Maintenance: Database - Update Structure" from CMS

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

@dautrich
Copy link

dautrich commented Dec 30, 2023

I've tested this PR locally with XAMPP (PHP 8.2.12).

On J5.1_Alpha2 the command doesn't exist:
Bildschirminhalt erfassen-5


On 5.1.0-alpha2-dev+pr.42568 the command exists.

  • After installation, the database structure was not up-to-date:

Bildschirminhalt erfassen-2

  • I did: Maintenance - Update Database structure:

Bildschirminhalt erfassen-3

  • Before and after the database repair, I issued the CLI command:

Bildschirminhalt erfassen-4

@dautrich
Copy link

I have tested this item ✅ successfully on 163552f


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

Co-authored-by: Quy <quy@nomonkeybiz.com>
@Hackwar Hackwar added the PBF Pizza, Bugs and Fun label Feb 21, 2024
@nielsnuebel
Copy link
Contributor

I have tested this item ✅ successfully

@nielsnuebel
Copy link
Contributor

I have tested this item ✅ successfully on 02534fa


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

1 similar comment
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 02534fa


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 24, 2024
@bembelimen bembelimen self-assigned this Mar 1, 2024
@bembelimen bembelimen merged commit 93912f5 into joomla:5.1-dev Mar 4, 2024
0 of 2 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 4, 2024
@bembelimen bembelimen added this to the Joomla! 5.1.0 milestone Mar 4, 2024
@bembelimen
Copy link
Contributor

Thanks

@alikon alikon deleted the fix-structure branch March 4, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature PBF Pizza, Bugs and Fun PR-5.1-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants