Commit 84bad295 authored by drumm's avatar drumm

#72865 by AjK, use more standard is_numeric() instead of ctype_digit().

parent 9adf9d8c
......@@ -924,7 +924,7 @@ function comment_delete($cid) {
// We'll only delete if the user has confirmed the
// deletion using the form in our else clause below.
if (is_object($comment) && ctype_digit($comment->cid) && $_POST['edit']['confirm']) {
if (is_object($comment) && is_numeric($comment->cid) && $_POST['edit']['confirm']) {
drupal_set_message(t('The comment and all its replies have been deleted.'));
// Delete comment and its replies.
......@@ -937,7 +937,7 @@ function comment_delete($cid) {
drupal_goto("node/$comment->nid");
}
else if (is_object($comment) && ctype_digit($comment->cid)) {
else if (is_object($comment) && is_numeric($comment->cid)) {
$output = confirm_form('comment_confirm_delete',
array(),
t('Are you sure you want to delete the comment %title?', array('%title' => theme('placeholder', $comment->subject))),
......@@ -1105,7 +1105,7 @@ function comment_multiple_delete_confirm() {
$comment_counter = 0;
foreach (array_filter($edit['comments']) as $cid => $value) {
$comment = _comment_load($cid);
if (is_object($comment) && ctype_digit($comment->cid)) {
if (is_object($comment) && is_numeric($comment->cid)) {
$subject = db_result(db_query('SELECT subject FROM {comments} WHERE cid = %d', $cid));
$form['comments'][$cid] = array('#type' => 'hidden', '#value' => $cid, '#prefix' => '<li>', '#suffix' => check_plain($subject) .'</li>');
$comment_counter++;
......@@ -1657,7 +1657,7 @@ function theme_comment_post_forbidden($nid) {
}
function _comment_delete_thread($comment) {
if (!is_object($comment) || !ctype_digit($comment->cid)) {
if (!is_object($comment) || !is_numeric($comment->cid)) {
watchdog('content', t('Can not delete non-existent comment.'), WATCHDOG_WARNING);
return;
}
......
......@@ -924,7 +924,7 @@ function comment_delete($cid) {
// We'll only delete if the user has confirmed the
// deletion using the form in our else clause below.
if (is_object($comment) && ctype_digit($comment->cid) && $_POST['edit']['confirm']) {
if (is_object($comment) && is_numeric($comment->cid) && $_POST['edit']['confirm']) {
drupal_set_message(t('The comment and all its replies have been deleted.'));
// Delete comment and its replies.
......@@ -937,7 +937,7 @@ function comment_delete($cid) {
drupal_goto("node/$comment->nid");
}
else if (is_object($comment) && ctype_digit($comment->cid)) {
else if (is_object($comment) && is_numeric($comment->cid)) {
$output = confirm_form('comment_confirm_delete',
array(),
t('Are you sure you want to delete the comment %title?', array('%title' => theme('placeholder', $comment->subject))),
......@@ -1105,7 +1105,7 @@ function comment_multiple_delete_confirm() {
$comment_counter = 0;
foreach (array_filter($edit['comments']) as $cid => $value) {
$comment = _comment_load($cid);
if (is_object($comment) && ctype_digit($comment->cid)) {
if (is_object($comment) && is_numeric($comment->cid)) {
$subject = db_result(db_query('SELECT subject FROM {comments} WHERE cid = %d', $cid));
$form['comments'][$cid] = array('#type' => 'hidden', '#value' => $cid, '#prefix' => '<li>', '#suffix' => check_plain($subject) .'</li>');
$comment_counter++;
......@@ -1657,7 +1657,7 @@ function theme_comment_post_forbidden($nid) {
}
function _comment_delete_thread($comment) {
if (!is_object($comment) || !ctype_digit($comment->cid)) {
if (!is_object($comment) || !is_numeric($comment->cid)) {
watchdog('content', t('Can not delete non-existent comment.'), WATCHDOG_WARNING);
return;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment