From 954178c4b1639bd7362ca9f78d3339c6f1c99f2e Mon Sep 17 00:00:00 2001
From: Nazar Kotjuk <kotzillla@gmail.com>
Date: Thu, 21 Nov 2013 10:00:53 +0200
Subject: [PATCH] fix bug with selects

---
 proj/settings.py               |  4 +--
 proj/views.py                  | 52 ++++++++++++++++++++---------
 static/custom_js/select_tag.js | 60 +++++++++++++++++++++++++++-------
 3 files changed, 87 insertions(+), 29 deletions(-)

diff --git a/proj/settings.py b/proj/settings.py
index 64c23040..f552c124 100644
--- a/proj/settings.py
+++ b/proj/settings.py
@@ -129,13 +129,13 @@ TEMPLATE_LOADERS = (
 )
 
 MIDDLEWARE_CLASSES = (
-    'django.middleware.cache.UpdateCacheMiddleware',
+#    'django.middleware.cache.UpdateCacheMiddleware',
     'django.middleware.common.CommonMiddleware',
     'django.contrib.sessions.middleware.SessionMiddleware',
     'django.middleware.csrf.CsrfViewMiddleware',
     'django.contrib.auth.middleware.AuthenticationMiddleware',
     'django.contrib.messages.middleware.MessageMiddleware',
-    'django.middleware.cache.FetchFromCacheMiddleware',
+#    'django.middleware.cache.FetchFromCacheMiddleware',
     # Uncomment the next line for simple clickjacking protection:
     # 'django.middleware.clickjacking.XFrameOptionsMiddleware',
     #'debug_toolbar.middleware.DebugToolbarMiddleware',#должно быть последним полем
diff --git a/proj/views.py b/proj/views.py
index 6830baac..fa026682 100644
--- a/proj/views.py
+++ b/proj/views.py
@@ -45,23 +45,40 @@ def settings(request):
 
 
 from directories.models import Currency
+from country.models import Country
+from country.forms import CountryForm
+from file.forms import FileModelForm
 from django.views.decorators.cache import cache_page
 
-from django.core.cache import cache
+#from django.core.cache import cache
 
 #@cache_page(30)
 def test(request):
     ids = [i for i in range(1, 5000)]
+    form = CountryForm()
+    file_form = FileModelForm
+    c = Country.objects.get(id=5)
+    country_id = getattr(c, 'id')
+
+    #args = {}
+    for i in range(1,10000):
+        args = {'languages': LANGUAGES, 'form': form, 'file_form': file_form, 'obj_id': country_id, 'files':FileModel.objects.filter(content_type=ContentType.objects.get_for_model(c), object_id=getattr(c, 'id'))}
+        #args['languages'] = LANGUAGES
+        #args['form'] = form
+        #args['file_form'] = file_form
+        #args['files'] = FileModel.objects.filter(content_type=ContentType.objects.get_for_model(c), object_id=getattr(c, 'id'))
+        #args['obj_id'] = country_id
 
 
-    currs = [item for item in Currency.objects.all()]
-    currs = []
-    for i in ids:
-        currs.append(Currency.objects.get(id=i))
 
-    cache.set('key', currs[100].currency, 60)
+    #currs = [item for item in Currency.objects.all()]
+    #currs = []
+    #for i in ids:
+    #    currs.append(Currency.objects.get(id=i))
 
-    return HttpResponse('success')
+    #cache.set('key', currs[100].currency, 60)
+
+    return HttpResponse(args)
 
 
 
@@ -77,15 +94,20 @@ def ajax_city(request):
 def ajax_tag(request):
     """
 
-    returns html <option> tags filled with tags filtered by theme values from request
+    return array of tags (id, name) filtered by theme values from request
 
     """
-    try:
-        data = request.GET['id'].replace('theme=', '').split('&')
-        objects = Tag.objects.filter(theme__in=data)
-        return render_to_response('select.html', {'objects': objects})
-    except:
-        return HttpResponse('error')
+    if request.GET:
+        ids = request.GET.get('id')
+        if ids:
+            #get id values in list
+            data = ids.replace('theme=', '').split('&')
+            #generate list of tags
+            tags = ['[%d, "%s"]'%(int(item.id), item.name) for item in Tag.objects.filter(theme__in=data)]
+            return HttpResponse('[%s]'%', '.join(tags), content_type='application/json')
+        else:
+            #request empty - send empty array
+            return HttpResponse('[]', content_type='application/json')
 
 
 from django.core import serializers
@@ -108,7 +130,7 @@ def ajax_post_file(request, obj_id=None):
                 #initial model object
                 obj = Model.objects.get(id=obj_id)
                 file = file_form.save(request.FILES, obj)
-                data = serializers.serialize('json', [file])
+                #data = serializers.serialize('json', [file])
                 #return HttpResponse(data)
                 files = FileModel.objects.filter(content_type=ContentType.objects.get_for_model(obj),object_id=getattr(obj, 'id'))
             else:
diff --git a/static/custom_js/select_tag.js b/static/custom_js/select_tag.js
index c5f7b3b8..4d14c68a 100644
--- a/static/custom_js/select_tag.js
+++ b/static/custom_js/select_tag.js
@@ -1,15 +1,51 @@
 $(document).ready(function(){
             $('#id_theme').change(function(){
                 $.get(
-                        "/admin/ajax_tag/", {'id': $(this).serialize()}, function(j){
-                            if (j!='error'){
-                                $('#id_tag').html(j);
-                                $('#id_tag').attr('disabled', false);
-                            }
-                            else {
-                                //delete all options
-                                $('#id_tag').find('option').remove();
-                            }
-                        });
-            });
-        });
\ No newline at end of file
+                        "/admin/ajax_tag/", {'id': $(this).serialize()}, function(data){
+                                var optionValues = [];
+                                var getValues = [];
+                                var selectedValues = []
+                                //push values sended from server in array
+                                $.each(data, function(i, elem){
+                                    getValues.push(elem[0].toString())
+                                });
+                                //delete options if they aren't in getvalues
+                                //otherwise push it in array
+                                //also push in array already selected values
+                                $('#id_tag option').each(function() {
+                                    var check = $.inArray($(this), getValues);
+                                    if ($(this).is(':selected') ){
+                                        selectedValues.push($(this).val())
+                                    }
+                                    if (check == -1){
+                                        $(this).remove()
+                                    }
+                                    else{
+                                        optionValues.push($(this).val());
+                                    }
+                                });
+                                //generate new options
+                                //old options unchanged
+                                var  html = ''
+                                $.each(data, function(i, elem){
+                                    var check = $.inArray(elem[0].toString(), optionValues);
+
+                                    if (check == -1){
+                                       html += '<option value="';
+                                       html += elem[0];
+                                       html += '">';
+                                       html += elem[1];
+                                       html += '</option>';
+                                    }
+                                });
+                                $('#id_tag').append(html);
+                                //select previous selected values
+                                $('#id_tag option').each(function() {
+                                    var check = $.inArray($(this).val(), selectedValues)
+                                    if (check != -1){
+                                        $(this).attr('selected', 'selected');
+                                    }
+                                });
+                        });//end get
+            });//end change
+        });//end ready
\ No newline at end of file