Bonjour à tous,

Mon but est de recopier les données d'un abonné modèle (id_abo=1) lors de la création d'un nouvel abonné (par exemple id_abo=2).
Je me pose des questions concernant :
  1. le choix de la méthode: N'y a t-il pas plus simple?
  2. la sécurisation: risque d'injection ou de piratage?
  3. l'optimisation: remplacement de la boucle foreach?


Code php : Sélectionner tout - Visualiser dans une fenêtre à part
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
<?php
function createDefaultRightsGroups($id_abo) {
	global $db;
 
	try {
		$db->beginTransaction();
 
		$select = "
			SELECT designation, comment, access_type, rights
			FROM lst_rights
			WHERE id_abo=1
		;";
		$data = $db->query($select)->fetchAll();
 
		$insert = "
			INSERT IGNORE INTO lst_rights
				(id_abo, designation, comment, access_type, rights
				VALUES(:id_abo, :designation, :comment, :access_type, :rights)
		;";
		$result = $db->prepare($insert);
		$result->bindParam('id_abo', $id_abo);
		foreach ($data as $row) {
			$result->bindParam('designation', $row->designation);
			$result->bindParam('comment', $row->comment);
			$result->bindParam('access_type', $row->access_type);
			$result->bindParam('rights', $row->rights);
			$result->execute();
		}
	}
	catch (PDOException $e) {
		$db->rollBack();
		return false;
	}
	$db->commit();
	return true;	
}
createDefaultRightsGroups(2);
?>

EDIT: Par ailleurs, la colonne 'comment' de la première ligne provoque une erreur de syntaxe 4200/1064.
Contenu de l'objet $row:
Code : Sélectionner tout - Visualiser dans une fenêtre à part
1
2
3
4
5
6
object(stdClass)[4]
  public 'designation' => string 'Superviseur' (length=11)
  public 'comment' => string 'Le groupe superviseur comprend les personnes responsables de l'application.
Ce groupe détient tous les droits.' (length=112)
  public 'access_type' => string '3' (length=1)
  public 'rights' => string '2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,' (length=38)