lang'o'login #1

Open
RealET wants to merge 3 commits from langologin into master
RealET commented 10 months ago
Collaborator

Avec cette PR, au login, on bascule sur la langue de l'auteur.

Serait à associer avec spip/spip#4912 pour bien faire les choses jusqu'au bout.

Avec cette PR, au login, on bascule sur la langue de l'auteur. Serait à associer avec https://git.spip.net/spip/spip/issues/4912#issuecomment-28727 pour bien faire les choses jusqu'au bout.
RealET added 2 commits 10 months ago

Vu que par défaut la lang de spip_auteurs c'est en priorité pour l'admin, et que par défaut ça permet d'avoir une langue différente en admin et en public, du coup est-ce que ce comportement devrait pas plutôt être seulement sur option ? À n'activer que quand on est sûr que c'est bien ça qu'on veut (par ex pour des sites/applis qui ont plein de comptes visiteurs publics)

Vu que par défaut la lang de spip_auteurs c'est en priorité pour l'admin, et que par défaut ça permet d'avoir une langue différente en admin et en public, du coup est-ce que ce comportement devrait pas plutôt être seulement sur option ? À n'activer que quand on est sûr que c'est bien ça qu'on veut (par ex pour des sites/applis qui ont plein de comptes visiteurs publics)
Collaborator

La fonction n'a pas l'air de marcher et envoie un cookie vide.

Donc vérifier le sql_fetsel peut etre ? Dans mon test en plus on a déja la langue dans $flux['args']['row']. Enfin le commenatire au dessus de la fonction n'a rien a voir.

Au final ca fonctionne mieux avec ceci chez moi :

function autolang_preparer_visiteur_session($flux) {
	
	if (!empty($flux['args']['row']['id_auteur'])) {
		$langue = $flux['args']['row']['lang'];
		
		if ($langue !== '' AND !isset($_COOKIE["spip_lang"])) {
			include_spip('inc/cookie');
			spip_setcookie('spip_lang', $langue);
		}
	}
	return $flux;
}
La fonction n'a pas l'air de marcher et envoie un cookie vide. Donc vérifier le sql_fetsel peut etre ? Dans mon test en plus on a déja la langue dans `$flux['args']['row']`. Enfin le commenatire au dessus de la fonction n'a rien a voir. Au final ca fonctionne mieux avec ceci chez moi : ``` function autolang_preparer_visiteur_session($flux) { if (!empty($flux['args']['row']['id_auteur'])) { $langue = $flux['args']['row']['lang']; if ($langue !== '' AND !isset($_COOKIE["spip_lang"])) { include_spip('inc/cookie'); spip_setcookie('spip_lang', $langue); } } return $flux; } ```
Poster
Collaborator

@booz : j'ai fait ça pour @rastapopoulos.
Je n'en ai ni l'usage, ni la possibilité de tester.

Donc, je te fais confiance ;-)

@booz : j'ai fait ça pour @rastapopoulos. Je n'en ai ni l'usage, ni la possibilité de tester. Donc, je te fais confiance ;-)
RealET added 1 commit 9 months ago
This pull request has changes conflicting with the target branch.
paquet.xml
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.