Produits Entr'ouvert » Combo

Il reste un "modifier" en haut à droite, un peu perturbant pour l'instant. Qu'en faire ?

Par ailleurs je ne mettrais pas le titre "Options" sur la colonne, mais plutôt "Paramètres" (qui n'ont rien d'optionnels)

Il reste un "modifier" en haut à droite, un peu perturbant pour l'instant. Qu'en faire ?

Il reprend pile les champs présents dans la boite "ajouter", cette symétrie ajouter/modifier me va bien pour le moment.

Ok pour "options" -> "paramétrage".

Frédéric Péters a écrit :

Il reste un "modifier" en haut à droite, un peu perturbant pour l'instant. Qu'en faire ?

Il reprend pile les champs présents dans la boite "ajouter", cette symétrie ajouter/modifier me va bien pour le moment.

Oui pour son utilité, c'est son emplacement que je trouve maintenant un peu étrange, puisqu'il s'agit de paramètres de la page, que j'aurais bien vus affichés comme les autres (slug, présence dans les menus, etc). Mais on peut y aller comme cela (avec les modifs que tu proposes ci-dessus c'est déjà super mieux qu'avant).

Un bogue :

diff --git a/combo/manager/forms.py b/combo/manager/forms.py
index 063baa7..26b492c 100644
--- a/combo/manager/forms.py
+++ b/combo/manager/forms.py
@@ -55,7 +55,7 @@ class PageSelectTemplateForm(forms.ModelForm):
         fields = ('template_name',)

     def __init__(self, *args, **kwargs):
-        super(PageForm, self).__init__(*args, **kwargs)
+        super(PageSelectTemplateForm, self).__init__(*args, **kwargs)
         templates = [(x[0], x[1]['name']) for x in settings.COMBO_PUBLIC_TEMPLATES.items()]
         templates = [x for x in templates if self.template_exists(x[0])]
         templates.sort(lambda x, y: cmp(x[1], y[1]))

Ensuite, ptit soucis : quand on clique sur "Template" le select ne selectionne pas la valeur actuelle (pas encore compris pourquoi, flemme de comprendre forms.Select)

En fait la page est créée avec un template «two-columns-sidebar» qui n'est plus visible ensuite.

Le test qui montre le soucis (qui plante actuellement) :

diff --git a/tests/test_manager.py b/tests/test_manager.py
index 4995953..108fb66 100644
--- a/tests/test_manager.py
+++ b/tests/test_manager.py
@@ -83,6 +83,7 @@ def test_edit_page(app, admin_user):
     assert Page.objects.all()[0].slug == 'foobar'
     # template name
     resp = resp.click(href='.*/template')
+    assert Page.objects.all()[0].template_name == 'standard'
     assert resp.form['template_name'].value == 'standard'
     resp.form['template_name'].value = 'standard-sidebar'
     resp = resp.form.submit()

C'est lié au template_exists qui ne considère pas les templates qui n'en sont pas vraiment ; filtrage qui n'est pas fait dans Page.save()

ok, merci je regarde.

Ack !