Skip to content

[4.0] Fix js ajax for pre update checker#29980

Merged
wilsonge merged 4 commits intojoomla:4.0-devfrom
degobbis:j4dev/fixAjaxUpdateChecker
Jul 19, 2020
Merged

[4.0] Fix js ajax for pre update checker#29980
wilsonge merged 4 commits intojoomla:4.0-devfrom
degobbis:j4dev/fixAjaxUpdateChecker

Conversation

@degobbis
Copy link
Contributor

@degobbis degobbis commented Jul 5, 2020

Summary of Changes

Fix js ajax for pre update checker

Testing Instructions

  • Install an extension
  • Go to joomla update
    [update 07.07.2020]
  • Click on Options on the right corner
  • Change Update Chanel to Custom Url
  • Change Minimum Stability to Development
  • Insert https://update.joomla.org/core/nightlies/next_major_list.xml as Custom URL than Save and close
  • Check for updates

Actual result BEFORE applying this Pull Request

Notice the pre update checker cant find the combat tag
All installed extensions show the flag: Missing Compatibility Tag

grafik

Expected result AFTER applying this Pull Request

The pre update checker can now find the combat tag (npm required)
All installed extensions without update server show the flag: Missing Compatibility Tag
All installed extensions with update server but without update block with a <targetplatform> tag for Joomla 4 show the flag: No
All installed extensions with update server and an update block with a <targetplatform> tag for Joomla 4 show the flag: Yes

grafik

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jul 5, 2020
@degobbis
Copy link
Contributor Author

degobbis commented Jul 7, 2020

I have updated the test instructions because the Pre-Checker was not triggered in the default configuration.

@degobbis degobbis force-pushed the j4dev/fixAjaxUpdateChecker branch from 7d3ac74 to f0f1cd1 Compare July 7, 2020 12:52
@chmst
Copy link
Contributor

chmst commented Jul 7, 2020

I have tested this item ✅ successfully on f0f1cd1


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

@chmst
Copy link
Contributor

chmst commented Jul 7, 2020

screen shot 2020-07-07 at 20 05 28

Result after patch


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

@bonzani
Copy link

bonzani commented Jul 12, 2020

I have tested this item ✅ successfully on d876f4c


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

@degobbis
Copy link
Contributor Author

RTC

@SharkyKZ
Copy link
Contributor

RTC.


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 13, 2020
@wilsonge wilsonge merged commit 63ab8a4 into joomla:4.0-dev Jul 19, 2020
@wilsonge
Copy link
Contributor

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 19, 2020
@wilsonge wilsonge added this to the Joomla 4.0 milestone Jul 19, 2020
@degobbis degobbis deleted the j4dev/fixAjaxUpdateChecker branch July 19, 2020 17:05
dgrammatiko added a commit to dgrammatiko/joomla-cms that referenced this pull request Jul 21, 2020
…outs

* '4.0-dev' of github.com:joomla/joomla-cms: (612 commits)
  [4.0] Smart Search: Fixing ordering, order direction and disabled button (joomla#29474)
  [4.0] Generate routed Modal links for iframes when not on the root (joomla#30007)
  [4.0] Get menu directly in com_tags menu route helper (joomla#30039)
  Remove collapse when resizing from mobile to desktop (joomla#30132)
  [4.0] Wrap component output in `main` element to make Cassiopeia more accessible (joomla#29870)
  [4.0] Webauthn gmp warning (joomla#29731)
  [4.0] Refactor to return early, remove if depths and throw NotAllowed (joomla#29694)
  [4.0] CLI help text (joomla#29811)
  Feature/draggable typo fixes (joomla#29987)
  [4.0] Removing unnecessary workaround in finder indexer (joomla#30037)
  [4.0] Optimizing Smart Search for larger content (joomla#30008)
  [4.0] Fix js ajax for pre update checker (joomla#29980)
  [4.0] Cassiopea: Fixing modals custom-select fields display (joomla#30097)
  [4.0][com_fields] Fix draggable sorting (joomla#30094)
  [4.0] Correct incorrect @return documentation (joomla#30092)
  [4.0] Menu items modal: adding missing filters (joomla#30087)
  short to long php open tags with echo (joomla#30089)
  Use new Toolbar (joomla#30085)
  [4.0] Center status/date created headers (joomla#29249)
  [4.0] Fix Cassiopea searchtools alignment in modals (joomla#30077)
  ...

# Conflicts:
#	administrator/components/com_templates/src/View/Template/HtmlView.php
#	installation/sql/postgresql/base.sql
#	libraries/src/Application/AdministratorApplication.php
#	libraries/src/Application/SiteApplication.php
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

Comments