[PHP]Po zaznaczeniu checbox chce pobierać plik Opcje

[amii]

Bardzo dziwna sprawa. Po pierwsze to funkcja dodaj() jest przekopiowana ze skryptu gdzie wszystko działa OK. Po drugie inne checkboxy działają OK. Problem jest taki, że jak zaznaczam checkboxy dla akcji pinguj (tej z pliku functions.php) to firefox chce mi pobierać plik admin.php a IE wyświetla: Program Internet Explorer nie może wyświetlić witryny sieci Web.

Plik admin.php

[PHP] pobierz, plaintext 
echo '<form method="post" id="formularz_checkbox" name="formularz_checkbox" action="'.SITE.'admin.php?action=dodaj&amp;task=usun">';
                $przycisk =  html_input('submit', 'usun_zaznaczone_name', 'usun_zaznaczone_id', 'Usun 
 
zaznaczone', '', 'button', 'onclick="java script: return pop()"', '', '', '', '', '', '', '', '');
                $przycisk2 = html_input('submit', 'pinguj_zaznaczone_name', 'pinguj_zaznaczone_id', 'Pinguj 
 
zaznaczone', '', 'button', '', '', '', '', '', '', '', '', '');
                echo '<p><input type="checkbox" name="zaznacz_all_name" id="zaznacz_all_id" 
 
onClick="zaznacz(this, \'formularz_checkbox\')" /> Zaznacz wszystko: ' .$przycisk . $przycisk2. ' </p>';
 
                while ($row2 = mysql_fetch_array($zapytanie3)) {
                $wartosc_bool = $row2['tick'];
 
                if($wartosc_bool) $zdjecie = '<img float="left" display="inline" src="'.SITE.'images/index.png" alt="strona zaindeksowana" height="16" width="16">';
                else $zdjecie = '<img float="left" display="inline" src="'.SITE.'images/index_no.png" alt="strona niezaindeksowana" height="16" width="16">';
 
                $link = '<a target="_blank" href="http://www.google.pl/#hl=pl&amp;source=hp&amp;q=site:'.$row2['url'].'&amp;aq=f">'.$row2['url'].'</a>';
                $podstaw = '<input type="checkbox" name="checkbox_name[]" id="checkbox_id_'.$row2['url'].'" 
 
value="'.$row2['url'].'<:(IMG:style_emoticons/default/snitch.gif) '.$row2['id'].'" />';
 
                echo '<p>' . $podstaw. '&nbsp;&nbsp;&nbsp; ' .$zdjecie. ' &nbsp;&nbsp;&nbsp;<a target="_blank" href="http://www.google.pl/#hl=pl&amp;source=hp&amp;q=site:'.$row2['url'].'&amp;aq=f">'.$row2['url'].'</a> 
 
</p>';
                }
                echo $renderPager1;  //wyswietlenie paginatora
                echo '</form>';
[PHP] pobierz, plaintext 

Plik functions.php

[PHP] pobierz, plaintext 
// Funkcja generująca przyciski formularza
function html_input($type, $name, $id, $value, $label, $css, $script1, $script2, $script3, $checked, $rows, $cols, $method, $action, $legend) {
	$lbl = !empty($label) ? '<label for="'.$id.'">'.$label.'</label>' : '';
	$ID = !empty($id) ? ' id="'.$id.'"' : '';
	$style = !empty($css) ? ' class="'.$css.'"' : '';
	$js1 = !empty($script1) ? ' '.$script1 : '';
	$js2 = !empty($script2) ? ' '.$script2 : '';
	$js3 = !empty($script3) ? ' '.$script3 : '';
	$attribs = $ID.$style.$js1.$js2.$js3;
	$val = ' value="'.$value.'"';
	$input = '<input type="'.$type.'" name="'.$name.'"'.$attribs;
	switch($type) {
		case 'form': $output = (!empty($method) && $method != 'end') ?
			'<form method="'.$method.'" action="'.$action.'"'.$attribs.'>' : '</form>'; break;
		case 'fieldset': $output = (!empty($legend) && $legend != 'end') ?
			'<fieldset><legend'.$attribs.'>'.$legend.'</legend>' : '</fieldset>'; break;
		case 'text':
		case 'password': $output = '<p>'.$lbl.':<br />'.$input.$val.' /></p>'; break;
		case 'checkbox':
		case 'radio': $check = $checked == 'ok' ? ' checked="checked"' : ''; $output = '<p>'.$input.$check.' /> '.$lbl.'</p>'; break;
		case 'hidden':
		case 'submit':
		case 'reset':
		case 'button': $output = $input.$val.' />'; break;
		case 'textarea':
			$output = '<p>'.$lbl.':<br />
			<textarea name="'.$name.'" rows="'.$rows.'" cols="'.$cols.'"'.$attribs.'>'.$value.
			'</textarea></p>'; break;
	}
	return $output;
}
 
 
function dodaj() {
if (isset($_GET['task']))  {   
 
     switch($_GET['task']) {     
 
case 'usun':
 
   if (isset($_POST['pinguj_zaznaczone1_name'])) {
       $title = array('Alkomaty dowodowe', 'Uzdatnianie wody', 'Filtry do wody', 'Alkomaty');  //tytuły do pingowania
       $losowy_rekord = mt_rand(0, count($title)-1);
          foreach($_POST['checkbox_name1'] as $key => $value) {
          $value = explode('<:(IMG:style_emoticons/default/snitch.gif) ', $value);
          $extract = trim($value[0]);
          ping($title[$losowy_rekord], $extract);
 
          }
          unset($_POST['pinguj_zaznaczone1_name']);
       }
 
       if (isset($_POST['pinguj_zaznaczone_name'])) {
       $title = array('Alkomaty dowodowe', 'Uzdatnianie wody', 'Filtry do wody', 'Alkomaty');  //tytuły do pingowania
       $losowy_rekord = mt_rand(0, count($title)-1);
          foreach($_POST['checkbox_name'] as $key => $value) {
          $value = explode('<:(IMG:style_emoticons/default/snitch.gif) ', $value);
          $extract = trim($value[0]);
          ping($title[$losowy_rekord], $extract);
 
          }
          unset($_POST['pinguj_zaznaczone_name']);
       }
     }
  }
}
[PHP] pobierz, plaintext 

[wNogachSpisz]

[PHP] pobierz, plaintext 
html_input($type, $name, $id, $value, $label, $css, $script1, $script2, $script3, $checked, $rows, $cols, $method, $action, $legend)
[PHP] pobierz, plaintext 

omfg....

Nie lepiej dać coś w stylu

[PHP] pobierz, plaintext 
$settings = array(
  'type' => 'text',
  'id' => 'myId',
  ...
);
 
html_input($settings)
[PHP] pobierz, plaintext 

--------------------------------------------------------

Czepiania ciąg dalszy:

[PHP] pobierz, plaintext 
$lbl = !empty($label) ? '<label for="'.$id.'">'.$label.'</label>' : '';
$ID = !empty($id) ? ' id="'.$id.'"' : '';
$style = !empty($css) ? ' class="'.$css.'"' : '';
$js1 = !empty($script1) ? ' '.$script1 : '';
[PHP] pobierz, plaintext 

W ten sposób nie sprawdza się czy argument został przekazany....
Taki kod generuje mase ostrzeżeń,
nawet jeśli masz wyłączone pokazywanie błędów, to mimo wszytko PHP przejdzie procedurę jego obsługi, to pochłania mase zasobów..

Używaj isset();

--------------------------------------------------------

Pokaż jak wygląda funkcja ping()

Ten post edytował wNogachSpisz 25.02.2011, 15:05:44