[#17766] - jalilyacob
- Closed
- 30 Aug 2017
- Medium
- Build: 3.9-dev
- # 17766
- Diff
- joomla:staging
- Success continuous-integration/drone/push the build was successful Details
- Success continuous-integration/travis-ci/push The Travis CI build passed Details
- Success continuous-integration/appveyor/branch AppVeyor build succeeded Details
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
JDatabaseQuery: Add support to multiple-table deletion with Join of the delete() function.
Testing Instructions
$query->delete('a, b')->from('#__a AS a')->leftJoin('#__b AS b ON a.id = b.id')->where('a.id = 1');
Expected result
DELETE a, b FROM #__a AS a LEFT JOIN #__b AS b ON a.id = b.id WHERE a.id = 1;
Actual result
DELETE a, b FROM #__a AS a LEFT JOIN #__b AS b ON a.id = b.id WHERE a.id = 1;
Documentation Changes Required
Added the following usage instruction:
Other Usage (Added to support multiple-delete using join):
$query->delete('a, b')->from('#__a AS a')->leftJoin('#__b AS b ON a.id = b.id')->where('a.id = 1');
Output Query:
DELETE a, b FROM #__a AS a LEFT JOIN #__b AS b ON a.id = b.id WHERE a.id = 1;
joomla-cms-bot
- | Category | ⇒ | Administration com_admin com_categories com_contact com_content com_finder com_menus com_modules com_newsfeeds Language & Strings Modules Front End com_search |
| Status | New | ⇒ | Pending |
I am closing this at this time. Please resubmit your pr to the staging branch
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-30 08:05:40 |
| Closed_By | ⇒ | brianteeman | |
| Labels |
Added:
?
?
|
||
I guess you messed up your branch. Also please give the pull request a proper title.