google_id absent de la table

[brunodegoyrans]

Dans le fichier magiclogin_with_google.php, il est fait référence au champs google_id de la table spip_auteurs.
Dans l'installation que j'ai, ce champs n'a pas été ajouté alors que le champs facebook_id a bien été ajouté.
Est-ce normal ?

Du coup, cela pourrait expliquer que j'obtiens systématiquement le formulaire de signup même si je me suis déjà connecté avec GG ?

[brunodegoyrans]

Dans magiclogin_administrations.php :

	$maj['create'] = array(
		array('sql_alter',"TABLE spip_auteurs ADD twitter_token VARCHAR(255) DEFAULT '' NOT NULL"),
		array('sql_alter',"TABLE spip_auteurs ADD twitter_token_secret VARCHAR(255) DEFAULT '' NOT NULL"),
		array('sql_alter',"TABLE spip_auteurs ADD facebook_id VARCHAR(255) DEFAULT '' NOT NULL"),
	);

	$maj['0.2.0'] = array(
		array('sql_alter',"TABLE spip_auteurs ADD twitter_token VARCHAR(255) DEFAULT '' NOT NULL"),
		array('sql_alter',"TABLE spip_auteurs ADD twitter_token_secret VARCHAR(255) DEFAULT '' NOT NULL"),
		array('sql_alter',"TABLE spip_auteurs ADD facebook_id VARCHAR(255) DEFAULT '' NOT NULL"),
	);
	$maj['0.2.1'] = array(
		array('sql_alter',"TABLE spip_auteurs ADD google_id VARCHAR(255) DEFAULT '' NOT NULL"),
	);

Le champs google_id n'est ajouté qu'en cas de mise à jour du plugin en 0.2.1. Mais si on part d'une installation nouvelle du plugin, est-ce qu'on ne tombe pas dans le cas $maj['create'] et donc la colonne google_id n'est pas ajoutée ?

[Cerdic]

En effet c'est visiblement la cause du bug

[brunodegoyrans]

Je ne maîtrise pas trop le mécanisme de xxx_upgrade mais je pense qu'il suffit d'ajouter la ligne de google_id dans $maj['create']

	$maj['create'] = array(
		array('sql_alter',"TABLE spip_auteurs ADD twitter_token VARCHAR(255) DEFAULT '' NOT NULL"),
		array('sql_alter',"TABLE spip_auteurs ADD twitter_token_secret VARCHAR(255) DEFAULT '' NOT NULL"),
		array('sql_alter',"TABLE spip_auteurs ADD facebook_id VARCHAR(255) DEFAULT '' NOT NULL"),
		array('sql_alter',"TABLE spip_auteurs ADD google_id VARCHAR(255) DEFAULT '' NOT NULL"),
	);

Je commiterai la correction prochainement.

[brunodegoyrans]

A partir du moment où le bugue est corrigé, le google_id est bien enregistré en base. Pour les connexions suivantes, le visiteur a juste à cliquer sur le bouton GG et il est directement connecté. Il n'y a plus de mail de confirmation qui lui est envoyé systématiquement.

Avant la correction, le google_id n'était pas enregistré. Donc en cas de reconnexion, magiclogin voyait que le compte existait déjà, mais sans lien avec un compte Google. Donc on tombait dans le cas d'un auteur existant et alors un mail de confirmation était envoyé.