on click

[Pouncer]

i got a weird problem

<SCRIPT LANGUAGE="JavaScript">

function processAnn () {
	answer = confirm('Confirm ' + form1.textarea.value)
	
	if (answer == 1) {
		<?
		$query = "UPDATE `announcement` SET `Announcement` = 'hi'";
		$result = mysql_query($query) or die("Query error<br/>" . mysql_error());
		
		?> 
	}
}

</SCRIPT>

<input name="Save" type="submit" class="style11" id="Save" value="Save announcement" onclick="processAnn()" />

now evertime i refresh the page, it ust writes 'hi' to the database :S

so it keeps on doing this

$query = "UPDATE `announcement` SET `Announcement` = 'hi'";

but it should only do that when i click 'Ok' in the confirm msgbox

[visualAd]

i got a weird problem

<SCRIPT LANGUAGE="JavaScript">

function processAnn () {
	answer = confirm('Confirm ' + form1.textarea.value)
	
	if (answer == 1) {
		<?
		$query = "UPDATE `announcement` SET `Announcement` = 'hi'";
		$result = mysql_query($query) or die("Query error<br/>" . mysql_error());
		
		?> 
	}
}

</SCRIPT>

<input name="Save" type="submit" class="style11" id="Save" value="Save announcement" onclick="processAnn()" />

now evertime i refresh the page, it ust writes 'hi' to the database :S

so it keeps on doing this

$query = "UPDATE `announcement` SET `Announcement` = 'hi'";

but it should only do that when i click 'Ok' in the confirm msgbox

Ajax isn't that easy.

[Pouncer]

ok, but i dont see why it cant work properly, the page is a .php page anyway

cant i have php inside javascript?

how can i go about doing this properly then?

[visualAd]

ok, but i dont see why it cant work properly, the page is a .php page anyway

cant i have php inside javascript?

how can i go about doing this properly then?

PHP is executed before the pages is sent to the browser. The HTML source code is the output of your PHP script. If you want PHP to execute on a Javascript event you'll either need to use Ajax (which as you are a beginner I do not recommend) or use a link:

<a href="page.php?action=updateannounce&set=hi">Update Announcement</a>

If your page you'll need to have something like this:

if (isset($_GET['action'])){
    switch ($_GET['action']) {
        case 'updateannounce':
            update_announcement(@$_GET['set']);
            break;
    }
}

function update_announcement($text)
{
    $text=mysql_escape_string($text);

    $query = "UPDATE `announcement` SET `Announcement` = '$text'";
    $result = mysql_query($query) or die("Query error<br/>" . mysql_error());
}