Projet

Général

Profil

Bug #18426

hobo : gestion complète des thèmes

Ajouté par Frédéric Péters il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 septembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour le moment il y a un raccourci qui fait que publik-base est attribué partout, et au-delà de ça le code marche vraiment uniquement pour ce qui est dans publik-base-theme, ne prend pas en compte les overlay et du coup ne marcherait pas pour les thèmes imio ou sictiam.


Fichiers

Révisions associées

Révision 22a8be9a (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

hobo: improve theme handling with full support for overlays (#18426)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 6 ans

Il y a encore une référence à publik-base dans le code mais celle-là est encore nécessaire pour le moment (par exemple à Vincennes c'est vincennes-2014 déclaré de manière globale mais bien publik-base renseigné dans w.c.s.).

#2

Mis à jour par Frédéric Péters il y a plus de 6 ans

Avec le fichier qui déclare les thèmes utilisés pour les tests.

#4

Mis à jour par Thomas Noël il y a plus de 6 ans

Dans le code de hobo (hobo/agent/common/management/commands/hobo_deploy.py), la gestion du os.unlink m'a l'air assez différente ; dans hobo_deploy on fait des rmdir si c'est pas un lien mais un répertoire, et on tente un unlink si y'a pas d'overlay défini (et si le unlink rate, c'est pas grave). Je me demande si ça permet pas de laisser les "static" et les "templates" tranquilles s'ils existent parce que quelqu'un veut jouer avec.

Bon j'avoue, ça me semble être ok, mais je suis plutôt paumé à tenter de comparer le code de hobo_deploy avec ça (ça n'a sans doute pas vraiment de sens). Si y'a un autre relecteur, sinon c'est quand même un "ack" de ma part.

#5

Mis à jour par Frédéric Péters il y a plus de 6 ans

Yep, le code dans Hobo il a été fait pour prendre en compte des situations qui avaient été gérées manuellement avant. (ce code-ci je l'ai écrit sans y regarder)

#6

Mis à jour par Thomas Noël il y a plus de 6 ans

Ack (ne considérons plus correctes les situations manuelles)

#7

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 22a8be9acf8c99487815b5daa6ae558071d0f551
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Sep 4 15:33:54 2017 +0200

    hobo: improve theme handling with full support for overlays (#18426)
#8

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF