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

[3.10] Disable the critical plugins check for non-major updates. #35105

Merged
merged 1 commit into from Aug 13, 2021

Conversation

zero-24
Copy link
Contributor

@zero-24 zero-24 commented Aug 12, 2021

Summary of Changes

Disable the critical plugins check for non-major updates for example when updating between 3.10 -> 3.10

Testing Instructions

Install 3.10-rc2
Install https://regularlabs.com/whatnothing
Use the nighly update server: https://update.joomla.org/core/nightlies/next_minor_list.xml

Set the update server to 4: https://update.joomla.org/core/nightlies/next_major_list.xml
than you get the warning again
image

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

None

cc @joomla/cms-release-team @HLeithner @bembelimen

@alikon
Copy link
Contributor

alikon commented Aug 13, 2021

I have tested this item ✅ successfully on ad540f7


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

@HLeithner HLeithner merged commit 0db5e12 into joomla:3.10-dev Aug 13, 2021
@zero-24 zero-24 deleted the disable_minor_plugin_warnings branch August 13, 2021 09:15
@zero-24
Copy link
Contributor Author

zero-24 commented Aug 13, 2021

Thanks 👍

@regularlabs
Copy link
Contributor

Will the same be applied to the check in Joomla 4?

@zero-24
Copy link
Contributor Author

zero-24 commented Aug 13, 2021

On a quick look i would say no cause a plugin that is installed into 4.0 should have declared himself to be compatible with 4.0 right? Or do I miss something?

@regularlabs
Copy link
Contributor

Not all extensions have (or can have) update sites attached to them. So the pre-update check is unable to check if it is compatible or not.
So that means you get warnings for every minor Joomla update.

Also: the same situation will present itself when there will be a Joomla 5...

@zero-24
Copy link
Contributor Author

zero-24 commented Aug 13, 2021

Ok fine for me than to adjust the check in 4.0 to do the check only for joomla 5 cc @wilsonge when you merge this one up.

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.

None yet

5 participants