Question

I seem to be missing something quite fundamental here and yet my code doesn’t seem to be any different to any of the numerous online tutorials that I have looked at.
What I would like is for someone to look at this and say….Oh you have forgotten to…etc;

This is what I have on a separate update page which is intended to perform the update then cycle back to the main admin page:

require_once('../Connections/MyConn.php');
$sql_statement = "UPDATE skyscrapers SET ";
$sql_image = "Ad_image = '" . $_REQUEST['image'] . "', ";
$sql_expire = "Ad_Expires = '" . $_REQUEST['expire'] . "'";

$result = mysql_query($sql_statement . $sql_image . $sql_expire . " WHERE Ad_ID=" . $_REQUEST['ADID']);
if (!$result) {
  echo("<p>Error performing query: " . mysql_error() . "</p>");
  exit();
} 

mysql_close ($MyConn); 
header("location:Admin_skyscrapers.php");

However when I run this I get the following error:-

“Error performing query: No database selected”

Well, haven’t I selected the database in the connection script which already works everywhere else?

I realise the code isn’t very pretty and I am being naughty using the url to pass variables at the moment – I do promise to change this when I get it to work 🙂

So, any pointers would be helpful, thanks in advance.

Edit to add…

This is the connection script with the sensitive stuff redacted:-

# FileName="Connection_php_mysql.htm"
# Type="MYSQL"
# HTTP="true"
$hostname_MyConn = "*************.co.uk";
$database_MyConn = "db**********";
$username_MyConn = "dbo*********";
$password_MyConn = "*****";
$MyConn = mysql_pconnect($hostname_MyConn, $username_MyConn, $password_MyConn) or trigger_error(mysql_error(),E_USER_ERROR); 

Answer 1

This may or may not be declared in your MyConn.php but all you need is a line:

mysql_select_db($db_name);

Where $db_name is the name of your database.

This should come before you attempt to execute the query.