[RESOLVED] trouble inserting data from a form into database

griever

On a custom wordpress blog, I am trying to allow a user to submit a comment on a post using a custom comment form to insert the comment content and comment author into the database and then load the updated page with the new comment on it. Right now I have been messing with the action= part of the form code. All my code is contained in the index.php file so I used that file for the action value, but when I do and submit a comment, it returns a fatal error:

Cannot use string offset as an array

for this line of code: $author[$row['user_nicename']][] = $row;

I've had this problem before and changed the variable names and it worked but not this time. I tried using different variable names for row and author, but it didn't help. user_nicename is referencing a part of a table.

Since I am submitting the form to the same page that displays the form, I thought I should use "$_SERVER['php_self']" instead, but when I submit a comment I get a 404 error that The requested URL /Comment does not exist.

Everything runs fine until I submit a comment. This is my index.php with the form code toward the bottom:

<?php
get_header ();
?>



<div id="box">


<?php

$qry =  "SELECT u.id, user_nicename, post_date, post_content, post_title, post_type, post_excerpt, comment_content, comment_author, comment_approved, comment_count, wp_posts.ID as post_id
         FROM wp_posts
         INNER JOIN wp_users AS u ON post_author = u.id
         LEFT JOIN wp_comments ON wp_posts.ID = comment_post_ID
         WHERE post_type='post'
         ORDER BY user_nicename ASC, post_date DESC";

$res = mysql_query($qry);

if ($res) {

while ($row = mysql_fetch_assoc($res)) {
   $author[$row['user_nicename']][] = $row;       
}

foreach($author as $k => $v) {
    $orderbydate[$v[0]['post_date']][$k] = $v;
}
krsort($orderbydate);
foreach ($orderbydate as $k => $v) {
    ksort($orderbydate[$k], SORT_STRING);
}
}



$divIds = array('one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine');
$i = 0;

foreach ($orderbydate as $date => $authors) {
    foreach ($authors as $author=> $posts) {

      echo '<div id="'.$divIds[$i++].'">';
           if($author=="tony") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/tonyname.jpg">';
           }
           elseif($author=="hippie") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hippiename.jpg">';
           }
           elseif($author=="eriku") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/erikuname.jpg">';
           }
           elseif($author=="ben") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/benname.jpg">';
           }
           elseif($author=="hingyi") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hingyiname.jpg">';
           }
           elseif($author=="chris") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/chrisname.jpg">';
           }
           elseif($author=="gabe") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/gabename.jpg">';
           }
           elseif($author=="bender") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/bendername.jpg">';
           }
            elseif($author=="jake") {
                echo '<img src="http://www.hippievstony.com/wp-content/uploads/jakename.jpg">';
           }

        $lastPost = null;
             foreach($posts as $post) {                              

                 if ($lastPost != $post['post_id']) {                         
                 echo '<h2>' . $post['post_title'] . '</h2><small>' . $post['post_date'] . '</small>';
                 echo '<p>' . $post['post_content'] . '</p>';
                 echo '<p>' . $post['post_excerpt'] . '</p>';                  






	  $post_id = $post['post_id'];
                echo '<form method="POST" action='.$_SERVER['php_self'].' Comment <input type="text" name="comment"/>';

                     if ($user_ID) {
                          echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                          echo "logged in as $user_login";
                          }
                     else echo 'Name <input type="text" name="author"/>';
                     echo '<input type="text" name="post_id" style="display: none;" value="'.$post_id.'"/>';
                     echo '<p><input name="submit" type="submit" id="submit" value="submit" tabindex="5" /></p>';

                     if (empty($_POST['comment_post_id']) || !(($post_id = (int) $_POST['comment_post_id']) > 0)) {
                          # bad comment_post_id;
                          } 
                     $sql=("INSERT INTO wp_comments(comment_author, user_id, comment_content) VALUES ('$_POST[author]','$user_login', '$_POST[comment]')");


                echo '</form>'; 





                 echo '<p>' . $post['comment_content'] . '</p>';
                 echo '<p><small>-' . $post['comment_author'] . '</small></p>';

                 }

             $lastPost = $post['post_id'];
             }


             if ($post['comment_approved']=1) {
                 if ($post['comment_author']){

                      echo '<p>' . $post['comment_content'] . '</p>';
                      echo '<p><small>-' . $post['comment_author'] . '</small></p>';
                      }
                 else echo '<p>no comment author</p>';            
            }     
            else echo '<p>comment awaiting approval</p>';    
            echo '<p>' . $post['comment_count'] . '</p>';




    echo '</div>';
}
}
get_sidebar ();
get_footer();
?>

thanks for any ideas anyone has

tony

anhxtanh3087

Price $0.00
Disk Space 3000 MB
Bandwidth Unlimited
Host your own domain Yes
Free subdomain you.200u.com you.201w.com Yes
MySQL Databases Unlimited
Multiple Package Per Account Unlimited
Allow Your Advertisement Yse
E-mail Addresses 10
FTP Accounts 10
Control Panel see demo cPanel
Autoinstaller (Softaculous) 100 scripts | List All
Parked Domains Unlimited
Add-on Domains Unlimited
Sub-domains Unlimited

http://www.free-web-host.me/

Demo: http://anhxtanh3087.200u.com/kill_ads.html

halojoy

Try $SERVER['PHP_SELF']
Maybe the capital letter matters ....

echo '<form action='.$_SERVER['PHP_SELF'].' Comment';

griever

hey halo thanks for the suggestion but I'm still getting a "Cannot use string offset as an array" for the line:

$author[$row['user_nicename']][] = $row;

halojoy

Try to change it to

$author[$row['user_nicename']] = $row;

will give
$author['tony'] = $row
and next
$author['hippie'] = $row

griever

Changing that seems to make all the posts disappear and be replaced by numbers, seemingly broken. The thing is, that that line works fine when the page itself loads up, but only causes an error when I try to submit a comment. I'm hoping there's something I can change in the form section to fix this.

johanafm

The [] should be there. However, there is definitely code not shown in this post. For the error message you get, $author must be assigned a string before you get to this place in the code. If you're done using $author for its previous purpose at this point, $author = array(). Otherwise, choose a new name for your variable.

griever

ah man! I thought it was $row that was causing a problem but now I see further down I'm casting $author as a string for the author images. That makes sense that it would only give me that fatal error if it had to loop back through such as when I submit a comment and $author is still left as a string from the first time through

Now when I submit a comment, it reloads the current page but no comment gets submitted. Nothing for approval or anything. Does my INSERT INTO statement look ok?(down at the bottom) I checked the database tables like you said and saw that it takes both comment_author (for users not logged in) and user_id (for users logged in). And that $j loop is so that the form just gets printed once after all the comments have been printed.

<?php get_header ();?>

<div id="box">

<?php

$qry =  "SELECT u.id, user_nicename, post_date, post_content, post_title, post_type, post_excerpt, comment_content, comment_author, comment_approved, comment_count, wp_posts.ID as post_id
         FROM wp_posts
         INNER JOIN wp_users AS u ON post_author = u.id
         LEFT JOIN wp_comments ON wp_posts.ID = comment_post_ID
         WHERE post_type='post'
         ORDER BY user_nicename ASC, post_date DESC";

$res = mysql_query($qry);
$author = array();

if ($res) {
     while ($row = mysql_fetch_assoc($res)) {
          $author[$row['user_nicename']][] = $row;       

     }

 foreach($author as $k => $v) {
     $orderbydate[$v[0]['post_date']][$k] = $v;
 }

 krsort($orderbydate);
 foreach ($orderbydate as $k => $v) {
     ksort($orderbydate[$k], SORT_STRING);
 }
}



$divIds = array('one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine');
$i = 0;

foreach ($orderbydate as $date => $authors) {
     foreach ($authors as $author=> $posts) {   

          echo '<div id="'.$divIds[$i++].'">';

      if($author=="tony") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/tonyname.jpg">';
      }
      elseif($author=="hippie") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hippiename.jpg">';
      }
      elseif($author=="eriku") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/erikuname.jpg">';
      }
      elseif($author=="ben") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/benname.jpg">';
      }
      elseif($author=="hingyi") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hingyiname.jpg">';
      }
      elseif($author=="chris") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/chrisname.jpg">';
      }
      elseif($author=="gabe") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/gabename.jpg">';
      }
      elseif($author=="bender") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/bendername.jpg">';
      }
      elseif($author=="jake") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/jakename.jpg">';
      }

      $lastPost = null;
      foreach($posts as $post) {

           if ($lastPost != $post['post_id']) {                         
                echo '<h2>' . $post['post_title'] . '</h2><small>' . $post['post_date'] . '</small>';
                echo '<p>' . $post['post_content'] . '</p>';
                echo '<p>' . $post['post_excerpt'] . '</p>';                  
                $j=1;
           }             
           $lastPost = $post['post_id'];


           if ($post['comment_count']>=1) {
           echo '<p>' . $post['comment_content'] . '</p>';
           echo '<p><small>-' . $post['comment_author'] . '</small></p>';
           }

           if ($post['comment_count']<=$j) {
                $post_id = $post['post_id'];
                echo '<form method="POST" action='.$_SERVER['PHP_SELF'].' Comment <input type="text" name="comment"/>';

                if ($user_ID) {
                     echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                     echo "  ready to comment as $user_login";
                }

                else echo 'Name <input type="text" name="author"/>';
                echo '<input type="text" name="post_id" style="display: none;" value="'.$post_id.'"/>';
                echo '<p><input name="submit" type="submit" id="submit" value="submit" tabindex="5" /></p>';

                if (empty($_POST['comment_post_id']) || !(($post_id = (int) $_POST['comment_post_id']) > 0)) {
                     # bad comment_post_id;
                }

                $sql=("INSERT INTO wp_comments(comment_author, user_id, comment_content) VALUES ('$_POST[author]','$user_login', '$_POST[comment]')");
                echo '</form>';
           }
           $j++;
      } 
      echo '</div>';
 }
}

get_sidebar ();
get_footer();
?>

griever

shoot sorry this is the latest code I've been messing with. I added the real_escape_string for the three values I was adding to the database and then the INSERT INTO statement with the three keys that correspond to the values, and then the values themselves, and put it into an if statement checking to see if there is a comment_id assigned (if there is a comment), but I don't think it's even going into that statement, so something is wrong with my if statement. I tried removing that if statement and leaving the real escape string lines and the INSERT INTO statement but got the same result.

<?php get_header ();?>

<div id="box">

<?php

$qry =  "SELECT u.id, user_nicename, post_date, post_content, post_title, post_type, post_excerpt, comment_content, comment_author, comment_approved, comment_count, comment_post_ID, comment_ID, wp_posts.ID as post_id
         FROM wp_posts
         INNER JOIN wp_users AS u ON post_author = u.id
         LEFT JOIN wp_comments ON wp_posts.ID = comment_post_ID
         WHERE post_type='post'
         ORDER BY user_nicename ASC, post_date DESC";

$res = mysql_query($qry);
$author = array();

if ($res) {
     while ($row = mysql_fetch_assoc($res)) {
          $author[$row['user_nicename']][] = $row;       

     }

 foreach($author as $k => $v) {
     $orderbydate[$v[0]['post_date']][$k] = $v;
 }

 krsort($orderbydate);
 foreach ($orderbydate as $k => $v) {
     ksort($orderbydate[$k], SORT_STRING);
 }
}



$divIds = array('one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine');
$i = 0;

foreach ($orderbydate as $date => $authors) {
     foreach ($authors as $author=> $posts) {   

          echo '<div id="'.$divIds[$i++].'">';

      if($author=="tony") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/tonyname.jpg">';
      }
      elseif($author=="hippie") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hippiename.jpg">';
      }
      elseif($author=="eriku") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/erikuname.jpg">';
      }
      elseif($author=="ben") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/benname.jpg">';
      }
      elseif($author=="hingyi") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hingyiname.jpg">';
      }
      elseif($author=="chris") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/chrisname.jpg">';
      }
      elseif($author=="gabe") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/gabename.jpg">';
      }
      elseif($author=="bender") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/bendername.jpg">';
      }
      elseif($author=="jake") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/jakename.jpg">';
      }

      $lastPost = null;
      foreach($posts as $post) {

           if ($lastPost != $post['post_id']) {                         
                echo '<h2>' . $post['post_title'] . '</h2><small>' . $post['post_date'] . '</small>';
                echo '<p>' . $post['post_content'] . '</p>';
                echo '<p>' . $post['post_excerpt'] . '</p>';                  
                $j=1;
           }             
           $lastPost = $post['post_id'];

           if ($post['comment_count']>=1) {
           echo '<p>' . $post['comment_content'] . '</p>';
           echo '<p><small>-' . $post['comment_author'] . '</small></p>';
           }

           if ($post['comment_count']<=$j) {
                $post_id = $post['post_id'];
                echo '<form method="POST" action='.$_SERVER['PHP_SELF'].' Comment <input type="text" name="comment"/>';

                if ($user_ID) {
                     echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                     echo "  ready to comment as $user_login";
                }

                else echo 'Name <input type="text" name="author"/>';
                echo '<input type="text" name="post_id" style="display: none;" value="'.$post_id.'"/>';
                echo '<p><input name="submit" type="submit" id="submit" value="submit" tabindex="5" /></p>';

                if (empty($_POST['comment_ID']) || !(($post_id = (int) $_POST['comment_ID']) > 0)) {
                     # bad comment_ID;
                }

                if (isset($_POST['comment_id'])) {
                     $_POST[author]=mysql_real_escape_string($_POST[author]);
                     $user_login=mysql_real_escape_string($user_login);
                     $_POST[comment]=mysql_real_escape_string($POST[comment]);

                     mysql_query("INSERT INTO wp_comments(comment_author, u.id, comment_content) VALUES ('$_POST[author]','$user_login', '$_POST[comment]')");
                }

                echo '</form>';
           }

           $j++;
      } 

      echo '</div>';
 }
}

get_sidebar ();
get_footer();
?>

johanafm

There is no input with name="comment_ID". And since you are performing an INSERT, I doubt you should have one. So checking for isset($_POST|'comment_ID']) will always return false.

<input name="submit" type="submit" id="submit" value="submit" tabindex="5" /></p>';

You could check for any one of the form's elements being set. I usually stick with the submit button, and name it to something appropriate that tells me what's supposed to happen. E.g. I'd name this one submit_comment, rather than just submit.
Then if (isset($_POST['submit'comment'])) ...

griever

ok I changed that the "input name" to "submit_comment" and changed the isset statement to (isset($_POST['submit_comment'])). Based on throwing around some echo statements, it seems like it's going into the statement:

if (empty($POST['comment_ID']) || !(($post_id = (int) $POST['comment_ID']) > 0)) {

bad comment_ID;

}

I changed comment_ID to submit_comment too and it still behaved the same for every post. When I submitted a comment, it still however went through the rest of the code and outputted an echo I placed in the ISSET "if" statement, but no comment appeared. Does this mean the problem lies with the values I have in my INSERT INTO statement?

griever

alright I got some hard data that might help. It seems like when I try to post a comment with the #bad comment_id if statement in there, nothing happens. When I take that "if" statement out and post a comment, nothing gets displayed but 23 comments get added to the database in the wp_comments table (I have 23 posts right now, so I'm assuming it generates one empty comment for each post for some reason). I found this out by looking at the database in phpmyadmin. I made a screenshot of part of the table here. And actually now it inserts the comment_author since I changed a few things. Is my bad comment if statement messed up? Any idea why it creates a comment for every post?

Here is my form section currently. I added a bit in the section for people who are logged in already to set $_POST[author]=$user_login so that i can insert that value at the end:

if ($post['comment_count']<=$j) {
                    $post_id = $post['post_id'];
                    echo '<form method="POST" action='.$_SERVER['PHP_SELF'].' Comment <input type="text" name="comment"/>';

                if ($user_ID) {
                     $_POST[author]=$user_login;
                     echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                     echo "  ready to comment as $user_login";
                }

                else echo 'Name <input type="text" name="author"/>';
                echo '<input type="text" name="post_id" style="display: none;" value="'.$post_id.'"/>';
                echo '<p><input name="submit_comment" type="submit" id="submit" value="submit" tabindex="5" /></p>';

                if (empty($_POST['submit_comment']) || !(($post_id = (int) $_POST['submit_comment']) > 0)) {
                     # bad comment_ID;
                }

                if (isset($_POST['submit_comment'])) {

                     $_POST[author]=mysql_real_escape_string($_POST[author]);
                     $user_login=mysql_real_escape_string($user_login);
                     $_POST[comment]=mysql_real_escape_string($POST[comment]);

                     mysql_query("INSERT INTO wp_comments(comment_author, comment_content) VALUES ('$_POST[author]', '$_POST[comment]')");
                     echo "two";
                }

                echo '</form>';
           }

           $j++;

thanks a lot you guys

johanafm

   echo '<p><input name="submit" type="submit" id="submit" value="submit" tabindex="5" /></p>';

if (empty($_POST['submit_comment']) || !(($post_id = (int) $_POST['submit_comment']) > 0)) {
                         # bad comment_ID;
                    }

Above:
if (empty()): This means the form was NOT sent. The submit comes with post data when form is sent, not otherwise.
if (!(int) > 0): The value of submit is ALWAYS 'submit', which ALWAYS becomes 0.

Conclusion: If the form is not sent, submit is empty. if form is sent, submit is 0. So, wether the form is submitted or not, you always get the comment #bad comment id.
There is NO id (in this regard). You post a NEW comment. It has no id (yet). It will get one when you insert into the db.
Also, the above has NOTHING to do with wether anything is inserted into the db or not (since you get that right a few lines after).

But...

	foreach($posts as $post) {
		mysql_query("INSERT INTO wp_comments(comment_author, u.id, comment_content) VALUES ('$_POST[author]','$user_login', '$_POST[comment]')");

Yes, there will obviously be one insert for every post in the database.

There can only ever be one form submitted to the server in one request. The form does have something to do with a specific post in this case, since you do comment on a specific post. But at the time of the posting, there is NO inherent connection between this post data (as in form data) and whatever post (as in from wp_posts) you are currently outputting.
However, when you do output a specific post and create the form, you do need to somehow deal with creating a connection between this specific post and the data that will be sent along with this specific form. Preferably by somehow retaining the post_id. Which can be done in several ways. In form's action attribute. In submit's value attribute. In value attribute of input which is either of type hidden or somehow not displayed due to CSS.
So, perhaps change the value of your submit buttons to the post_id of whatever post they are show next to (and after this, the empty() || !(int) > 0 will work. But for post_id, not comment_id. You comment a post, not a comment. Well, unless you also allow people to comment comments. But, once again, if (empty()) does NOT mean it's a bad post_id, it simply means that no comment was posted.

The forms should be sent (created) in the loop showing all the posts.
Handling form submission should NOT be done in the loop.
You need to create a relation between a post and a posted comment. There is no post_id in the form.
You obviously also must insert this into the db. There is no post_id in the above sql statement.

griever

Ok before I thought I could tackle 1+2, I thought I should try to make sure that the insert is inserting the comment_author, comment_content, and comment_post_ID to the database. So far the comment_author inserts fine. The comment_post_ID only inserts when I set submit's value="'.$post_id.'" . This is fine except it displays the different post_IDs on the submit buttons. I tried using the line:

echo '<p><input type="hidden" name="comment_post_ID" value="'.$post_id.'" /></p>';

but that didn't insert the comment_post_ID for some reason.

I also can't get the comment_content to insert either. I tried making the syntax the same as for the form for "author" since that is inserting fine, but no luck so far. I remember once I saw the content got inserted but I can't remember what combination made it work! Here's what I have so far for the form section:

if($post['comment_count']<=$j) {
                    $post_id = $post['post_id'];
                    echo '<form method="POST" action="'.$_SERVER['PHP_SELF'].'"/>';
                    echo 'Comment <input type="text" name="comment"/>';

                if ($user_ID) {
                     $_POST[author]=$user_login;
                     echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                     echo "  ready to comment as $user_login";
                }

                else echo 'Name <input type="text" name="author"/>';

                echo '<p><input name="submit_comment" type="submit" id="submit" value="submit" tabindex="5" /></p>';
                echo '<p><input type="hidden" name="comment_post_ID" value="'.$post_id.'" /></p>';

                if (empty($_POST['submit_comment']) || !(($post_id = (int) $_POST['submit_comment']) > 0)) {
                     echo "one";
                     # bad comment_ID;
                }

                if (isset($_POST['submit_comment'])) {

                     $_POST[author]=mysql_real_escape_string($_POST[author]);
                     $user_login=mysql_real_escape_string($user_login);
                     $_POST[comment]=mysql_real_escape_string($POST[comment]);

                     mysql_query("INSERT INTO wp_comments(comment_author, comment_content, comment_post_ID) VALUES ('$_POST[author]', '$_POST[comment]', '$post_id')");
                     echo "stuff got inserted";
                }

                echo '</form>';
           }

           $j++;

griever

Alright I got the comment_post_ID sorted out, but I still can't get the comment_content to insert. If I set the value of the comment_content key in the insert statement to a number instead of $_POST[comment], it inserts that value. I just can't figure out what is stopping the comments insert that isn't stopping the author insert.

For number 1 and 2, I think the form creation is done inside the loop. I'm confused as to how the submission part can be moved outside of the loop. I tried moving around the if statement containing the INSERT INTO statement but I don't think I found the right spot for it because I still get duplicates of the comment in the database(sometimes 32, sometimes 8) Here is my current index.php:

<?php get_header ();?>

<div id="box">

<?php

$qry =  "SELECT u.id, user_nicename, post_date, post_content, post_title, post_type, post_excerpt, comment_content, comment_author, comment_approved, comment_count, comment_post_ID, comment_ID, wp_posts.ID as post_id
         FROM wp_posts
         INNER JOIN wp_users AS u ON post_author = u.id
         LEFT JOIN wp_comments ON wp_posts.ID = comment_post_ID
         WHERE post_type='post'
         ORDER BY user_nicename ASC, post_date DESC";

$res = mysql_query($qry);
$author = array();

if ($res) {
     while ($row = mysql_fetch_assoc($res)) {
          $author[$row['user_nicename']][] = $row;       

     }

 foreach($author as $k => $v) {
     $orderbydate[$v[0]['post_date']][$k] = $v;
 }

 krsort($orderbydate);
 foreach ($orderbydate as $k => $v) {
     ksort($orderbydate[$k], SORT_STRING);
 }
}



$divIds = array('one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine');
$i = 0;

foreach ($orderbydate as $date => $authors) {
     foreach ($authors as $author=> $posts) {   

          echo '<div id="'.$divIds[$i++].'">';

      if($author=="tony") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/tonyname.jpg">';
      }
      elseif($author=="hippie") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hippiename.jpg">';
      }
      elseif($author=="eriku") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/erikuname.jpg">';
      }
      elseif($author=="ben") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/benname.jpg">';
      }
      elseif($author=="hingyi") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hingyiname.jpg">';
      }
      elseif($author=="chris") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/chrisname.jpg">';
      }
      elseif($author=="gabe") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/gabename.jpg">';
      }
      elseif($author=="bender") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/bendername.jpg">';
      }
      elseif($author=="jake") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/jakename.jpg">';
      }

      $lastPost = null;
      foreach($posts as $post) {

           if($lastPost != $post['post_id']) {                         
                echo '<h2>' . $post['post_title'] . '</h2><small>' . $post['post_date'] . '</small>';
                echo '<p>' . $post['post_content'] . '</p>';
                echo '<p>' . $post['post_excerpt'] . '</p>';                  
                $j=1;
           }             
           $lastPost = $post['post_id'];

           if ($post['comment_count']>=1) {
           echo '<p>' . $post['comment_content'] . '</p>';
           echo '<p><small>-' . $post['comment_author'] . '</small></p>';
           }





           if($post['comment_count']<=$j) {
                $post['comment_approved']=0;
                $post_id = $post['post_id'];
                echo '<form method="POST" action="'.$_SERVER['PHP_SELF'].'"/>';
                echo 'Comment <input type="text" name="comment" id="comment"/>'; 

                if ($user_ID) {
                     $_POST[author]=$user_login;
                     $post['comment_approved']=1;
                     echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                     echo "  ready to comment as $user_login";
                }

                else echo 'Name <input type="text" name="author"/>';

                echo '<p><input name="submit_comment" type="submit" id="submit" value="fly like the eagle" tabindex="5" /></p>';
                echo '<p><input type="hidden" name="number_of_the_post" value="'.$post_id.'" /></p>';

                if (empty($_POST['submit_comment']) || !(($post_id = (int) $_POST['submit_comment']) > 0)) {
                     # bad comment_ID;
                }

                if (isset($_POST['submit_comment'])) {

                     $_POST[author]=mysql_real_escape_string($_POST[author]);
                     $user_login=mysql_real_escape_string($user_login);
                     $_POST[comment]=mysql_real_escape_string($POST[comment]);
                     $post_id=mysql_real_escape_string($post_id);
                     mysql_query ("INSERT INTO wp_comments(comment_content, comment_author, comment_post_ID) VALUES ('$_POST[comment]', '$_POST[author]', '$_POST[number_of_the_post]')");
                }

                echo '</form>';
           }

           $j++;
      }

      echo '</div>';          
 }
}

get_sidebar ();
get_footer();
?>

and here is just the form part:

code=php {
$post['comment_approved']=0;
$post_id = $post['post_id'];
echo '<form method="POST" action="'.$_SERVER['PHP_SELF'].'"/>';
echo 'Comment <input type="text" name="comment" id="comment"/>';

                if ($user_ID) {
                     $_POST[author]=$user_login;
                     $post['comment_approved']=1;
                     echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                     echo "  ready to comment as $user_login";
                }

                else echo 'Name <input type="text" name="author"/>';

                echo '<p><input name="submit_comment" type="submit" id="submit" value="fly like the eagle" tabindex="5" /></p>';
                echo '<p><input type="hidden" name="number_of_the_post" value="'.$post_id.'" /></p>';

                if (empty($_POST['submit_comment']) || !(($post_id = (int) $_POST['submit_comment']) > 0)) {
                     # bad comment_ID;
                }

                if (isset($_POST['submit_comment'])) {

                     $_POST[author]=mysql_real_escape_string($_POST[author]);
                     $user_login=mysql_real_escape_string($user_login);
                     $_POST[comment]=mysql_real_escape_string($POST[comment]);
                     $post_id=mysql_real_escape_string($post_id);
                     mysql_query ("INSERT INTO wp_comments(comment_content, comment_author, comment_post_ID) VALUES ('$_POST[comment]', '$_POST[author]', '$_POST[number_of_the_post]')");
                }

                echo '</form>';
           }

           $j++;
      }[/code]

griever

Ok so now the submit is inserting all the info into the database (I had to change "comment" to something else, I guess "comment" is getting used by something else...).

When I submit a comment, the page that displays it exactly the same. Then when I reload the page, one of the following happens:

If I submit a comment for a post which has no existing comments, it outputs the box to type in your comment and the submit button 23 times, once for each post. It does not output the comment.

-or-

If I submit a comment for a post which has at least one existing comment, it outputs everything including the comment 23 times for each existing comment plus 23 times for the comment i submitted.

I understand why it's posting duplicates because of where I have the form and the INSERT statement, I am trying to figure out why it doesn't output the comment when there are no existing comments (when this is the first comment).

I'm still trying to move the INSERT part around to make it only insert once...

griever

It's now just inserting one instance of the comment! Now I'm trying to manually update the comment_count value in the wp_posts table. Do I need a separate INSERT INTO statement if I am dealing with a new table?

What I am trying to do is count the number of rows in the wp_comments table that have a particular comment_post_id (one for each comment that has been posted) and then insert that number as the new value of comment_count in the wp_posts table. Something like this? ($post should be the array)

$i=0;
$post['post_id']=$the_current_post;
foreach($post as $comment_post_id=>$the_current_post) {
i++;
}
$post['comment_count']=$i;

here is my index.php so far:

<?php get_header ();?>

<div id="box">

<?php

$qry =  "SELECT u.id, user_nicename, post_date, post_content, post_title, post_type, post_excerpt, comment_content, comment_author, comment_approved, comment_count, comment_post_ID, comment_ID, wp_posts.ID as post_id
         FROM wp_posts
         INNER JOIN wp_users AS u ON post_author = u.id
         LEFT JOIN wp_comments ON wp_posts.ID = comment_post_ID
         WHERE post_type='post'
         ORDER BY user_nicename ASC, post_date DESC";

$res = mysql_query($qry);
$author = array();

if ($res) {
     while ($row = mysql_fetch_assoc($res)) {
          $author[$row['user_nicename']][] = $row;       

     }

 foreach($author as $k => $v) {
     $orderbydate[$v[0]['post_date']][$k] = $v;
 }

 krsort($orderbydate);
 foreach ($orderbydate as $k => $v) {
     ksort($orderbydate[$k], SORT_STRING);
 }
}



$divIds = array('one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine');
$i = 0;

foreach ($orderbydate as $date => $authors) {
     foreach ($authors as $author=> $posts) {   

          echo '<div id="'.$divIds[$i++].'">';

      if($author=="tony") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/tonyname.jpg">';
      }
      elseif($author=="hippie") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hippiename.jpg">';
      }
      elseif($author=="eriku") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/erikuname.jpg">';
      }
      elseif($author=="ben") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/benname.jpg">';
      }
      elseif($author=="hingyi") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/2009/12/hingyiname.jpg">';
      }
      elseif($author=="chris") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/chrisname.jpg">';
      }
      elseif($author=="gabe") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/gabename.jpg">';
      }
      elseif($author=="bender") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/bendername.jpg">';
      }
      elseif($author=="jake") {
           echo '<img src="http://www.hippievstony.com/wp-content/uploads/jakename.jpg">';
      }

      $lastPost = null;
      foreach($posts as $post) {

           if($lastPost != $post['post_id']) {                         
                echo '<h2>' . $post['post_title'] . '</h2><small>' . $post['post_date'] . '</small>';
                echo '<p>' . $post['post_content'] . '</p>';
                echo '<p>' . $post['post_excerpt'] . '</p>';                                    
                $j=1;
           }             
           $lastPost = $post['post_id'];

           if ($post['comment_count']>=1) {
           echo '<p>' . $post['comment_content'] . '</p>';
           echo '<p><small>-' . $post['comment_author'] . '</small></p>';
           }






         if($post['comment_count']<=$j) {

                $post_id = $post['post_id'];
                echo '<form method="POST" action="'.$_SERVER['PHP_SELF'].'"/>';
                echo 'Comment <input type="text" name="commenty"/>'; 

                if ($user_ID) {
                     $_POST[author]=$user_login;
                     $post['comment_approved']=1;
                     echo '<input type="text" name="author" style="display:none" value="$user_login"/>';
                     echo "  ready to comment as $user_login";
                }

                else {echo 'Name <input type="text" name="author"/>';
                      $post['comment_approved']=0;
                     }

                echo '<p><input name="submit_comment" type="submit" id="submit" value="fly like the eagle" tabindex="5" /></p>';
                echo '<p><input type="hidden" name="number_of_the_post" value="'.$post_id.'" /></p>';

                if (empty($_POST['submit_comment']) || !(($post_id = (int) $_POST['submit_comment']) > 0)) {
                     # bad comment_ID;
                }

                if (isset($_POST['submit_comment'])) {
                $_POST[commenty]=mysql_real_escape_string($_POST[commenty]);
                $_POST[author]=mysql_real_escape_string($_POST[author]);
                $_POST[number_of_the_post]=mysql_real_escape_string($_POST[number_of_the_post]);  

                mysql_query ("INSERT INTO wp_comments(comment_content, comment_author, comment_post_ID) VALUES ('$_POST[commenty]', '$_POST[author]', '$_POST[number_of_the_post]')");
                mysql_query ("INSERT INTO wp_posts(comment_count) VALUES ('$post[comment_count]')");
                }

                $_POST['submit_comment']=NULL;
                echo '</form>';
           }

           $j++;

      }

      echo '</div>';          
 }
}

get_sidebar ();
get_footer();
?>

thanks a lot

johanafm

Yes, you cannot insert into multiple tables at once. But you don't want to insert, you want one insert and one update. The comment is inserted, since the comment does not yet exist. However, the post most exist. Else, what is getting commented? So, the post should be updated. You don't want a new post for each posted comment...

And when you insert the comment you include the id of the post. So updating the relevant post later on is easy.

mysql_query ("INSERT INTO wp_comments(comment_content, comment_author, comment_post_ID) VALUES ('$_POST[commenty]', '$_POST[author]', '$_POST[number_of_the_post]')");
mysql_query ('UPDATE wp_posts SET comment_count = comment_count + 1 WHERE whatever_the_post_id_column_is_called = '.$_POST['number_of_the_post']);

Very important: sanitize user input
Very important: make sure the first query doesn't fail (or the db will end up in a non-consistent state)
Very important: make sure the second query doesn't fail (or the db will end up in a non-consistent state)

Easiest solution for the last two items are to wrap them in a transaction. Rollback on failure, commit on success.

griever

Does it matter that comment_count is in the wp_posts table and comment_post_ID is from the wp_comments table for the update to be successful?

Also, if there are 4 comments, does the UPDATE add one to comment_count for each time it finds a comment_post_ID = $_POST['number_of_the_post'] like a foreach statement or does it just run once? I like the idea that comment_count is tied to the actual number of comments, not just +1 when a comment gets posted because of comments getting deleted etc...

Thanks I'm working on the commit and release stuff now!

griever

I changed the quotes around in the UPDATE line but I still can't get it to update the value.

mysql_query ("INSERT INTO wp_comments(comment_content, comment_author, comment_post_ID) VALUES ('$_POST[commenty]', '$_POST[author]', '$_POST[number_of_the_post]')");
mysql_query ("UPDATE wp_posts SET comment_count=comment_count+1 WHERE comment_post_ID = '$_POST[number_of_the_post]'");

For commit and rollback, would it be something like this?
I was searching the wordpress site for mention of it but no luck.