From: yomguy <yomguy@parisson.com>
Date: Thu, 1 Dec 2011 09:37:18 +0000 (+0100)
Subject: fix collection add, item copy and add
X-Git-Tag: 1.3~6
X-Git-Url: https://git.parisson.com/?a=commitdiff_plain;h=e81af8b48bc2a97304187c0e6bb52799b0f9b78a;p=telemeta.git

fix collection add, item copy and add
---

diff --git a/telemeta/templates/telemeta_default/collection_add.html b/telemeta/templates/telemeta_default/collection_add.html
index 27d0296c..abbda5b7 100644
--- a/telemeta/templates/telemeta_default/collection_add.html
+++ b/telemeta/templates/telemeta_default/collection_add.html
@@ -16,12 +16,14 @@
         <table>
         <tr><td colspan="2">{% for error in form.non_field_errors %}<li class="error">{{ error }}</li>{% endfor %}</td></tr>
         {% for field in form %}
-            {% if not field.html_name == "copied_from_item" and not field.html_name == "doctype_code" %}
-            <tr>
-                <tr><td class="error">{{ field.errors }}</td></tr>
-                <td>{{ field.label_tag }}:</td><td> {{ field }}</td>
-            </tr>
-            {% endif %}
+         <tr>
+        {% if field.html_name == "copied_from_item" or field.html_name == "doctype_code" %}
+            <td>{{ field.label_tag.as_hidden }}</td><td>{{ field.as_hidden }}</td>
+        {% else %}
+            <tr><td class="error">{{ field.errors }}</td></tr>
+            <td>{{ field.label_tag }}:</td><td> {{ field }}</td>
+        {% endif %}
+        </tr>
         {% endfor %}
         </table>
         <div align="center" style="margin-top:3ex;">
diff --git a/telemeta/templates/telemeta_default/mediaitem_add.html b/telemeta/templates/telemeta_default/mediaitem_add.html
index 31aefe87..ee05ac19 100644
--- a/telemeta/templates/telemeta_default/mediaitem_add.html
+++ b/telemeta/templates/telemeta_default/mediaitem_add.html
@@ -22,14 +22,23 @@
      <form method="post" id="_addItemForm" action="" enctype="multipart/form-data">{% csrf_token %}
        <ul>{% for error in form.non_field_errors %}<li class="error">{{ error }}</li>{% endfor %}</ul>
        <table>
-       {% for field in form %}
-        {% if not field.html_name == "copied_from_item" %}
-        <tr>
-            <tr><td class="error">{{ field.errors }}</td></tr>
-            <td>{{ field.label_tag }}:</td><td> {{ field }}</td>
-        </tr>
-        {% endif %}
-       {% endfor %}
+        {% for field in form %}
+            <tr>
+            {% if field.html_name == "copied_from_item" %}
+                <td>{{ field.label_tag.as_hidden }}{{ field.as_hidden }}</td>
+            {% else %}
+                <tr><td class="error">{{ field.errors }}</td></tr>
+                <td>{{ field.label_tag }}:</td>
+                    {% if field.html_name == "collection" %}
+                    <td>&nbsp;{% trans "Title" %} : {{ item.collection.title }}<br />
+                        &nbsp;{% trans "Code" %} : {{ item.collection.code }}<br />
+                        {{ field }}</td>
+                    {% else %}
+                    <td>{{ field }}</td>
+                    {% endif %}
+            {% endif %}
+            </tr>
+    {% endfor %}
        </table>
        <div align="center" style="margin-top:3ex;">
         <a href="{% url telemeta-items %}"
diff --git a/telemeta/templates/telemeta_default/mediaitem_copy.html b/telemeta/templates/telemeta_default/mediaitem_copy.html
index 3a0c060b..64bbaac7 100644
--- a/telemeta/templates/telemeta_default/mediaitem_copy.html
+++ b/telemeta/templates/telemeta_default/mediaitem_copy.html
@@ -24,19 +24,21 @@
      <ul>{% for error in form.non_field_errors %}<li class="error">{{ error }}</li>{% endfor %}</ul>
       <table>
        {% for field in form %}
-        {% if not field.html_name == "copied_from_item" %}
-        <tr>
-            <tr><td class="error">{{ field.errors }}</td></tr>
-            <td>{{ field.label_tag }}:</td>
-             {% if field.html_name == "collection" %}
-              <td>&nbsp;{% trans "Title" %} : {{ item.collection.title }}<br />
-                  &nbsp;{% trans "Code" %} : {{ item.collection.code }}<br />
-                  {{ field }}</td>    
-             {% else %}
-              <td>{{ field }}</td>
-             {% endif %}
-        </tr>
-        {% endif %}
+            <tr>
+            {% if field.html_name == "copied_from_item" %}
+                <td>{{ field.label_tag.as_hidden }}{{ field.as_hidden }}</td>
+            {% else %}
+                <tr><td class="error">{{ field.errors }}</td></tr>
+                <td>{{ field.label_tag }}:</td>
+                    {% if field.html_name == "collection" %}
+                    <td>&nbsp;{% trans "Title" %} : {{ item.collection.title }}<br />
+                        &nbsp;{% trans "Code" %} : {{ item.collection.code }}<br />
+                        {{ field }}</td>
+                    {% else %}
+                    <td>{{ field }}</td>
+                    {% endif %}
+            {% endif %}
+            </tr>
        {% endfor %}
        </table>
        <div align="center" style="margin-top:3ex;">
diff --git a/telemeta/web/base.py b/telemeta/web/base.py
index 8ce79687..f040a291 100644
--- a/telemeta/web/base.py
+++ b/telemeta/web/base.py
@@ -417,14 +417,14 @@ class CollectionView(object):
     @method_decorator(permission_required('telemeta.add_mediacollection'))
     def collection_copy(self, request, public_id, template='telemeta/collection_edit.html'):
         if request.method == 'POST':
-            new_collection = MediaCollection()
-            form = MediaCollectionForm(data=request.POST, files=request.FILES, instance=new_collection)
+            collection = MediaCollection()
+            form = MediaCollectionForm(data=request.POST, files=request.FILES, instance=collection)
             if form.is_valid():
                 code = form.cleaned_data['code']
                 if not code:
                     code = public_id
                 form.save()
-                new_collection.set_revision(request.user)
+                collection.set_revision(request.user)
                 return HttpResponseRedirect('/collections/'+code)
         else:
             collection = MediaCollection.objects.get(public_id=public_id)
@@ -678,14 +678,14 @@ class ItemView(object):
     def item_copy(self, request, public_id, template='telemeta/mediaitem_copy.html'):
         """Copy a given item"""        
         if request.method == 'POST':
-            new_item = MediaItem()
-            form = MediaItemForm(data=request.POST, files=request.FILES, instance=new_item)
+            item = MediaItem()
+            form = MediaItemForm(data=request.POST, files=request.FILES, instance=item)
             if form.is_valid():
                 form.save()
                 code = form.cleaned_data['code']
                 if not code:
-                    code = str(new_item.id)
-                new_item.set_revision(request.user)
+                    code = str(item.id)
+                item.set_revision(request.user)
                 return HttpResponseRedirect('/items/'+code)
         else:
             item = MediaItem.objects.get(public_id=public_id)