-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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] [com_content] [wip] convert to prepared statement #25179
Conversation
@@ -50,6 +51,7 @@ public function association($articleid) | |||
} | |||
|
|||
// Get the associated menu items | |||
$associations = array_values($associations); | |||
$db = Factory::getDbo(); | |||
$query = $db->getQuery(true) | |||
->select('c.*') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Convert 57-67.
@@ -171,13 +181,13 @@ public function getItem($pk = null) | |||
{ | |||
throw new \Exception(Text::_('COM_CONTENT_ERROR_ARTICLE_NOT_FOUND'), 404); | |||
} | |||
|
|||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes i wasn't able to understand what's the meaning of that if
, and plus testing work well without
any help ?
Co-Authored-By: Quy <quy@fluxbb.org>
Co-Authored-By: Quy <quy@fluxbb.org>
Co-Authored-By: Quy <quy@fluxbb.org>
Co-Authored-By: Quy <quy@fluxbb.org>
Co-Authored-By: Quy <quy@fluxbb.org>
Co-Authored-By: Quy <quy@fluxbb.org>
Co-Authored-By: Quy <quy@fluxbb.org>
Co-Authored-By: Quy <quy@fluxbb.org>
@alikon it would be cool if can bring this PR back. |
re-opened as requested, but it will conflict with #25760 |
sorry but too much changes since it was opened,or i'm a git dumb, but ,i cannot manage it, better to start from scratch with a new one |
Summary of Changes
use prepared statement for SQL
Testing Instructions
test com_content
Expected result
should work as before
Actual result
N/A