Forum PHP.pl

Cześć,

Pisze plugin do Wordpressa i mam problem. Nie działa mi przekazywanie zmiennych wprowadzonych przez użytkownika. Jeżeli wyrzucę z mojego kodu operatory IF i Switch to zaczyna działać. Mam tu na myśli funkcję, która znajduje się na samym dole.

[PHP] pobierz, plaintext 
function ow_display_user() {
    wp_enqueue_script('jquery-ui-datepicker');
    wp_enqueue_style('jquery-style', 'http://ajax.googleapis.com/ajax/libs/jqueryui/1.8.2/themes/smoothness/jquery-ui.css');
 
    global $wpdb;
    $results = $wpdb->get_results ("SELECT adres FROM wp_ow_adres;");
    ?>
<h2>Odczyt wodomierzy</h2>
<form method="post" action="">
    <table>
        <tbody>
            <tr>
                <th><label for="Adres">Adres zamieszkania</label></th>
                <td><select id="adres" name="adres">
                        <option value="">Wybierz swój adres</option>
                        <?php 
                        foreach ( $results as $result ) {
        echo '<option>'.$result->adres.'</option>';
    }
    ?>
                </select></td>
            </tr>
            <input type="submit" name="submit" value="Wybierz" />
        </tbody>
    </table>
</form>
<?php 
$adres = $_POST["adres"];
global $wpdb;
if (isset($_POST['submit'])){
        $spr_liczbe_wodo = $wpdb->get_var("select liczba_wodomierzy from wp_ow_adres where adres='$adres'");
        switch ($spr_liczbe_wodo) {
            case 1:
                ?>
<form name="adres_form" method="post" action="">
    <table>
        <tbody>
            <tr>
                <td>Adres zamieszkania:</td>
                <td><?php echo $adres;?></td>
            </tr>
            <tr>
                <td>Stan wodomierza:</td>
                <td><input type="text" name="stan_wodomierza" /></td>
            </tr>
            <tr>
                <td>Data odczytu:</td>
                <td><input type="text" name="data_odczytu" id="datepicker" /></td>
            </tr>
            <tr>
                <td>Kod lokalu:</td>
                <td><input type="text" name="kod_lokalu" /></td>
            </tr>
            <tr>
                <td></td>
                <td><input type="submit" name="submit" value="Zapisz" /></td>
            </tr>
        </tbody>
    </table>
</form>
<?php 
break;
case 2:
    ?>
<form name="adres_form" method="post" action="">
    <table>
        <tbody>
            <tr>
                <td>Adres zamieszkania:</td>
                <td><?php echo $adres;?></td>
            </tr>
            <tr>
                <td>Woda ciepła:</td>
                <td><input type="text" name="woda_ciepla" /></td>
            </tr>
            <tr>
                <td>Woda zimna:</td>
                <td><input type="text" name="woda_zimna" /></td>
            </tr>
            <tr>
                <td>Data odczytu:</td>
                <td><input type="text" name="data_odczytu" id="datepicker" /></td>
            </tr>
            <tr>
                <td>Kod lokalu:</td>
                <td><input type="text" name="kod_lokalu" /></td>
            </tr>
            <tr>
                <td></td>
                <td><input type="submit" name="submit" value="Zapisz" /></td>
            </tr>
        </tbody>
    </table>
</form>
<?php
break;
case 4:
    ?>
<form method="POST" action="">
    <table>
        <tbody>
            <tr>
                <td>Adres zamieszkania:</td>
                <td><?php echo $adres;?></td>
            </tr>
            <tr>
                <td>Woda ciepła (kuchnia):</td>
                <td><input type="text" name="woda_ciepla_kuchnia" /></td>
            </tr>
            <tr>
                <td>Woda ciepła (łazienka):</td>
                <td><input type="text" name="woda_ciepla_lazienka" /></td>
            </tr>
            <tr>
                <td>Woda zimna (kuchnia):</td>
                <td><input type="text" name="woda_zimna_kuchnia" /></td>
            </tr>
            <tr>
                <td>Woda zimna (łazienka):</td>
                <td><input type="text" name="woda_zimna_lazienka" /></td>
            </tr>
            <tr>
                <td>Data odczytu:</td>
                <td><input type="text" name="data_odczytu" id="datepicker" /></td>
            </tr>
            <tr>
                <td>Kod lokalu:</td>
                <td><input type="text" name="kod_lokalu" /></td>
            </tr>
            <tr>
                <td></td>
                <td><input type="submit" name="submit" value="Zapisz" /></td>
            </tr>
        </tbody>
    </table>
</form>
<?php
if (isset($_POST['submit'])){
    global $wpdb;
    $kod_lokalu = $_POST['kod_lokalu'];
    $spr_kod_lokalu = $wpdb->get_var("select kod_lokalu from wp_ow_adres where adres='. $adres .'");
    if ($spr_kod_lokalu == $kod_lokalu) {
            $kod_lokalu = $_POST['data_odczytu'];
            $woda_ciepla_kuchnia = $_POST['woda_ciepla_kuchnia'];
            $woda_ciepla_lazienka = $_POST['woda_ciepla_lazienka'];
            $woda_zimna_kuchnia = $_POST['woda_zimna_kuchnia'];
            $woda_zimna_lazienka = $_POST['woda_zimna_lazienka'];
            $data_odczytu = '0000-00-00 00:00:00';
 
            $data = array(
                    'id' => '',
                    'adres' => $adres,
                    'data_odczytu' => $data_odczytu,
                    'woda_ciepla_kuchnia' => $woda_ciepla_kuchnia,
                    'woda_ciepla_lazienka' => $woda_ciepla_lazienka,
                    'woda_zimna_kuchnia' => $woda_zimna_kuchnia,
                    'woda_zimna_lazienka' => $woda_zimna_lazienka,
            );
 
            $wpdb->insert(`wp_ow_odczyty`, $data);
    }
}
break;
}
 
    }
}
[PHP] pobierz, plaintext 

--edycja

Mam wrażenie jakby formularze się sumowały. Jeżeli to jest możliwe.

Ten post edytował suchy1 30.12.2013, 10:25:54

[PHP] pobierz, plaintext 
$spr_liczbe_wodo = $wpdb->get_var("select liczba_wodomierzy from wp_ow_adres where adres='$adres'");
[PHP] pobierz, plaintext 

tutaj kropek wydaje sie brakowac

Jeżeli dodam kropki kolo $adres to nie działa. Wrzucam jeszcze raz kod w mniejszej formie. Obecnie 'woda_ciepla_kuchnia' => $woda_ciepla_kuchnia, zawsze przyjmuje wartość 0 niezależnie od tego co użytkownik wprowadzi.

[PHP] pobierz, plaintext 
function ow_display_user() {
	wp_enqueue_script('jquery-ui-datepicker');
	wp_enqueue_style('jquery-style', 'http://ajax.googleapis.com/ajax/libs/jqueryui/1.8.2/themes/smoothness/jquery-ui.css');
 
	global $wpdb;
	$results = $wpdb->get_results ("SELECT adres FROM wp_ow_adres;");
	?>
<form method="post" action="">
	<table>
		<tbody>
			<tr>
				<th><label for="Adres">Adres zamieszkania</label></th>
				<td><select id="adres" name="adres">
						<option value="">Wybierz swój adres</option>
						<?php 
						foreach ( $results as $result ) {
	        echo '<option>'.$result->adres.'</option>';
	    }
	    ?>
				</select></td>
			</tr>
			<tr>
				<td><input type="submit" name="submit" value="Wybierz" />
 
				<td>
 
			</tr>
		</tbody>
	</table>
</form>
<?php 
$adres = $_POST["adres"];
$woda_ciepla_kuchnia = $_POST['woda_ciepla_kuchnia'];
if (isset($_POST['submit'])){
	$spr_liczbe_wodo = $wpdb->get_var("select liczba_wodomierzy from wp_ow_adres where adres='$adres'");
 
	if ($spr_liczbe_wodo == 4) {
 
		echo 	'<form method="post" action="">';
		echo 	'		<table>';
		echo 	'			<tbody>';
		echo 	'				<tr>';
		echo 	'					<td>Adres zamieszkania:</td>';
		echo 	'					<td>'. $adres .'</td>';
		echo 	'				</tr>';
		echo 	'				<tr>';
		echo 	'					<td>Woda ciepła (kuchnia):</td>';
		echo 	'					<td><input type="text" id="woda_ciepla_kuchnia" name="woda_ciepla_kuchnia" /></td>';
		echo 	'				</tr>			';			
		echo 	'				<tr>';
		echo 	'					<td></td>';
		echo 	'					<td><input type="submit" id="submit" name="submit" value="Zapisz" /></td>';
		echo 	'				</tr>';
		echo 	'			</tbody>';
		echo 	'		</table>';
		echo 	'	</form>';
 
			(isset($_POST['submit']));	
					    	$woda_ciepla_kuchnia = $_POST['woda_ciepla_kuchnia'];
						    $data_odczytu = '0013-00-00 22:00:00';
 
						    $data = array(
								'id' => '',
						    	'adres' => 'dasd',
						        'data_odczytu' => $data_odczytu,
						        'woda_ciepla_kuchnia' => $woda_ciepla_kuchnia,
 
						        );
 
						    $wpdb->insert(wp_ow_odczyty, (array)$data);
 
						    }
 
	}
 
}
[PHP] pobierz, plaintext 

Jak wspomniał fate. Wykonaj zapytanie z linii 35 np. w phpmyadminie i sprawdź wynik.
Dodatkowo http://codex.wordpress.org/Debugging_in_WordPress twoim przyjacielem (IMG:style_emoticons/default/wink.gif)

Wynik jest prawidłowy. System wchodzi w tego IF i prezentuje formularz. Jeżeli wybiore inny adres to nie prezentuje formularza więc to jest ok - moim zdaniem bo działa. Po uruchomieniu DEBUG w WP mam tylko kilka NOTICE do zmiennych, które nie zostały zdefiniowane.

Tylko woda_ciepla_kuchnia jest 0? Pozostałe dane przyjmują dane z formularza?

Sprawdź te NOTICEy - czy jeden z nich dotyczy $_POST['woda_ciepla_kuchnia']? Czy odpowiada linii 33 z Twojego ostatniego listingu?

Nie mogę wejść do ostatniego IF, jakby isset nie działał prawidłowo. Jeżeli go usunę to dodaje się wpis do bazy. Oczywiście z zerową wartością bo insert leci do bazy w momencie wyświetlenie formularza.

--edycja

Mam tylko jeden NOTICE dotyczący $adres = $_POST["adres"]; a nie powinno być także dla $_POST['woda_ciepla_kuchnia'] (IMG:style_emoticons/default/questionmark.gif)

Ten post edytował suchy1 30.12.2013, 12:50:27

Teraz zauważyłem w Twoim pierwszym listingu, że po podaniu i wysłaniu adresu, wyświetlą się dwa formularze, i oba z przyciskiem o nazwie "submit". Dobrze myślę, że ten pierwszy - z rozwijalną listą adresów zamieszkania - zawsze się wyświetla? Spróbuj na początek dać unikalne nazwy wszystkim formularzom i wszystkim przyciskom submit. Zobacz co się dzieje i daj znać.
Tuż pod od linią

[HTML] pobierz, plaintext 
<h2>Odczyt wodomierzy</h2>
[HTML] pobierz, plaintext 

dodaj

[PHP] pobierz, plaintext 
<pre style="text-align: left;">
<?php if ($_SERVER['REQUEST_METHOD'] === 'POST') var_dump($_POST);
</pre>
[PHP] pobierz, plaintext 

i przeanalizuj co tam drukuje.
Dla pewności pytam: ten pierwszy listing w Twoim pierwszym poście to pełny listing funkcji ow_display_user()?

<option value="">Wybierz swój adres</option>

$_POST adres jest puste bo optiony nie maja zadnego value tylko tekst opisujacy.

Dziękuje wszystkim za dotychczasowa pomoc. Cała funkcja:

[PHP] pobierz, plaintext 
function ow_display_user() {
	?>
	<pre style="text-align: left;">
	<?php if ($_SERVER['REQUEST_METHOD'] === 'POST') var_dump($_POST); ?>
	</pre>
	<? 
	wp_enqueue_script('jquery-ui-datepicker');
	wp_enqueue_style('jquery-style', 'http://ajax.googleapis.com/ajax/libs/jqueryui/1.8.2/themes/smoothness/jquery-ui.css');
 
	global $wpdb;
	$results = $wpdb->get_results ("SELECT adres FROM wp_ow_adres;");
	?>
<form name="one" method="post" action="">
	<table>
		<tbody>
			<tr>
				<th><label for="Adres">Adres zamieszkania</label></th>
				<td><select id="adres" name="adres">
						<option value="">Wybierz swój adres</option>
						<?php 
						foreach ( $results as $result ) {
	        echo '<option>'.$result->adres.'</option>';
	    }
	    ?>
				</select></td>
			</tr>
			<tr>
				<td><input type="submit" name="ok" value="Wybierz" />
 
				<td>
 
			</tr>
		</tbody>
	</table>
</form>
<?php 
$adres = $_POST["adres"];
 
if (isset($_POST['ok'])){
	$spr_liczbe_wodo = $wpdb->get_var("select liczba_wodomierzy from wp_ow_adres where adres='$adres'");
 
	if ($spr_liczbe_wodo == 4) {
	?>
		<form name="two" method="post" action="http://localhost.local/">
		<table>
		<tbody>
		<tr>
		<td>Adres zamieszkania:</td>
		<td><?php $adres?></td>
 		</tr>
 		<tr>
 		<td>Woda ciepła (kuchnia):</td>
 		<td><input type="text" id="woda_ciepla_kuchnia" name="woda" /></td>
 		</tr>
 		<tr>
 		<td></td>
 		<td><input type="submit" id="zapisz" name="zapisz" value="Zapisz" /></td>
 		</tr>
 		</tbody>
 		</table>
 		</form>
	<?
 
 		if(isset($_POST['zapisz']))
 		{
 		$woda_ciepla_kuchnia=1;
 		$data_odczytu = '0013-00-00 22:00:00';
 
				$data = array(
				'id' => '',
				'adres' => 'dasd',
				'data_odczytu' => $data_odczytu,
				'woda_ciepla_kuchnia' => 1,
 
				);
 
				$wpdb->insert(wp_ow_odczyty, (array)$data);
 
						    }
						    }
 
	}
 
}
[PHP] pobierz, plaintext 

Po dodaniu:

[PHP] pobierz, plaintext 
    <pre style="text-align: left;">
    <?php if ($_SERVER['REQUEST_METHOD'] === 'POST') var_dump($_POST);
    </pre>
[PHP] pobierz, plaintext 

wyświetla się:

[PHP] pobierz, plaintext 
array(2) {
  ["woda"]=>
  string(1) "4"
  ["zapisz"]=>
  string(6) "Zapisz"
}
[PHP] pobierz, plaintext 

ale dalej nie wpadam w if(isset($_POST['zapisz']))


Masz rację ale ta zmienna jest przekazywana dalej.

Ten post edytował suchy1 30.12.2013, 13:51:50

Bo jest w warunku

[PHP] pobierz, plaintext 
	if ($spr_liczbe_wodo == 4) {
[PHP] pobierz, plaintext 

Zgadza się. Warunek jest prawdziwy bo wyświetla mi się formularz.

Bo jest w warunku z wiersza 39:

[PHP] pobierz, plaintext 
if (isset($_POST['ok'])){
[PHP] pobierz, plaintext 

czyli w warunku sprawdzającym, czy wciśnięto przycisk OK, a wciśnięcie obu przycisków jednocześnie nie jest możliwe. Wyrzuć cały blok od wiersza 64 do 79 gdzieś na zewnątrz i podrasuj, żeby przechwytywał adres z poprzedniego formularza (np. zapisz adres w sesji).

@bostaf możesz mi pomóc jeszcze bardziej ? Nie znam się na sesji. Chodzi o to aby użytkownik wybrał swój adres. Następnie sprawdzam ile ma np. wodomierzy. na tej podstawie wyświetlam odpowiedni formularz. Dlatego tyle warunków dałem.

Ten post edytował suchy1 30.12.2013, 14:18:59

No wlasnie mowisz ze dalej ale w momentach w ktorych próbujesz z niej korzystac nie jest przypadkiem pusta

Sesja to oględnie mówiąc zmienne zapisywane po stronie serwera. Zasugerowałem Ci zmienne sesyjne, bo przetwarzasz kilka formularzy szeregowo i trzeba jakoś te informacje z poszczególnych etapów zapamiętać. W sesji najłatwiej.

W tym pierwszym bloku warunkowym if (isset($_POST['ok'])){ zapamiętaj adres w zmiennej sesyjnej, np.:

[PHP] pobierz, plaintext 
$_SESSION['wybrany_adres'] = $_POST['adres'];
[PHP] pobierz, plaintext 

a w drugim bloku warunkowym (if(isset($_POST['zapisz']))), zapisując dane w bazie odczytaj wartość tej zmiennej i od razu ją zlikwiduj, żeby nie śmiecić:

[PHP] pobierz, plaintext 
$data = array(
    'id' => '',
    'adres' => $_SESSION['wybrany_adres'],
    // ...
)
unset($_SESSION['wybrany_adres']);
[PHP] pobierz, plaintext