Skip to content

Create "update:check-modules" command #1528

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
Merged
Quetzacoalt91 merged 6 commits into from
Jan 9, 2026
Merged

Create "update:check-modules" command #1528

Quetzacoalt91 merged 6 commits into from
Jan 9, 2026
+752 −4

Conversation

@M0rgan01
Copy link
Contributor

@M0rgan01 M0rgan01 commented Nov 14, 2025 •
edited
Loading

Questions Answers
Description? Added a command to check which modules will be updated, which become incompatible, or which do not change version during the update process.

The channel can be specified (ONLINE, ONLINE_RECOMMENDED, LOCAL). An example of the output is available below. This data uses only marketplace data.

This order does not modify anything on the store; it is present for informational purposes only.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company -
How to test? use new update:check-modules command
image

@M0rgan01 M0rgan01 force-pushed the modules-invest branch 14 times, most recently from a2f9e2e to 8a74759 Compare November 17, 2025 15:27
@M0rgan01 M0rgan01 changed the title Create "CheckModulesCommand" Create "update:check-modules" command Nov 17, 2025
@M0rgan01 M0rgan01 marked this pull request as ready for review November 17, 2025 15:31
@Quetzacoalt91 Quetzacoalt91 added this to the 7.6.0 milestone Nov 18, 2025
Quetzacoalt91
Quetzacoalt91 reviewed Nov 18, 2025
View reviewed changes
@M0rgan01 M0rgan01 force-pushed the modules-invest branch 2 times, most recently from 8a4af39 to 552a74d Compare November 19, 2025 08:26
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 19, 2025

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Quetzacoalt91 Quetzacoalt91 mentioned this pull request Nov 27, 2025
Merged
@Quetzacoalt91 Quetzacoalt91 mentioned this pull request Dec 12, 2025
Open
Nakahiru
Nakahiru previously approved these changes Jan 7, 2026
View reviewed changes
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Jan 7, 2026
@Quetzacoalt91 Quetzacoalt91 added the enhancement Type: Improvement label Jan 7, 2026
ga-devfront
ga-devfront previously approved these changes Jan 7, 2026
View reviewed changes
@AureRita AureRita self-assigned this Jan 8, 2026
@M0rgan01 M0rgan01 dismissed stale reviews from ga-devfront and Nakahiru via 26c0cb8 January 8, 2026 15:35
Quetzacoalt91
Quetzacoalt91 reviewed Jan 8, 2026
View reviewed changes
classes/Services/MarketplaceService.php
/** @var Translator */
private $translator;

const ADDONS_API_URL = 'https://api.addons.prestashop.com';
Copy link
Member

@Quetzacoalt91 Quetzacoalt91 Jan 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This constant will be set two times in the project, here and in classes/UpgradeTools/Module/Source/Provider/MarketplaceSourceProvider.php. Can we merge them in a single constant?

Copy link
Contributor Author

@M0rgan01 M0rgan01 Jan 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 8, 2026

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

AureRita
AureRita approved these changes Jan 9, 2026
View reviewed changes
Copy link
Contributor

@AureRita AureRita left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @M0rgan01

Thank you for your PR, I tested it and it seems to works as you can see :

Capture.video.du.2026-01-08.17-29-20.mp4

Command tested :

php bin/console update:check-modules --help ✔️
php bin/console update:check-modules --zip=prestashop_9.1.0.zip admin-devphp ✔️
bin/console update:check-modules --channel=local --zip=prestashop_9.1.0.zip admin-dev ✔️
bin/console update:check-modules --channel=local admin-dev ✔️
bin/console update:check-modules --channel=online admin-dev ✔️
bin/console update:check-modules --channel=online_recommanded admin-dev ✔️

Tested on :
8.1.4
8.2.3
9.0.2

Because the PR seems to works as expected, It's QA ✅ :

Thank you

@Quetzacoalt91 Quetzacoalt91 merged commit bc0882e into PrestaShop:dev Jan 9, 2026
56 checks passed
@Quetzacoalt91 Quetzacoalt91 deleted the modules-invest branch January 9, 2026 10:10
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Jan 9, 2026
@ps-jarvis ps-jarvis moved this from Merged to Ready for review in PR Dashboard Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Quetzacoalt91 Quetzacoalt91 Quetzacoalt91 approved these changes

@ga-devfront ga-devfront ga-devfront approved these changes

@AureRita AureRita AureRita approved these changes

@Nakahiru Nakahiru Nakahiru left review comments

Assignees

@M0rgan01 M0rgan01

@AureRita AureRita

Labels

enhancement Type: Improvement QA ✔️

Projects

PR Dashboard
Status: Ready for review

Milestone

7.6.0

Development

Successfully merging this pull request may close these issues.

6 participants

@M0rgan01 @Quetzacoalt91 @Nakahiru @ga-devfront @AureRita @ps-jarvis