Skip to content

Conversation

@richard67
Copy link
Member

@richard67 richard67 commented Mar 30, 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

@HLeithner
Copy link
Member

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

@HLeithner HLeithner merged commit 386284b into joomla:5.0-dev Mar 30, 2023
@richard67
Copy link
Member Author

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.

@richard67 richard67 deleted the 5.0-dev-script-php-clean-up-pkg-search branch March 30, 2023 19:09
@richard67 richard67 added this to the Joomla! 5.0 milestone Apr 4, 2023
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.

3 participants