[Static]

[Steven]

I'm getting these errors now when previewing:

Warning: Missing argument 12 for generate_comment(), called in C:\wamp\www\commentics_1.1 - Copy\comments\includes\app\processor.php on line 375 and defined in C:\wamp\www\commentics_1.1 - Copy\comments\includes\functions\comments.php on line 26

Notice: Undefined variable: reply_to in C:\wamp\www\commentics_1.1 - Copy\comments\includes\functions\comments.php on line 31

Notice: Undefined variable: reply_to in C:\wamp\www\commentics_1.1 - Copy\comments\includes\functions\comments.php on line 171

[Static]

For which change?

[Steven]

After the last two changes in post 41.

[Static]

Did you add the lines? you were supposed to replace the lines?

[Steven]

Yeah I think so. I'll let you finish all your changes/testing and then if you could zip all the files together and I'll take another look.

[Steven]

I'm a bit concerned about the combo box in edit_comment.php

It seems to list every comment for the page. If the page has thousands of comments then the combo box could easily crash the browser. I've seen this problem first-hand.

Is it important to have the ability to alter which comment the reply is for?

[Static]

You're right, but I don't know what to do about it? Any suggestions? Maybe a simple text field where you can insert the id, and when you do, a small preview appears to the side of it?

[Steven]

Perhaps you could limit the results e.g. the last 50. If the admin is altering it then it's very likely to be a recent comment.

[Steven]

Like this:

PHP Code:

<?php 

$comments_reply = mysql_query("SELECT * FROM `".$mysql_table_prefix."comments` WHERE is_approved = '1' AND page_id = '$page_id' AND reply_to = '0' ORDER BY dated ASC LIMIT 50");