PHPhulp.nl PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
D

Stom checkbox probleem

Door Donald Boers op 11-12-2015 09:05 gewijzigd op 11-12-2015 09:07
2.059 views
Op een website voor scooter onderdelen heb ik op de update pagina voor een onderdeel 5 chekboxen staan met modellen waar dit onderdeel voor geschikt is (komt uit tafel modellen):

<?php foreach ( $selected_modellen as $model ): ?>
<input name="model_id[]" type="checkbox" value="<?php echo $model['model_id']; ?>"> 		
<?php endforeach; ?>

Bij het invoeren van de diversen onderdelen heb ik de modellen waarop een onderdeel past, ook gebruikmakend van checkboxen, ingevoerd in een many-to-many tafel (onderdeel_modellen):

foreach ($model_id as $model)
{
	$this->onderdelen->onderdeel_modellen_toevoegen($onderdeel_id, $model);
}

Deze gegevens haal ik vervolgens ten behoeve van de update pagina uit de database:

public function get_onderdeel_modellen($onderdeel_id)
{
	$sql = "SELECT *
              FROM onderdeel_modellen
			 WHERE onderdeel_id = ?";

	$stmt = $this->pdo->prepare($sql);
	$stmt->execute(array($onderdeel_id));
		
	return $stmt->fetchAll(); 
}

Waar ik nu mee worstel is hoe ik het juiste aantal checkboxen in de bovenstaande loop gechecked krig op basis van de bovenstaande query. Ik heb een aantal dingen geprobeerd zoals:

<input name="model_id[]" type="checkbox" value="<?php echo $model['model_id']; ?>" <?php if ($onderdeel_modellen['model_id'] == $model['model_id']) ?> checked <?php endif; ?>>

Maar ik krijg het niet werkend
J
Jan de Laet
11-12-2015 09:36
Je value is nu $model['model_id'], zou dat niet de 'id' van tabel onderdeel_modellen moeten zijn?
D
Donald Boers
11-12-2015 10:19 gewijzigd op 11-12-2015 10:24
Jan de Laet op 11/12/2015 09:36:06

Je value is nu $model['model_id'], zou dat niet de 'id' van tabel onderdeel_modellen moeten zijn?

Nee alleen de model_id's die in onderdeel_modellen staan en bij het betreffende product horen dienen gecheked te zijn.
Dit is de data in de tafel modellen die ik gebruik voor het genereren van de model checkboxen:

INSERT INTO `modellen` (`model_id`, `model`) VALUES
(1, 'A35'),
(2, 'Flexer'),
(3, 'PackR'),
(4, 'Revival'),
(5, 'Standaard'),
(6, 'Youngster');

en dit is een voorbeeld van data in tafel onderdeel_modellen:

INSERT INTO `onderdeel_modellen` (`id`, `onderdeel_id` `model_id`) VALUES
(7, 1, 213),
(8, 2, 213),
(9, 3, 213);

dus bij product 213 dienen alleen de eerste drie van de totaal zes checkboxen gechecked te zijn.

Maakt dit het wat duidelijker

Toevoeging op 11/12/2015 11:45:07:

Ik heb het geprobeerd met een ingesloten foreach loop:

<?php foreach ($modellen as $model): ?> // foreach loop voor het genereren van de checkboxen
	<?php 
		$checked = array();
		foreach ($selected_modellen as $selected): 
			$checked[] = $selected['model_id'];?>                	
    <?php endforeach; ?>
    <input name="model_id[]" type="checkbox" value="<?php echo $model['model_id']; ?>" <?php if ($checked == $model['model_id']): ?>checked<?php endif; ?>>
<?php endforeach; ?>

Maar helaas zonder resultaat. Wat zie ik over het hoofd?
M
Max Vaessen
11-12-2015 12:57 gewijzigd op 11-12-2015 12:59
Hi Ronald,

Zou je voor mij eens een opzet van de arrays '$modellen' en '$selected_modellen' kunnen laten zien?
Dan kan ik er even naar kijken.

PS: conventie tip: in het scriptje waar je de input genereert met daarbij of deze selected is of niet, gebruik je veel < ?php. Persoonlijk zou ik alles in 1 < ?php zetten en de input dus printen / echo'en. Dit is on-the-long-run een stuk efficienter omdat de PHP parser niet voor elke regel moet kijken of hij PHP of HTML voorgeschoteld krijgt en opnieuw moet parsen.

Max
D
Donald Boers
11-12-2015 13:27 gewijzigd op 11-12-2015 13:29
Hi Max. Hartelijk dank voor je reactie en tips

//Model modellen

SELECT model_id
     , model
  FROM modellen


//Model onderdeel_modellen

SELECT model_id
  FROM onderdeel_modellen
 WHERE onderdeel_id = ?


In de Controller


$onderdeel_id      = filter_input(INPUT_POST, 'onderdeel_id', FILTER_SANITIZE_NUMBER_INT);
$selected_modellen = $this->artikelen->get_selected_modellen($onderdeel_id);
$modellen          = $this->artikelen->get_modellen_dropdown();


Waarbij $selected_modellen en $modellen uiteraard naar de view gaan. Is dit voldoende info?

J
Jan de Laet
11-12-2015 13:41 gewijzigd op 11-12-2015 13:41
In je reactie van 10:19 lijken mij in de onderdeel_modellen de velden onderdeel_id en model_id omgewisseld.
Bij 213 is onderdeel en 1 is model_id
M
Max Vaessen
11-12-2015 13:42
Hoi Donald,

Zou je voor mij eens een print_r(); willen doen van de arrays die je in je controller hebt staan? Dan kan ik de situatie iets gemakkelijker nabootsen, dan op wat uitleg afgaan (foutgevoelig, ook van mijn kant) :)

D
Donald Boers
11-12-2015 14:39 gewijzigd op 11-12-2015 14:40
Max Vaessen op 11/12/2015 13:42:24

Hoi Donald,

Zou je voor mij eens een print_r(); willen doen van de arrays die je in je controller hebt staan? Dan kan ik de situatie iets gemakkelijker nabootsen, dan op wat uitleg afgaan (foutgevoelig, ook van mijn kant) :)

Hoi Max. Ik ben er uit. In de Controller heb ik met de resultaten van de selected_modellen een nieuwe array samengesteld:

$id = array();
foreach ($selected_modellen as $selected)
{
	$id[] =  $selected['model_id'];	
}


en vervolgens de $id array naar de view gestuurd inplaats van $selected_modellen en dan in de view:

<?php foreach ($modellen as $model): ?>          	
	<input name="model_id[]" type="checkbox" value="<?php echo $model['model_id']; ?>" <?php if (in_array($model['model_id'], $id)): ?>checked<?php endif; ?>><?php echo $model['model']; ?>
<?php endforeach; ?>


Hartelijk dank voor je tijd en input. Heeft me reuze geholpen


M
Max Vaessen
11-12-2015 14:47 gewijzigd op 11-12-2015 14:47
Hoi Donald,

Zo veel had ik nog niet gedaan :), maar fijn dat je geholpen bent!

Succes!

Reageren

Inloggen om te reageren