vb.org Archive

vb.org Archive (https://vborg.vbsupport.ru/index.php)
-   vB3 Programming Discussions (https://vborg.vbsupport.ru/forumdisplay.php?f=15)
-   -   MySQL syntax error for delete (https://vborg.vbsupport.ru/showthread.php?t=142052)

djbaxter 03-14-2007 04:05 PM
MySQL syntax error for delete
 
What is wrong with this query?

Code:
DELETE FROM vbpmtext, vbpm, vbpmreceipt WHERE (vbpm.pmid = vbpmreceipt.pmid) AND (vbpm.pmtextid = vbpmtext.pmtextid) AND vbpmtext.dateline < $time
I am trying to run it from this PHP utility but it also gives me an error run from phpMyAdmin:

PHP Code:
<?php 

require ("global.php"); 
$time = time() - (90 * 24 * 60 * 60); 

 $query = "DELETE FROM vbpmtext, vbpm, vbpmreceipt WHERE (vbpm.pmid = vbpmreceipt.pmid) AND (vbpm.pmtextid = vbpmtext.pmtextid) AND vbpmtext.dateline < $time";
 mysql_query($query);
 print $query; 
?>
This version seems to work:

Quote:
DELETE FROM vbpmtext, vbpm, vbpmreceipt USING vbpmtext, vbpm, vbpmreceipt WHERE (vbpm.pmid = vbpmreceipt.pmid) AND (vbpm.pmtextid = vbpmtext.pmtextid) AND vbpmtext.dateline < $time
in the sense that it doesn't generate a syntax error. Anyone see anything wrong with that?

nexialys 03-14-2007 05:15 PM
You can't delete 3 table elements at the same time... you have to run three queries for that.

harmor19 03-14-2007 09:26 PM
You don't need to put the query into a variable then the variable into a method.
You can just use $db->query_write()
Example:
PHP Code:
$db->query_write("DELETE FROM " . TABLE_PREFIX . "temp_host_request WHERE userid = '".$db->escape_string($vbulletin->GPC['postuserid'])."' "); 

The syntax is "DELETE FROM [table]". You don't list the fields.

Dismounted 03-15-2007 04:39 AM
That depends if he's using a vBulletin backend or not.

harmor19 03-15-2007 05:12 AM
Quote:
Originally Posted by Dismounted (Post 1203888)
That depends if he's using a vBulletin backend or not.
He called "global.php" so I'm assuming he is.

Marco van Herwaarden 03-15-2007 06:45 AM
What is the MySQL version you are using and what is the exact error message you get?

djbaxter 03-15-2007 12:31 PM
Update: I'm not getting error messages using either of the following MySQL statements:

Version 1 (works):

PHP Code:
 $query = "DELETE FROM " . TABLE_PREFIX . "pmtext, " . TABLE_PREFIX . "pm, " . TABLE_PREFIX . "pmreceipt USING " . TABLE_PREFIX . "pmtext, " . TABLE_PREFIX . "pm, " . TABLE_PREFIX . "pmreceipt WHERE (" . TABLE_PREFIX . "pm.pmid = " . TABLE_PREFIX . "pmreceipt.pmid) AND (" . TABLE_PREFIX . "pm.pmtextid = " . TABLE_PREFIX . "pmtext.pmtextid) AND " . TABLE_PREFIX . "pmtext.dateline < $time"; 
Version 2 (works):

PHP Code:
 $query = "DELETE " . TABLE_PREFIX . "pmtext . * , " . TABLE_PREFIX . "pm . * , " . TABLE_PREFIX . "pmreceipt . * FROM " . TABLE_PREFIX . "pmtext, " . TABLE_PREFIX . "pm, " . TABLE_PREFIX . "pmreceipt WHERE " . TABLE_PREFIX . "pm.pmid = " . TABLE_PREFIX . "pmreceipt.pmid AND " . TABLE_PREFIX . "pm.pmtextid = " . TABLE_PREFIX . "pmtext.pmtextid AND " . TABLE_PREFIX . "pmtext.dateline < $time"; 
Is there any advantage of one over the other?

I will release this as an add-on when it's fully tested in case anyone else might find it useful.

I'm using PHP v4.4.4 with MySQL v4.1.21.

Cap'n Steve 03-17-2007 08:34 AM
This is an interesting topic, and I'll have to take a look at your version 1 query since I've never used USING. I've had a lot of trouble doing cross-table deletes recently, and so far I've found that listing the fields (table1.*, table2.*) and avoiding aliases seems to work the best.


All times are GMT. The time now is 02:42 PM.

Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.

X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.01162 seconds
  • Memory Usage 1,744KB
  • Queries Executed 10 (?)
More Information
Template Usage:
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)bbcode_code_printable
  • (4)bbcode_php_printable
  • (2)bbcode_quote_printable
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)post_thanks_navbar_search
  • (1)printthread
  • (8)printthreadbit
  • (1)spacer_close
  • (1)spacer_open 
Phrase Groups Available:
  • global
  • postbit
  • showthread
Included Files:
  • ./printthread.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/class_bbcode_alt.php
  • ./includes/class_bbcode.php
  • ./includes/functions_bigthree.php 
Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • printthread_start
  • bbcode_fetch_tags
  • bbcode_create
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • printthread_post
  • printthread_complete