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

[4.0] Installer library query fixes #22536

Merged
merged 2 commits into from Oct 10, 2018
Merged

[4.0] Installer library query fixes #22536

merged 2 commits into from Oct 10, 2018

Conversation

mbabker
Copy link
Contributor

@mbabker mbabker commented Oct 7, 2018

Pull Request for Issues #21958 & #22531

Summary of Changes

As it seems nobody else will fix database query related issues, here's the pull request fixing two reported issues with queries in the extension installer library:

  • Renames a query parameter since parameter substitution is incorrectly replacing the second parameter
  • Adds a missing use statement for a PHP class

Testing Instructions

  • Code review
  • Attempt to uninstall a component or plugin

Expected result

Operations complete successfully

Actual result

Operations do not complete

Additional Comments

Any other style related fixes or errors are out of scope of this pull request and will not be addressed by me. Far too often unrelated fixes are being requested in pull requests making it easier for pull requests to introduce more changes and newer bugs, changes should be isolated and tested/reviewed separately.

@ReLater
Copy link
Contributor

ReLater commented Oct 7, 2018

I have tested this item ✅ successfully on 5c873d7


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

1 similar comment
@ghazal
Copy link
Contributor

ghazal commented Oct 8, 2018

I have tested this item ✅ successfully on 5c873d7


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

@Quy
Copy link
Contributor

Quy commented Oct 8, 2018

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 8, 2018
@wilsonge wilsonge merged commit 59cc6fc into joomla:4.0-dev Oct 10, 2018
@joomla-cms-bot joomla-cms-bot added PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Oct 10, 2018
@wilsonge
Copy link
Contributor

Thanks!

@wilsonge wilsonge added this to the Joomla 4.0 milestone Oct 10, 2018
@mbabker mbabker deleted the installer-fixes branch October 10, 2018 13:01
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

6 participants