From e1902354b5d97cb2cbb36a3c6a00090d60dac753 Mon Sep 17 00:00:00 2001
From: Leif Johansson <leifj@sunet.se>
Date: Thu, 5 Aug 2010 10:19:08 +0200
Subject: form ui cleanup + remove links with names

---
 coip/apps/name/forms.py |  3 +--
 coip/apps/name/views.py | 10 +++++-----
 templates/base.html     |  2 +-
 templates/edit.html     |  2 +-
 4 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/coip/apps/name/forms.py b/coip/apps/name/forms.py
index a68102e..9dd03a7 100644
--- a/coip/apps/name/forms.py
+++ b/coip/apps/name/forms.py
@@ -29,8 +29,7 @@ class NewNameForm(forms.ModelForm):
         fields = ['type','value','short','description']
         
 class NameDeleteForm(forms.Form):
-    recursive = BooleanField(label="Also delete all nodes below this node?",required=False)
-    confirm = BooleanField(label="Confirm")
+    recursive = BooleanField(label="Also delete everything below this name?",required=False)
     
 class NameLinkForm(forms.ModelForm):
     class Meta:
diff --git a/coip/apps/name/views.py b/coip/apps/name/views.py
index de5c3af..bde9954 100644
--- a/coip/apps/name/views.py
+++ b/coip/apps/name/views.py
@@ -27,18 +27,18 @@ def delete(request,id):
     if request.method == 'POST':
         form = NameDeleteForm(request.POST)
         if form.is_valid():
-            if not form.cleaned_data['confirm']:
-                return HttpResponseRedirect("/name/id/%d" % name.id)
-                
             parent = name.parent
             if not form.cleaned_data['recursive'] and name.children.count() > 0:
                 return HttpResponseForbidden("Will not delete non-empty node")
             
+            for link in name.links:
+                link.delete()
+            
             if form.cleaned_data['recursive']:
                 name.remove(True)
             else:
                 name.remove(False)
-                
+            
             if parent:
                 return HttpResponseRedirect("/name/id/%d" % parent.id)
             else:
@@ -46,7 +46,7 @@ def delete(request,id):
     else:
         form = NameDeleteForm()
             
-    return respond_to(request,{'text/html': 'apps/name/edit.html'},{'form': form,'name': name,'formtitle': 'Remove name confirmation' ,'submitname': 'Delete'})
+    return respond_to(request,{'text/html': 'apps/name/edit.html'},{'form': form,'name': name,'formtitle': 'Remove %s' % (name.short) ,'submitname': 'Delete'})
 
 @login_required
 def add(request,id):
diff --git a/templates/base.html b/templates/base.html
index 78af32d..b857c03 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -77,7 +77,7 @@
 		   #content{ }
 		   #main{ font: 100% "Trebuchet MS", sans-serif; width:95%; }
 		   .code{font: 110% "Courier", fixed}
-		   .infopanel { padding-left: 10px; }
+		   .infopanel { padding-left: 10px; padding-bottom: 10px;}
 		   .infopanel label { display:block; }
 		   div .hidden {visibility:hidden;}
 		   div .ms2side__div {width: 300%;}
diff --git a/templates/edit.html b/templates/edit.html
index e29ec7c..214af95 100644
--- a/templates/edit.html
+++ b/templates/edit.html
@@ -6,7 +6,7 @@
 {% block content %}
 	<form method="POST">
 		<div class="ui-widget-content ui-corner-all infopanel">
-		    <h3>{{formtitle}}</h3>
+		    <h1>{{formtitle}}</h1>
 			<table>
 		      {% for field in form %}
 		        <tr>
-- 
cgit v1.1