page reloading ??

[g00fy_m]

hey to all,

i have an app that reads entries(emails) from a txt file and displays them in a textarea.

I have the page setup so i can add to and delete from the txt file, however when i do this i would like the page to just reload and display the updated textarea.

i have experimented with php_self but i get the `ol 'headers already sent' error and i cant see a way round it.

can anyone pls offer any hlp

thanx in advance

the functions below are contained within the file subskribe.php

/**
* Routine to echo the database file to the screen, used in admin section
* to diplay subskribers to a textarea.
*/
function displayUsers(){

  global $file;

  if(file_exists("$file") && is_readable("$file")){

$contents = file("$file");

sort($contents);

for ($i = 0; $i < count($contents); $i++){

  $str = $contents[$i].strstr("$contents[$i]", "<br>");
  print($str);
}
  }
}

/**
* Function to generate the administration page for the database file
*/
function showAdmin(){

  include('header.inc.php');

  print ('
  <TABLE align="center" width="50%" />
    <TR>
      <td align="center" /> 
        <form action="subskribe.php" method="POST">');         

  print('
          <TEXTAREA style="WIDTH: 337px; HEIGHT: 159px" rows=8 cols=35>
');
  // output all subskribers in the database to the textarea
  displayUsers();
  print ('
          </TEXTAREA>
          <BR><Br>
        ');
  print (entries() . ' subskribers in database.
          <BR> ' .
  $file .
  '  <br><br>
           <INPUT style="WIDTH: 192px; HEIGHT: 22px" size=23 name="subskriber" id="subskriber">
           &nbsp;&nbsp;
           <INPUT type=submit value="Add" name="add" id="add" onSubmit="">   

           &nbsp;&nbsp;
           <INPUT type=submit value="Remove" name="delete" id="delete">
         </form>
       </TD>
     </TR>
   </TABLE>
  ');

  include('footer.inc.php');
}

// ------------- check for delete action -------------- //
if (strtolower($_POST['delete']) === 'remove'){

   removeFromList($entry);

  }

/**
* Routine to remove an email entry from the database file
*/
function removeFromList($email){

  global $file;

  $found = false;
  $names = file("$file");

  for($i = 0; $i < count($names); $i++){

if($names[$i] == $email . "\n"){

  // remove from array
  unset($names[$i]);

  $fp = fopen("$file", "w");

  // write back out to disk
  fwrite($fp, implode("", $names));
  fclose($fp);
  $found = true;
  return 1;
  break;

}

  }

  if(!$found){
    return 0;
  }
}

kind regards

g00fy

[mogster]

If this is the actual script, just move the

// ------------- check for delete action -------------- // 
if (strtolower($_POST['delete']) === 'remove'){ 

   removeFromList($entry); 

  }

to the top of the script. And the function removeFromList($entry) must be declared before that.

Should help.

knutm :-)

[g00fy_m]

yeah sorry,

the script i posted was out of order, i have the $_POST at the top, the trouble is that once a user is added or deleted you are redirected to another page that reports whether or not the user action was successfull or error message,

but when you return to the page with javascript:history.go(-1) it doesnt refresh the textarea, so i was thinking of a way to (in admin section only) display the add / remove messages on the same page and use header('Location:'.$PHP_SELF); to reload the page but i always get the headers already sent error or is there another way in php ??

i could always have a javascript function in the page that reloaded the textarea but id rather do it in php if i only knew how 😃

TIA

g00fy

[mogster]

Use no-cache statements on the page with the textarea 😃

## The headers - possibly not required, but to overcome the browsers persistent idea of caching....well ##
header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
header("Cache-Control: no-cache, must-revalidate");
header("Pragma: no-cache");

knutm :-)

[g00fy_m]

yeah,

also tried that with meta tags 🙁

this is what i have now, i still need to refresh and get the `ol POSTDATA refresh warning 🙁

----- header.inc.php

<?php 

global $version;
global $logo_large;
global $logo_alt;

print('<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <HEAD>
    <TITLE>' . $version . ' :: ' . $page . '</TITLE>
    <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css">
<!--

a:link {
    font-family: Verdana, Arial, Helvetica, sans-serif;
    font-size: 10px;
    font-weight: bold;
    text-transform: capitalize;
  text-decoration:none;
  color:#000000;
    text-align: center;
    vertical-align: middle;
}

.reason {
    font-family: Verdana, Arial, Helvetica, sans-serif;
    font-size: 14px;
    font-style: italic;
    font-weight: bold;
    text-align: center;
    vertical-align: middle;
}

.postAmble {
    font-family: Verdana, Arial, Helvetica, sans-serif;
    font-size: 14px;
    font-style: italic;
    font-weight: bold;
  color: #FF0000;
    text-align: center;
    vertical-align: middle;
}

.rule {
    width: 30%;
  height: 2px;
}

.picCentre {
    text-align: center;
    vertical-align: middle;
}

.copyright {
    font-family: Verdana, Arial, Helvetica, sans-serif;
    font-size: 10px;
    font-weight: bold;
    text-transform: capitalize;
    text-align: center;
    vertical-align: middle;
}

.version {
    font-family: Verdana, Arial, Helvetica, sans-serif;
    font-size: 12px;
    font-weight: bold;
    text-align: center;
    vertical-align: middle;
}
-->
</style>
  </HEAD>
  <BODY>');      

   if(is_readable("$logo_large")){
     print('
       <div class="picCentre">
         <img src="' . $logo_large . '" border="0" align="middle" alt="' . $logo_alt .'">
       </div>
    ');
   }     

  print('
    <HR class="rule">
  ');

?>

-----  subskribe.php 

/**
* Function to generate the administration page for the database file
*/
function showAdmin(){


  header("Expires: Mon, 26 Jul 1997 05:00:00 GMT"); 
  header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT"); 
  header("Cache-Control: no-cache, must-revalidate"); 
  header("Pragma: no-cache");

  include('header.inc.php');

  print ('
  <TABLE align="center" width="50%" />
    <TR>
      <td align="center" /> 
        <form action="'.$_SERVER['PHP_SELF'].'" method="POST">');         

  print('
          <TEXTAREA style="WIDTH: 337px; HEIGHT: 159px" rows=8 cols=35>
');
  // output all subskribers in the database to the textarea
  displayUsers();
  print ('
          </TEXTAREA>
          <BR><Br>
        ');
  print (entries() . ' subskribers in database.
          <BR> ' .
  $file .
  '  <br><br>
           <INPUT style="WIDTH: 192px; HEIGHT: 22px" size=23 name="subskriber" id="subskriber">
           &nbsp;&nbsp;
           <INPUT type=submit value="Add" name="add" id="add" onSubmit="">   

           &nbsp;&nbsp;
           <INPUT type=submit value="Remove" name="delete" id="delete">
         </form>
       </TD>
     </TR>
   </TABLE>
  ');

  include('footer.inc.php');
}

thanx again

g00fy

[mogster]

Ah, I see. You're using the js just to get around the warning of doublepost by doing a back() to the page the data was posted to?

Well, use an ordinary link instead, in the success-script?
This will clear the post-vars and the page will reload the new data.

Or am I missing something here? 😃

knutm :-)

[g00fy_m]

the thing is....

the page i came from is generated by the script in the showAdmin() function, so i cant figure out where to link it to i tried '.$_SERVER['PHP_SELF'].' but it just refreshes the response page as it too is generated by the script.

am i going about this all wrong having everything in one script or not ??

i am only new to php, java & c++ are my forte'

i assume im on the same track, being that the link you are talking about is back to the referer page, which is subskribe.php

i can post he whole thin but it is getting up around 500 lines :o

thanx again

g00fy

[mogster]

Hmm.. tried a switched page before?

It's a nice way to control code-execution and viewing.
The switch is saved as a script, then the different cases is treated as a whole page. The cases can be specified with $POST, $GET and $_COOKIE.

Here's a suggestion:

## Function for finding values from two strings ##
function c_Cval($val_n, $val_o) {
if($val_n) {
$str = $val_n;
} else {
$str = $val_o;
}
return $str;
}

## Specify run control from vars ##
if(c_Cval($_GET['run_control'], $HTTP_GET_VARS['run_control'])) {
$run_control = c_Cval($_GET['run_control'], $HTTP_GET_VARS['run_control']);
} else {
$run_control = c_Cval($_POST['run_control'], $HTTP_POST_VARS['run_control']);
}

## main switch for page ##
switch($run_control) {

## delete id ##
case "Delete":
## do the deletion stuff here
if($deletion_was_successful) {
header("Location:".$_SERVER['PHP_SELF']."?run_control=success");
} else {
header("Location:".$_SERVER['PHP_SELF']."?run_control=failure");
}
break;

## delete id ##
case "Add":
## do the adding stuff here
if($adding_was_successful) {
header("Location:".$_SERVER['PHP_SELF']."?run_control=success");
} else {
header("Location:".$_SERVER['PHP_SELF']."?run_control=failure");
}
break;

## show success ##
case "success":
include('header.inc.php');
echo "Success!<br><a href=\"".$_SERVER['PHP_SELF']."\">Go back to list</a>";
include('footer.inc.php');
break;

## show failure ##
case "failure":
include('header.inc.php');
echo "Error!<br><a href=\"".$_SERVER['PHP_SELF']."\">Go back to list</a>";
include('footer.inc.php');
break;

## just show form with textfield ##
default:
include('header.inc.php');
print (' 
<TABLE align="center" width="50%" /> 
<TR> 
<td align="center" /> 
<form action="'.$_SERVER['PHP_SELF'].'" method="POST">');          

print(' 
<TEXTAREA style="WIDTH: 337px; HEIGHT: 159px" rows=8 cols=35> 
'); 
// output all subskribers in the database to the textarea 
displayUsers(); 
print (' 
</TEXTAREA> 
<BR><Br>'); 
print (entries() . ' subskribers in database. 
<BR> ' . 
$file . 
'  <br><br> 
<INPUT style="WIDTH: 192px; HEIGHT: 22px" size=23 name="subskriber" id="subskriber"> 
           &nbsp;&nbsp; 
           <INPUT type=submit value="Add" name="run_control" onSubmit="">    

           &nbsp;&nbsp; 
           <INPUT type=submit value="Delete" name="run_control">
         </form> 
       </TD> 
     </TR> 
   </TABLE> 
  '); 
include('footer.inc.php');
}

You get the drift?
This is probably full of parse-errors, but if you fix it, it'll be a much more sound structure as you can add as many cases as you like in the switch. So if a new need appear, just add it :p

[edit] If no value of run_control is specified, it defaults to default 😉 [/edit]
knutm :-)

[g00fy_m]

thanx heaps mogster,

the `ol case / switch statement ... of course

thanx again for all your help, i will try it later but the idea looks good 😃

ill mark as resolved for now as i shouldnt have any trouble implementing that

kind regards & thanx

g00fy