Delete Row - Refreshes Page but not DB

Discussion:

BenderisGreat

2013-09-25 17:00:18 UTC

The following code I wrote to perform two functions. Delete a row matching
the the row[id] and to also erase the associated attachment post. It
functions - The code shows the delete button in every row, and when pressed
the row is deleted. However after pressing delete the page is refreshed (not
sure why, I dont have any onclick or onsubmit events set) and the row
remains until a manual refresh is performed. Need a little assistance
figuring why;

<?php
if (isset($_POST['id'])) {
$id = $_POST['id'];
if (isset($_POST['deleteItem'])) {
get_delete_post_link( get_the_ID($row['file']));
$qry = "DELETE FROM wp_jo_plugin_options WHERE id = ".$id."";
$jo_remove_row = $wpdb->query($qry);
}
}
?>

Here is the code responsible for showing the table.

<?php
$jo_max_rows = get_option( 'jo_plugin_options');
$result_array = $wpdb->get_results('SELECT * FROM '. $wpdb->prefix .
jo_plugin_options .' WHERE user_id = '. $userid .' ORDER BY date DESC',
ARRAY_A);
$i = 0;
$limit = $jo_max_rows['jo_max_records'];
$count = count($result_array);
while ($i < $limit && $i < $count)
{
$row = $result_array[$i];
$jo_pic_icon = wp_get_attachment_url( $row['file'] );
?>

<?php echo $row['date']; ?>
<?php echo $row['length'] . '' . $selected ?>

<?php echo $row['ground'] . '' . $selected ?>

<?php $del_hide = (isset($table_fields['jo_checkbox_del']));
if (!$del_hide) { ?>

<form method="post" action="" />
<input type="hidden" name="id" value="<?php
echo $row['id']; ?>" />
<button type="submit" name="deleteItem" value="delete"
class=""></button>
<?php } else { }; ?>
</form>

<?php if(!empty($row['file'])) { ?>
<a href="<?php echo $jo_pic_icon; ?>">
<? } else { }; ?>

<?php
++$i; }
?>

--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42392.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.

BenderisGreat

2013-09-25 17:03:55 UTC

Permalink

The code kept rendering here, so I attached it in a text file;

--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.

J.D. Grimes

2013-09-25 19:07:00 UTC

Permalink

Post by BenderisGreat
The following code I wrote to perform two functions. Delete a row matching
the the row[id] and to also erase the associated attachment post. It
functions - The code shows the delete button in every row, and when pressed
the row is deleted. However after pressing delete the page is refreshed (not
sure why, I dont have any onclick or onsubmit events set) and the row
remains until a manual refresh is performed. Need a little assistance
figuring why;

I think perhaps you meant to say that the page is NOT automatically refreshed when you hit the delete button? The expected result from your code would be that the page should refresh. There is no way (without interference from some JavaScript somewhere) that anything is getting deleted without a refresh. Could you clarify that?

J.D.

BenderisGreat

2013-09-25 19:09:29 UTC

Permalink

The page does refresh, yes. But the row remains visible, as if it was not
deleted. I must then manually refresh the page for the row to show as
deleted. Cant figure why that is.

--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42401.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.

J.D. Grimes

2013-09-25 19:15:29 UTC

Permalink

Sounds like the code that deletes the row isn't being executed until after the table is displayed.

J.D.

Post by BenderisGreat
The page does refresh, yes. But the row remains visible, as if it was not
deleted. I must then manually refresh the page for the row to show as
deleted. Cant figure why that is.
--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42401.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.
_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

BenderisGreat

2013-09-25 19:22:41 UTC

Permalink

Right you are [again] J.D.

So should I just start sending you paypal payments for your assistance?
One more followup question - I don't know any Ajax- is there a simple way
(now that the form is fixed) to remove that row without a page refresh?

Again, not familiar with ajax so coding that myself isn't an option yet, but
maybe there is a simple way to implement something like that?

--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42403.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.

J.D. Grimes

2013-09-25 19:50:15 UTC

Permalink

It isn't hard to do, I've done it before. Do you know jQuery? jQuery's AJAX handlers are very easy to user (in my opinion, at least). Quick and probably somewhat dirty example:

jQuery( document ).ready( function( $ ) {

// When a button is clicked within the table with id="mytable"
$( '#mytable button' ).click( function( event ) {

// Keep the default action from happening (form submitted/page refreshed)
event.preventDefault();

if ( ! confirm( 'Are you sure you want to continue' ) )
return;

// Get the id of the row to delete.
// I'd recommend just making the button value the row ID, no need for the extra input.
// Then you can access it easier through $( this ).val();
var rowID = $( this ).parent().child( 'input' )[0].val();

// POST data.
var data = {
'action': 'my_delete_row_action',
'rowID': rowID
}

// Send AJAX request to delete this.
$.post(
ajaxurl,
data,
function ( response ) {

// Check if this was successful.
// Assumes you are using http://codex.wordpress.org/Function_Reference/wp_send_json_success
if ( ! response.success ) {

alert( 'Failure!' );
}

// Success! Hide the row.
$( '#mytable tr#myrow-' + rowID ).hide();
}
);
});
});

OK, that probably won't work off the bat, just an example. You'll need to give your table an HTML ID, and give each row a unique ID based on $i (in the example it's myrow-$i.

Also you'll need to see http://codex.wordpress.org/AJAX_in_Plugins for info on how to handle the AJAX request properly within WordPress.

I'm sure you can find some great tutorials out there to help you along, too.

J.D.

Post by BenderisGreat
Right you are [again] J.D.
So should I just start sending you paypal payments for your assistance?
One more followup question - I don't know any Ajax- is there a simple way
(now that the form is fixed) to remove that row without a page refresh?
Again, not familiar with ajax so coding that myself isn't an option yet, but
maybe there is a simple way to implement something like that?
--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42403.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.
_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

BenderisGreat

2013-10-19 07:21:19 UTC

Permalink

*I was having a hard time working out how to implement the ajax into
wordpress with the JSON callback. So, I hired a guy from freelancer. His
job was to setup my form as ajax, but im looking at his code now and it does
not make use the same code you used.

What he did was echo the row-ID in each
(is this safe?) and then created this button:*

<div id="delete<?php echo $row['id']; ?>">
<input type="submit" name="deleteItem"
value="delete" class="delete-box"
onclick="delete1(<?php echo $row['id']; ?>) ">delete
</div>

*and this <div>, which shows inside the
I click delete on:*

<div style="display:none" id="areusure<?php echo
$row['id']; ?>"></div>

*Then, in the head of the file this is the script that handles the selection
of the
etc...*

*and finally, here is the ajax call (which isnt written how you suggested
using ajaxurl in wordpress)*

<?php
if ( !isset($wp_did_header) ) {
$wp_did_header = true;
require_once('../../../wp-load.php' );
wp();
require_once( ABSPATH . WPINC . '/template-loader.php' );

}
global $wpdb;
if (isset($_REQUEST['id'])) {
$id = $_REQUEST['id'];
$file = $wpdb->get_var("select file from wp_jo_plugin_options WHERE id =
$id");
//wp_delete_attachment( $file );
$qry2 = $wpdb->prepare("CREATE TABLE IF NOT EXISTS t2 AS SELECT * from
wp_jo_plugin_options WHERE id = %s", $id);
$jo1 = $wpdb->query($qry2);
$qry = $wpdb->prepare("DELETE FROM wp_jo_plugin_options WHERE id = %s",
$id);
$jo_remove_row = $wpdb->query($qry);
}
if (isset($_REQUEST['undo'])) {
$undo = $_REQUEST['undo'];
$qry2 = $wpdb->prepare("insert ignore into wp_jo_plugin_options SELECT *
from t2 WHERE id = %s", $undo);
$jo_remove_row = $wpdb->query($qry2);
}
?>

*Now - I have no idea is this is bad code, but it does not match the same
layout you suggested. Is this safe? It cost me about 100 bucks on
freelancer.com. :-/*

Post by J.D. Grimes
It isn't hard to do, I've done it before. Do you know jQuery? jQuery's
AJAX handlers are very easy to user (in my opinion, at least). Quick and
jQuery( document ).ready( function( $ ) {
// When a button is clicked within the table with id="mytable"
$( '#mytable button' ).click( function( event ) {
// Keep the default action from happening (form submitted/page refreshed)
event.preventDefault();
if ( ! confirm( 'Are you sure you want to continue' ) )
return;
// Get the id of the row to delete.
// I'd recommend just making the button value the row ID, no need for the extra input.
// Then you can access it easier through $( this ).val();
var rowID = $( this ).parent().child( 'input' )[0].val();
// POST data.
var data = {
'action': 'my_delete_row_action',
'rowID': rowID
}
// Send AJAX request to delete this.
$.post(
ajaxurl,
data,
function ( response ) {
// Check if this was successful.
// Assumes you are using
http://codex.wordpress.org/Function_Reference/wp_send_json_success
if ( ! response.success ) {
alert( 'Failure!' );
}
// Success! Hide the row.
$( '#mytable tr#myrow-' + rowID ).hide();
}
);
});
});
OK, that probably won't work off the bat, just an example. You'll need to
give your table an HTML ID, and give each row a unique ID based on $i (in
the example it's myrow-$i.
Also you'll need to see http://codex.wordpress.org/AJAX_in_Plugins for
info on how to handle the AJAX request properly within WordPress.
I'm sure you can find some great tutorials out there to help you along, too.
J.D.
On Sep 25, 2013, at 3:22 PM, BenderisGreat <

Post by BenderisGreat
Right you are [again] J.D.
So should I just start sending you paypal payments for your assistance?
One more followup question - I don't know any Ajax- is there a simple way
(now that the form is fixed) to remove that row without a page refresh?
Again, not familiar with ajax so coding that myself isn't an option yet, but
maybe there is a simple way to implement something like that?
--
http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42403.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.
_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers
It isn't hard to do, I've done it before. Do you know jQuery? jQuery's
AJAX handlers are very easy to user (in my opinion, at least). Quick and
jQuery( document ).ready( function( $ ) {
// When a button is clicked within the table with id="mytable"
$( '#mytable button' ).click( function( event ) {
// Keep the default action from happening (form submitted/page refreshed)
event.preventDefault();
if ( ! confirm( 'Are you sure you want to continue' ) )
return;
// Get the id of the row to delete.
// I'd recommend just making the button value the row ID, no need for the extra input.
// Then you can access it easier through $( this ).val();
var rowID = $( this ).parent().child( 'input' )[0].val();
// POST data.
var data = {
'action': 'my_delete_row_action',
'rowID': rowID
}
// Send AJAX request to delete this.
$.post(
ajaxurl,
data,
function ( response ) {
// Check if this was successful.
// Assumes you are using
http://codex.wordpress.org/Function_Reference/wp_send_json_success
if ( ! response.success ) {
alert( 'Failure!' );
}
// Success! Hide the row.
$( '#mytable tr#myrow-' + rowID ).hide();
}
);
});
});
OK, that probably won't work off the bat, just an example. You'll need to
give your table an HTML ID, and give each row a unique ID based on $i (in
the example it's myrow-$i.
Also you'll need to see http://codex.wordpress.org/AJAX_in_Plugins for
info on how to handle the AJAX request properly within WordPress.
I'm sure you can find some great tutorials out there to help you along, too.
J.D.
On Sep 25, 2013, at 3:22 PM, BenderisGreat <

Post by BenderisGreat
Right you are [again] J.D.
So should I just start sending you paypal payments for your assistance?
One more followup question - I don't know any Ajax- is there a simple way
(now that the form is fixed) to remove that row without a page refresh?
Again, not familiar with ajax so coding that myself isn't an option yet, but
maybe there is a simple way to implement something like that?
--
http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42403.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.
_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42560.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.

Otto

2013-10-19 14:08:11 UTC

Permalink

On Sat, Oct 19, 2013 at 2:21 AM, BenderisGreat

Post by BenderisGreat
*and finally, here is the ajax call (which isnt written how you suggested
using ajaxurl in wordpress)*
<?php
if ( !isset($wp_did_header) ) {
$wp_did_header = true;
require_once('../../../wp-load.php' );
wp();
require_once( ABSPATH . WPINC . '/template-loader.php' );
}
global $wpdb;
if (isset($_REQUEST['id'])) {
$id = $_REQUEST['id'];
$file = $wpdb->get_var("select file from wp_jo_plugin_options WHERE id =
$id");
//wp_delete_attachment( $file );
$qry2 = $wpdb->prepare("CREATE TABLE IF NOT EXISTS t2 AS SELECT * from
wp_jo_plugin_options WHERE id = %s", $id);
$jo1 = $wpdb->query($qry2);
$qry = $wpdb->prepare("DELETE FROM wp_jo_plugin_options WHERE id = %s",
$id);
$jo_remove_row = $wpdb->query($qry);
}
if (isset($_REQUEST['undo'])) {
$undo = $_REQUEST['undo'];
$qry2 = $wpdb->prepare("insert ignore into wp_jo_plugin_options SELECT *
from t2 WHERE id = %s", $undo);
$jo_remove_row = $wpdb->query($qry2);
}
?>
*Now - I have no idea is this is bad code, but it does not match the same
layout you suggested. Is this safe? It cost me about 100 bucks on
freelancer.com. :-/*

No, that code is not safe. This code at the start, for example, is an
SQL Injection vulnerability:

if (isset($_REQUEST['id'])) {
$id = $_REQUEST['id'];
$file = $wpdb->get_var("select file from wp_jo_plugin_options
WHERE id = $id");

Because he's not prepare'd or otherwise sanitizing the id parameter,
it's easy to inject something there.

In general, anytime you include wp-load.php directly, you're probably
doing-it-wrong. The ajaxurl callback method suggested earlier is
better all around.

-Otto

Nikola Nikolov

2013-10-19 16:26:01 UTC

Permalink

PS: I would do a dispute on Freelancer.com(if that's where you hired your
guy) - that's clearly not a good quality code. And as Otto pointed out it's
a potential security hole to your site(or what could be even worse -
plugin). You might be able to get your money back, or get the job done
better.

Post by Otto
On Sat, Oct 19, 2013 at 2:21 AM, BenderisGreat

WHERE id =

Post by BenderisGreat
$id");
//wp_delete_attachment( $file );
$qry2 = $wpdb->prepare("CREATE TABLE IF NOT EXISTS t2 AS SELECT

* from

Post by BenderisGreat
wp_jo_plugin_options WHERE id = %s", $id);
$jo1 = $wpdb->query($qry2);
$qry = $wpdb->prepare("DELETE FROM wp_jo_plugin_options WHERE id

= %s",

Post by BenderisGreat
$id);
$jo_remove_row = $wpdb->query($qry);
}
if (isset($_REQUEST['undo'])) {
$undo = $_REQUEST['undo'];
$qry2 = $wpdb->prepare("insert ignore into wp_jo_plugin_options

SELECT *

Post by BenderisGreat
from t2 WHERE id = %s", $undo);
$jo_remove_row = $wpdb->query($qry2);
}
?>
*Now - I have no idea is this is bad code, but it does not match the same
layout you suggested. Is this safe? It cost me about 100 bucks on
freelancer.com. :-/*

No, that code is not safe. This code at the start, for example, is an
if (isset($_REQUEST['id'])) {
$id = $_REQUEST['id'];
$file = $wpdb->get_var("select file from wp_jo_plugin_options
WHERE id = $id");
Because he's not prepare'd or otherwise sanitizing the id parameter,
it's easy to inject something there.
In general, anytime you include wp-load.php directly, you're probably
doing-it-wrong. The ajaxurl callback method suggested earlier is
better all around.
-Otto
_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

BenderisGreat

2013-10-19 17:25:25 UTC

Permalink

I thought it looked weird. I spent some time last night working out the
correct code pasted here, and got it functioning. I dont think I can get my
money back but I will try. I am so glad I have this resource to check with,
or I would have never had JD showing me the proper way, and then never would
have questioned his code.

I finally got JDs code working but I have to place this in my functions for
ajax to work:

add_action('wp_head','custom_head');

function custom_head()
{
echo '';
}

Is that alright? Otherwise I get ajaxurl undefined error.

Additionally, the json response works too! However one query always fails.

$file = $_POST['file'];
$del_image = wp_delete_attachment( $row['file'] ); - fail

I also tried this as an sql query:

$file = $_POST['file'];
$fileQry = $wpdb->prepare("DELETE FROM wp_posts WHERE id = %s", $file);
$go_fileQry = $wpdb->query($fileQry);

It still fails. It IS possible to run multiple queries this way right?

--
View this message in context: http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42566.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.

Nikola Nikolov

2013-10-19 17:35:46 UTC

Permalink

About the ajaxurl - yes that is correct. WordPress only defines the global
JS variable ajaxurl on the admin side - so you have to do it yourself in
order to access it on the front-end.

Why are you not using the $file variable in your other code?

// You are assigning $_POST['file'] to the $file variable and not to
$row['file']
$file = $_POST['file'];
$del_image = wp_delete_attachment( $file );

Post by BenderisGreat
I thought it looked weird. I spent some time last night working out the
correct code pasted here, and got it functioning. I dont think I can get my
money back but I will try. I am so glad I have this resource to check with,
or I would have never had JD showing me the proper way, and then never would
have questioned his code.
I finally got JDs code working but I have to place this in my functions for
add_action('wp_head','custom_head');
function custom_head()
{
echo '';
}
Is that alright? Otherwise I get ajaxurl undefined error.
Additionally, the json response works too! However one query always fails.
$file = $_POST['file'];
$del_image = wp_delete_attachment( $row['file'] ); - fail
$file = $_POST['file'];
$fileQry = $wpdb->prepare("DELETE FROM wp_posts WHERE id = %s", $file);
$go_fileQry = $wpdb->query($fileQry);
It still fails. It IS possible to run multiple queries this way right?
--
http://wordpress-hackers.1065353.n5.nabble.com/Delete-Row-Refreshes-Page-but-not-DB-tp42393p42566.html
Sent from the Wordpress Hackers mailing list archive at Nabble.com.
_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

J.D. Grimes

2013-10-19 17:40:56 UTC

Permalink

Hey man, that's tough. I'm glad you've got the code working though.

Post by BenderisGreat
I finally got JDs code working but I have to place this in my functions for
add_action('wp_head','custom_head');
function custom_head()
{
echo '';
}
Is that alright? Otherwise I get ajaxurl undefined error.

I'm not sure why this would have an effect… If you are using the code on the front end of your site, ajaxurl won't be defined - it is only defined by default in the administration panels. You would need to use wp_localize_script().

Post by BenderisGreat
Additionally, the json response works too! However one query always fails.
$file = $_POST['file'];
$del_image = wp_delete_attachment( $row['file'] ); - fail
$file = $_POST['file'];
$fileQry = $wpdb->prepare("DELETE FROM wp_posts WHERE id = %s", $file);
$go_fileQry = $wpdb->query($fileQry);
It still fails. It IS possible to run multiple queries this way right?

Gregory Lancaster

2013-10-19 17:53:23 UTC

Permalink

Yep, I had to add true. So all this is executing based on the values of
this button:
<button type="submit" name="deleteItem" value="<?php echo $row['id']; ?>"
file="<?php echo $row['file']; ?>" class="delete-box"><span
class="glyphicon glyphicon-remove"></span></button>

But couldnt someone just change the values of the fileid or rowid and
delete other peoples content?

Post by J.D. Grimes
Hey man, that's tough. I'm glad you've got the code working though.

Post by BenderisGreat
I finally got JDs code working but I have to place this in my functions

for

Post by BenderisGreat
add_action('wp_head','custom_head');
function custom_head()
{
echo '';
}
Is that alright? Otherwise I get ajaxurl undefined error.

I'm not sure why this would have an effect… If you are using the code on
the front end of your site, ajaxurl won't be defined - it is only defined
by default in the administration panels. You would need to use
wp_localize_script().

Post by BenderisGreat
Additionally, the json response works too! However one query always

fails.

Post by BenderisGreat
$file = $_POST['file'];
$del_image = wp_delete_attachment( $row['file'] ); - fail
$file = $_POST['file'];
$fileQry = $wpdb->prepare("DELETE FROM wp_posts WHERE id = %s", $file);
$go_fileQry = $wpdb->query($fileQry);
It still fails. It IS possible to run multiple queries this way right?

Is it completely failing? The wp_delete_attachment() call will only move
the attachment to the trash by default -if you want to delete it
permanently you need to set the second parameter to true.
-J.D.
_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

J.D. Grimes

2013-10-19 17:56:06 UTC

Permalink

Yes, you should be using nonces - http://codex.wordpress.org/WordPress_Nonces

Post by Gregory Lancaster
Yep, I had to add true. So all this is executing based on the values of
<button type="submit" name="deleteItem" value="<?php echo $row['id']; ?>"
file="<?php echo $row['file']; ?>" class="delete-box"><span
class="glyphicon glyphicon-remove"></span></button>
But couldnt someone just change the values of the fileid or rowid and
delete other peoples content?

Post by J.D. Grimes
Hey man, that's tough. I'm glad you've got the code working though.

Post by BenderisGreat
I finally got JDs code working but I have to place this in my functions

for

Post by BenderisGreat
add_action('wp_head','custom_head');
function custom_head()
{
echo '';
}
Is that alright? Otherwise I get ajaxurl undefined error.

I'm not sure why this would have an effect… If you are using the code on
the front end of your site, ajaxurl won't be defined - it is only defined
by default in the administration panels. You would need to use
wp_localize_script().

Post by BenderisGreat
Additionally, the json response works too! However one query always

fails.

_______________________________________________
wp-hackers mailing list
http://lists.automattic.com/mailman/listinfo/wp-hackers

Gregory Lancaster

2013-10-19 18:11:52 UTC

Permalink

that would be handled via jquery/ajax though right? the button is
triggered that way.

Post by J.D. Grimes
Yes, you should be using nonces -
http://codex.wordpress.org/WordPress_Nonces

Post by J.D. Grimes
Hey man, that's tough. I'm glad you've got the code working though.

Post by BenderisGreat
I finally got JDs code working but I have to place this in my functions

for

Post by BenderisGreat
add_action('wp_head','custom_head');
function custom_head()
{
echo '';
}
Is that alright? Otherwise I get ajaxurl undefined error.

I'm not sure why this would have an effect… If you are using the code on
the front end of your site, ajaxurl won't be defined - it is only

defined