Hi Poj, I added it to the code but its now resulting in error:

# If the form was properly submitted, save the data
if ($input{"go"} eq "DELETE") {

<form method="post" onsubmit="return confirm('Do you really want to de
+lete?');">

$query = "DELETE
          FROM 
              \"Employees\" 
          WHERE 
              \"EmployeeID\"="." \'$input{empid}\'";

$db_handle -> do($query);
$action = "Record saved - $query";
....
[download]

Error :

Scalar found where operator expected at delemp.pl line 73, near "$quer
+y"
    (Missing semicolon on previous line?)
syntax error at delemp.pl line 73, near "$query "
[download]

If I comment out your code script gets executed, did I put it on a wrong part of the script ? Rgds Terry

As poj said, this:

<form method="post" onsubmit="return confirm('Do you really want to de
+lete?');">
[download]

isn't Perl. You'd want to put this into your HTML, replacing this line:

<form method=POST>
[download]

Also see window.confirm on the MDN.

Another option is to add a step with another form and hidden fields. Refactored it would look something like this.

#!/usr/local/bin/perl
use strict;
use CGI;
use CGI::Carp 'fatalsToBrowser'; #remove for prod
use DBI;

# get form parameters
my $q = new CGI;
my $action = $q->param('go');
my $empid  = $q->param('empid');

my $dbh = dbh(); # connect to db
$dbh->do("SET search_path to northwind") or die;

# If the confirm form was properly submitted, delete the record
my $msg;
# change validation to suit
if ( ($action eq "Confirm Delete") && ($empid =~ /\d+/)) {
  my $sql = qq! DELETE FROM "Employees" 
                WHERE "EmployeeID" = ? !;
               
  my $count = $dbh->do( $sql,undef,$empid );
  $msg = "$count Record deleted - $sql, $empid";
} else {
  $msg = "Please complete form";
}

# get employees
my $sql = qq!SELECT "EmployeeID" AS empid, 
             "FirstName"::text || ' ' ||"LastName"::text AS name 
             FROM "Employees" !;
my $ar = $dbh->selectall_arrayref($sql);

# Make up a pulldown menu 
my $options = qq!<option value="">select name</option>!;
for my $row (@$ar) {
  $options .= qq!<option value="$row->[0]">$row->[1]</option>\n!;
}

# build html page
my $style = q!
  body { background-color: pink ; color: #3300cc; }
  .container { width: 500px; clear: both; }
  .container input { width: 100%; clear: both;}
!;

# Send out the header and form
print $q->header;
print $q->start_html(-title=>'Delete an employee record',
                     -style=>{ -code=>$style } );
print qq!<h1 style="color:3300CC">Delete an employee</h1>!;

 # confirm delete form
if ( $action eq "DELETE" ) {
  print qq!<h3>Are you sure you want to delete $empid ?</h3>
      <form action="" method="post">
        <input type="hidden" name="empid" value="$empid"/>
        <input type="submit" name="go" value="Confirm Delete"/>
        <input type="submit" name="go" value="Cancel"/>
      </form>!;
} else {
  print qq!<div class="container">
  Select Employee name to delete :
<form method="post" action="">
  <select name="empid"> $options </select><br/>
  <input type="submit" name="go" value="DELETE"/>
</form></div><hr/>!;
 
  # Standard links to the rest of the application
  print <<"FOOTER";
<b>$msg</b>
<hr/>
Jump to - <a href="emp2.pl">View Employees Listing</a><br/>
Jump to - <a href="addemp.pl">Add an Employee</a><br/>
Jump to - <a href="updatephoto.pl">Add or update Employee Photo</a><br
+/>
<hr/>
Edited by Terry on July, 06 2014.
FOOTER
}
print $q->end_html;

# connect to database
sub dbh {
  my $dsn = 'DBI:Pg:dbname=northwind;host=localhost';
  my $user = '***';
  my $pwd  = '***';
  my $dbh = DBI -> connect($dsn,$user,$pwd,{'RaiseError' => 1});
  return $dbh;
}
[download]

poj

G'day Terry,

Take a look at "CGI: CREATING A JAVASCRIPT ACTION BUTTON" and ""alert" or prompt box from perl cgi script".

Hi, I added the following code to the script and its not working can you please help me correct it.

print header;
print <<EOF;
<script type="text/javascript">
function confirmOk() {
    return window.confirm("Confirm deletion?");
}
</script>
EOF
[download]

And

# If the form was properly submitted, save the data
if ($input{"go"} eq "DELETE") {

print button(-name=>'Delete',
                          -value=>'DO you really want to delete it ?',
                          -onClick=>"return confirmOk()");

$query = "DELETE ....
[download]

Rgds Terry

See these mostly non running examples Re^5: I need help with displaying newline or paragraph using perl on my website (pass arguments more subs) (esp part a part 5)
Re^9: FormBuilder fails in "Multi screen mode"
Re: reg access of stored session variable
Re^3: Trying to get an RSS to display
Re^5: Cookie and Session

Especially see notes about reading CGI docs/examples and never using that # Read information from the form stuff, thats what CGI modules does for you (CGI reads form data for you)

Then your program basically comes down to

Main( @ARGV );
exit( 0 );
sub Main {
    my $q = CGI->new; ## read form data
    print fancyDatabaseStuff( $q );
}
sub fancyDatabaseStuff {
    my( $q ) = @_;
    my $action = $q->param('action');
    if( !defined $action or !length $action ){
        return ShowEmployees( $q );
    }
    elsif( $action eq 'confirmDelete' ){
        return ConfirmDeleteEmployees( $q );
    }
    elsif( $action eq 'delete' ){
        return DeleteEmployees( $q );
    }
    else {
        return UnrecognizedAction( $q );
    }
}

sub UnrecognizedAction {
    my( $q ) = @_;
    my $header = $q->header;
    return qq{$header
<!DOCTYPE html>
<html>
<title>Ooops</title>
<p>Ooops, don't know what to do, I didn't understand that
};
}
[download]