Hello,
J'ai commencé à écrire les tests pour les modèles de données des journaux.
J'ai trouvé quelque chose d'assez intéressant, mais je ne sais pas si c'est une erreur ou non.
Actuellement, quand le staff convertit un journal en dépêche, la dépêche est préparée sous licence CC-By-SA quelque soit la licence du journal d'origine.
Ça me semble pas très correct pour l'auteur du journal qui a pris la peine d'enlever la coche "Je publie ce journal sous licence CC-BY-SA", non ?
Personnellement, je m'attendais à ce que la dépêche crée à partir du journal suive toujours la licence d'origine du journal. J'appliquerai donc un correctif comme ça:
def convert
@news = News.new title: title,
wiki_body: "**TODO** insérer une synthèse du journal",
wiki_second_part: wiki_body,
section_id: Section.default.id
@news.author_name = owner.try(:name)
@news.author_email = owner.try(:account).try(:email)
@news.transaction do
@news.save!
$redis.set "convert/#{@news.id}", self.id
- @news.node.update_column(:cc_licensed, true) if node.cc_licensed?
+ @news.node.update_column(:cc_licensed, node.cc_licensed?)
@news.links.create title: "Journal à l’origine de la dépêche", url: "https://#{MY_DOMAIN}/users/#{owner.to_param}/journaux/#{to_param}", lang: "fr"
@news.submit! unless node.cc_licensed?
@news
end
end
On remarquera d'ailleurs dans ce code, que, si le journal est sous licence CC-BY-SA, la dépêche est crée en rédaction et, si non, elle est directement mise en modération.
# Un peu d'histoire
Posté par Adrien Dorsaz (site web personnel, Mastodon) . Évalué à 2 (+0/-0). Dernière modification le 25 septembre 2021 à 22:25.
Par curiosité, j'ai cherché avec
git blame
la source de la ligne de code qui change la licence (cité dans l'entrée de suivi plus haut; merci àvim-fugitive
pour l'aide à la navigation avec leblame
).Je m'attendais à ce que, dans toute l'histoire, un malheureux commit aie inversé la valeur de cette ligne (soit en changeant le
unless
enif
ou quelque chose du genre).En réalité cette ligne a toujours été écrite ainsi (cf le commit d'origin, en mars 2011), et du coup, j'étais un peu déçu.
En réfléchissant un peu, je me suis dit que, en fait, si je relis en français cette ligne, elle dit
met "vrai" pour le choix de la licence CC-BY-SA si c'était le cas sur le journal
. Et donc, ça supposait, que le choix par défaut de la licence étaitfalse
à l'époque et que, il fallait donc bien passer àtrue
uniquement si le journal avait la licence CC-BY-SA.Une très rapide recherche dans les migrations SQL, m'a confirmé que, lorsque l'on a crée ce code (en mars 2011), la valeur de
cc_licensed
par défaut étaitfalse
et qu'elle a bougé àtrue
en décembre 2012:L'intention du code correspondait donc bien en 2011 à ce que je proposais dans le suivi, mais il n'a pas survécu au changement de valeur par défaut 1 année plus tard. Je vais proposer un correctif, maintenant que l'histoire m'a confirmé mon intuition :)
[^] # Re: Un peu d'histoire
Posté par Adrien Dorsaz (site web personnel, Mastodon) . Évalué à 2 (+0/-0).
Et voilà le correctif dont je parlais: Pull Request 318.
Dans ce pull request, j'ajoute également les tests que j'ai écris actuellement pour le modèle des journaux, puisque c'est eux qui m'ont aidé à trouver ce bug.
[^] # Re: Un peu d'histoire
Posté par Benoît Sibaud (site web personnel) . Évalué à 6 (+0/-0).
L'idée était que les dépêches en rédaction sont prévues pour être collaboratives, donc sous une licence libre, histoire que plusieurs personnes puissent intervenir dessus. Si l'auteur d'un journal a décoché la licence par défaut, alors son journal peut être sous une licence non-libre ou sous une autre licence libre. Du coup si on créé une dépêche basée sur ce journal, on n'a pas trop de garantie sur la licence, sur le fait qu'on peut laisser d'autres gens y contribuer ou même sur le fait qu'on peut éditer. Je serais d'avis de ne simplement pas permettre d'en faire une dépêche.
Raisons :
- si un auteur veut une licence non-libre, alors pas de dépêche collaborative
- si un auteur veut une autre licence que la CC By Sa 4.0 :
- il peut placer aussi son journal sous une autre licence libre de son choix, en laissant la case cochée
- s'il ne veut vraiment pas de la CC By Sa 4.0 (parce qu'il exècre cette licence ou qu'il n'en a pas le droit parce qu'il réutilise un contenu lui-même sous une autre licence par exemple), le cas me semble suffisamment rare pour ne pas se compliquer la vie et ne pas permettre d'en faire une dépêche (et si l'auteur veut une dépêche, il la soumet lui même en décochant la case).
La pire situation restant les gens qui cochent la case initialement, et qui après demandent à ce qu'elle soit décochée, ce qui posent problème vis-à-vis d'éventuels contributeurs.
[^] # Re: Un peu d'histoire
Posté par Adrien Dorsaz (site web personnel, Mastodon) . Évalué à 3 (+0/-0).
Merci pour le retour, j'ai ajouté un commit au PR pour refuser de proposer en dépêche un journal qui n'a pas coché la license CC-BY-SA 4.0.
C'est effectivement plus simple, puisque rien n'est sûr si la case n'a pas été cochée.
[^] # Re: Un peu d'histoire
Posté par Benoît Sibaud (site web personnel) . Évalué à 4 (+0/-0).
Fusionné et déployé https://github.com/linuxfrorg/linuxfr.org/commit/368abcf2ab3f998ca0c13b01fc628d7cec742c52
Merci.
Envoyer un commentaire
Suivre le flux des commentaires
Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.