[4.0] Correctly handle multiple critical extensions within an package that is joomla 4 compatible

[richard67]

Pull Request for Issue #34968 .

Summary of Changes

Port PR #34776 to the 4.0-dev branch.

Testing Instructions

1. Install latest akeeba, enable the system plugin (backup on update) and the action log plugin.
2. Point your update server to the Joomla 4 nightlies (custom URL: https://update.joomla.org/core/nightlies/next_major_list.xml) in order to get the pre-update checker shown.
3. Apply this PR
4. Clear the browser / JS cache.

Actual result BEFORE applying this Pull Request

The pre upgrade checker shows a message about potentially not compatible plugins within that package.

Expected result AFTER applying this Pull Request

The pre update checker shows no message about that plugins cause they are compatible..

Documentation Changes Required

None.

[dgrammatiko]

@richard67 you can filter the array:

if (extensionData.compatibilityData.resultGroup === 3) {
  PreUpdateChecker.nonCoreCriticalPlugins = PreUpdateChecker.nonCoreCriticalPlugins
    .filter((plugin) => plugin.package_id === extensionId || plugin.extension_id === extensionId);
}

[richard67]

@richard67 you can filter the array:

if (extensionData.compatibilityData.resultGroup === 3) {
  PreUpdateChecker.nonCoreCriticalPlugins = PreUpdateChecker.nonCoreCriticalPlugins
    .filter((plugin) => plugin.package_id === extensionId || plugin.extension_id === extensionId);
}

I should invert the condition, I think, because the old code removed elements from the array if matching that condition.

[richard67]

Setting release blocker as inherited from the issue.

[joomdonation]

I should invert the condition, I think, because the old code removed elements from the array if matching that condition.

I think the original code suggested by @dgrammatiko is correct. Filter will remove element from array when condition match, so we should not invert the condition. Could you please check that?

[richard67]

@joomdonation Are you sure? https://www.javascripttutorial.net/javascript-array-filter/ https://developer.mozilla.org/de/docs/Web/JavaScript/Reference/Global_Objects/Array/filter and so on ... either I understand them wrong or you are wrong.

[joomdonation]

@richard67 Sorry, my bad.

[joomdonation]

I have tested this item ✅ successfully on 509a356

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34971.}

[hans2103]

BEFORE

AFTER

[hans2103]

I have tested this item ✅ successfully on 509a356

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34971.}

[richard67]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34971.}

[Kostelano]

@hans2103 Sorry for the offtopic. Can you please tell me how to get such a page? Which update server should I use? Thanks in advance.

[richard67]

@Kostelano

• Point your update server to the Joomla 4 nightlies (custom URL: https://update.joomla.org/core/nightlies/next_major_list.xml) in order to get the pre-update checker shown.

• In case if you are on a current 4.0-dev branch or latest nightly so your version is already 4.0-rc6-dev, edit file libraries/src/Version.php and change const EXTRA_VERSION = 'rc6-dev'; to const EXTRA_VERSION = 'rc6-dev';. Then reload the page and use again the button to check for updates, and then the pre-update check should be shown.

[richard67]

Thanks all - testers, merger, and @dgrammatiko for the idea to use array.filter.