PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
30 Mar 2023

Pull Request for Issue # .

Summary of Changes

This pull request (PR) removes the special handling for the old search component's package extension which was once added with PR #35164 for updates from 3.10 to 4.x.

The update SQL scripts have already been removed with PR #40083 . This PR here removes the rest.

The PR is one in a sequence of PR's for removing obsolete actions from script.php (and other places like here in build.php) which are not relevant anymore because updates to 5.x will have to be done from a 4.4 version, like it is now with 4.x from 3.10.

Testing Instructions

Code review: Make sure that there is not anything related to "pkg_search" left in PHP code or SQL files or anywhere else, and that the manifest file "pkg_search.xml" is deleted on GitHub.

Optional real test: Make sure that updating the latest 4.4-dev nightly build to the update package created by drone works when using the upload & update method. Make also sure that the build.php script still works, i.e. run php ./build/build.php --remote=HEAD --exclude-gzip --exclude-bzip2 on the branch of this PR.

Actual result BEFORE applying this Pull Request

Obsolete manifest file "pkg_search.xml" is present in update packages, and obsolete code in script.php and build.php related to that file is present.

Expected result AFTER applying this Pull Request

No manifest file "pkg_search.xml" is present in update packages, and no obsolete code in script.php and build.php related to that file is present.

Updating from a 4.4-dev nightly build works.

php ./build/build.php --remote=HEAD --exclude-gzip --exclude-bzip2 still works. The update package doesn't contain the "pkg_search.xml" manifest file.

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2023
Category Administration com_admin Repository
avatar richard67 richard67 - open - 30 Mar 2023
avatar richard67 richard67 - change - 30 Mar 2023
Status New Pending
avatar richard67 richard67 - change - 30 Mar 2023
The description was changed
avatar richard67 richard67 - edited - 30 Mar 2023
avatar richard67 richard67 - change - 30 Mar 2023
Labels Added: PR-5.0-dev
avatar HLeithner
HLeithner - comment - 30 Mar 2023

Just to be sure, this doesn't effect if com_search is installed right?

avatar HLeithner HLeithner - change - 30 Mar 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-03-30 19:07:46
Closed_By HLeithner
avatar HLeithner HLeithner - close - 30 Mar 2023
avatar HLeithner HLeithner - merge - 30 Mar 2023
avatar richard67
richard67 - comment - 30 Mar 2023

Just to be sure, this doesn't effect if com_search is installed right?

@HLeithner Correct. If the search component is installed, it will be ok, just the missing package has been already added with the update from 3.10 to 4.x, and that part is removed here.

Add a Comment

Login with GitHub to post a comment