Anomalie #3981

Menu "Écrire une nouvelle traduction" non présent pour rédacteurs

Ajouté par Jacques Bouthier il y a 4 mois. Mis à jour il y a environ un mois.

Statut:FerméDébut:11/08/2017
Priorité:HautEchéance:
Assigné à:-% réalisé:

100%

Catégorie:-
Version cible:3.2
Resolution:fixed Navigateur:

Description

Le menu "Écrire une nouvelle traduction" n'est pas présent pour les rédacteurs pour un article à partir de SPIP 3.1.
C'est particulièrement gênant sur spip.net depuis le passage en 3.2 mais en fait ça semble vrai depuis SPIP 3.1 (SPIP 3.0 OK)

3981.diff Magnifier (1,331 ko) b b, 22/08/2017 19:44

Révisions associées

Révision 23690
Ajouté par brunobergot@gmail.com il y a 4 mois

Fix #3981 : permettre aux rédacteurs de créer une nouvelle traduction

Suite aux modifications relatives à #2746 un rédacteur ne pouvait plus créer de traduction sur un article dont il n'était pas l'auteur, on corrige ce bug.

Révision 23691
Ajouté par brunobergot@gmail.com il y a 4 mois

report de r23690

Fix #3981 : permettre aux rédacteurs de créer une nouvelle traduction

Suite aux modifications relatives à #2746 un rédacteur ne pouvait plus créer de traduction sur un article dont il n'était pas l'auteur, on corrige ce bug.

Historique

#1 Mis à jour par Jacques Bouthier il y a 4 mois

Il s'agit bien d'un problème d'affichage parce qu'en mettant les bonnes infos dans l'url on créé l'article, par exemple
http://monsite.tld/ecrire/?exec=article_edit&new=oui&lier_trad=40&id_rubrique=3
et il est bien possible de créer un article en lien avec l'article initial.

#2 Mis à jour par Franck D il y a 4 mois

Hello :-) , je viens de faire un test en 3.2 et le lien pour traduire un article est bien présent, mais par contre, il faut avant avoir cocher la case dans "ecrire/?exec=configurer_multilinguisme" pour que les auteurs puissent avoir cette possibilité
Pour spip.net il n'y a que b_b qui puisse voir si l'option est bien validé, mais regarde déjà, sur ton site perso si la case est bien coché "gérer les liens de traduction"
Franck

#3 Mis à jour par Franck D il y a 4 mois

Bon alors, j'ai pas fait les tests correctement...
En faite, le lien pour traduire un article n'est présent concernant les "rédacteurs" que concernant leur propre articles !
Ce qui fait qu'il ne peuvent traduire les articles des autres rédacteurs !

Par contre, une personne avec le statut d'admi peux traduire les articles des autres (logique)

Donc,logiquement, il faudrait que les personnes ayant le statut de rédacteur puissent traduire et proposé leur traduction à la publication concernant les articles des autres.

#4 Mis à jour par jluc - il y a 4 mois

Il semble que la création utilise https://core.spip.net/projects/spip/repository/revisions/22643
où c'est l'autorisation 'changertraduction' qui est utilisée,

Celle ci est définie dans inc/autoriser comme étant par défaut équivalente à l'autorisation pour 'modifier'.
Ça explique le problème car n'importe qui ne peut pas modifier n'importe quoi.

Il faudrait créer une autorisation 'creertraduction' et une fonction 'autoriser_creertraduction_dist' qui soit plus tolérante, et l'utiliser au bon endroit, au lieu de devoir utiliser 'changertraduction'

Mais je sais pas si ce bon endroit c'est le source du commit 22643 pointé plus haut, car je ne sais pas si seule la création y passe, ou si la modification y passe aussi...

#5 Mis à jour par jluc - il y a 4 mois

"changertraduction" a t il un sens possible ? les traductions sont des articles comme les autres...
La modification d'une traduction ne se fait elle pas comme la modif d'un article normal ?

#6 Mis à jour par Jacques Bouthier il y a 4 mois

En tout cas je constate que si je remplace la ligne 83 du fichier actuel

if (isset($valeurs['id_trad']) and $valeurs['_traduisible']) {

par

if (isset($valeurs['id_trad'])) {

Alors le menu apparait de nouveau pour les rédacteurs... Je ne sais pas quelles en sont les conséquences... mais voilà une bonne piste :)

#7 Mis à jour par b b il y a 4 mois

Hop, voici le patch que je propose, à tester :)

#8 Mis à jour par b b il y a 4 mois

  • Statut changé de Nouveau à En cours
  • Priorité changé de Normal à Haut
  • Version cible changé de 3.1 à 3.2

Priorité haute amha, et on cible sur la branche 3.2 pour que ça soit intégré avant la release (il ne faudra pas oublier de reporter en 3.1 bien sûr).

#9 Mis à jour par Jacques Bouthier il y a 4 mois

Je viens de tester le diff en 3.2 . Ca fonctionne. Un rédacteur peut maintenant proposer un article.
(il faudrait l'appliquer rapidement sur spip.net... avant que trop de rédacteurs ne renoncent ^^)

Je vais essayer en 3.1.

#10 Mis à jour par Anonyme il y a 4 mois

  • Statut changé de En cours à Fermé
  • % réalisé changé de 0 à 100

Appliqué par commit r23690.

#11 Mis à jour par b b il y a 4 mois

  • Resolution mis à fixed

Intégré en 3.2 et reporté en 3.1, spip.net est à jour, merci pour le retour :)

#12 Mis à jour par Jacques Bouthier il y a environ un mois

Il reste encore un souci dont je me suis rendu compte hier soir : lorsqu'on est connecté en tant que rédacteur sur spip.net on ne voit pas la liste des traductions existantes.
Du coup il est possible de commencer une traduction et se rendre compte en validant son article qu'il existe déjà une trad. Sur son propre article on voit bien les traductions déjà existantes.

#13 Mis à jour par b b il y a environ un mois

@Jacques amha tu devrais créer un ticket spécifique pour ce nouveau bug afin qu'on n'oublie pas de s'en occuper :)

Formats disponibles : Atom PDF

Ajouter une image à partir du presse-papier (Taille maximale: 1,25 Mo)