From 662a8a111721df714e9c95f94db91d2d03d7b19d Mon Sep 17 00:00:00 2001
From: =?utf8?q?=C5=81ukasz=20Rekucki?= <lrekucki@gmail.com>
Date: Sun, 14 Mar 2010 18:22:35 +0100
Subject: [PATCH] * Improved history view (still not there). * Added debug
 toolbar, to apps. * Fewer queries for toolbar (which should be cached).

* Store "last visited" in user's session. Closes #354.
---
 .../templates/compress/.tmp_css.html.32854~   |  1 -
 .../templates/compress/.tmp_js.html.60841~    |  1 -
 apps/toolbar/templates/toolbar/button.html    |  2 +-
 apps/toolbar/templatetags/toolbar_tags.py     |  2 +-
 apps/wiki/nice_diff.py                        | 26 +++++
 apps/wiki/templates/wiki/diff_table.html      | 20 ++++
 apps/wiki/templates/wiki/document_list.html   | 31 +++---
 apps/wiki/views.py                            | 39 ++++----
 platforma/settings.py                         | 12 ++-
 platforma/static/css/filelist.css             | 38 +++++---
 platforma/static/css/master.css               | 95 +++++++++++++------
 platforma/static/js/main.js                   |  6 +-
 platforma/templates/base.html                 |  2 +
 requirements.txt                              |  2 +
 scripts/nice_diff.py                          | 57 -----------
 15 files changed, 196 insertions(+), 138 deletions(-)
 delete mode 100644 apps/compress/templates/compress/.tmp_css.html.32854~
 delete mode 100644 apps/compress/templates/compress/.tmp_js.html.60841~
 create mode 100755 apps/wiki/nice_diff.py
 create mode 100644 apps/wiki/templates/wiki/diff_table.html
 delete mode 100644 scripts/nice_diff.py

diff --git a/apps/compress/templates/compress/.tmp_css.html.32854~ b/apps/compress/templates/compress/.tmp_css.html.32854~
deleted file mode 100644
index 3751a4a0..00000000
--- a/apps/compress/templates/compress/.tmp_css.html.32854~
+++ /dev/null
@@ -1 +0,0 @@
-<link href="{{ url }}" rel="stylesheet" type="text/css"{% if media %} media="{{ media }}"{% endif %}{% if title %} title="{{ title|default:"all" }}"{% endif %}{% if charset %} charset="{{ charset }}"{% endif %} />
\ No newline at end of file
diff --git a/apps/compress/templates/compress/.tmp_js.html.60841~ b/apps/compress/templates/compress/.tmp_js.html.60841~
deleted file mode 100644
index 8419c20a..00000000
--- a/apps/compress/templates/compress/.tmp_js.html.60841~
+++ /dev/null
@@ -1 +0,0 @@
-<script type="text/javascript" src="{{ url }}" charset="utf-8"></script>
\ No newline at end of file
diff --git a/apps/toolbar/templates/toolbar/button.html b/apps/toolbar/templates/toolbar/button.html
index 25a7c429..8335918a 100644
--- a/apps/toolbar/templates/toolbar/button.html
+++ b/apps/toolbar/templates/toolbar/button.html
@@ -2,7 +2,7 @@
 <a href="{{button.link}}" target="_new">
 {% endif %}
 <button type="button"
-        ui:action="{{ button.scriptlet.name }}"
+        ui:action="{{ button.scriptlet_id }}"
         ui:action-params="{{ button.params|escape }}"
         {% if button.key %}ui:hotkey="{{ button.hotkey_code }}"{% endif %}
         {% if button.tooltip %}title="{{ button.full_tooltip }}"{% endif %} >
diff --git a/apps/toolbar/templatetags/toolbar_tags.py b/apps/toolbar/templatetags/toolbar_tags.py
index 14d3679d..ac7ebbb1 100644
--- a/apps/toolbar/templatetags/toolbar_tags.py
+++ b/apps/toolbar/templatetags/toolbar_tags.py
@@ -5,7 +5,7 @@ register = template.Library()
 
 @register.inclusion_tag('toolbar/toolbar.html')
 def toolbar():
-    return {'toolbar_groups': models.ButtonGroup.objects.all()}
+    return {'toolbar_groups': models.ButtonGroup.objects.all().select_related() }
 
 @register.inclusion_tag('toolbar/button.html')
 def toolbar_button(b):
diff --git a/apps/wiki/nice_diff.py b/apps/wiki/nice_diff.py
new file mode 100755
index 00000000..612fa22a
--- /dev/null
+++ b/apps/wiki/nice_diff.py
@@ -0,0 +1,26 @@
+#!/usr/bin/env python
+
+import difflib
+import re
+
+from django.template.loader import render_to_string
+from django.utils.html import escape as html_escape
+
+DIFF_RE = re.compile(r"""\x00([+^-])""" ,re.UNICODE)
+NAMES = { '+': 'added', '-': 'removed', '^': 'changed' }
+
+def diff_replace(match):
+	return """<span class="diff_mark diff_mark_%s">""" %  NAMES[match.group(1)]
+
+def filter_line(line):
+	return  DIFF_RE.sub(diff_replace, html_escape(line)).replace('\x01', '</span>')
+
+def html_diff_table(la, lb):
+	return render_to_string("wiki/diff_table.html", {
+		"changes": [(a[0],filter_line(a[1]),
+					 b[0],filter_line(b[1]),
+					 change) for a, b, change in difflib._mdiff(la, lb)]
+	}); 
+										
+	
+__all__ = ['html_diff_table']
\ No newline at end of file
diff --git a/apps/wiki/templates/wiki/diff_table.html b/apps/wiki/templates/wiki/diff_table.html
new file mode 100644
index 00000000..91ac55c0
--- /dev/null
+++ b/apps/wiki/templates/wiki/diff_table.html
@@ -0,0 +1,20 @@
+<table class="diff_table">
+	<thead>
+		<tr>
+			<th colspan="2">Stara wersja</th>
+			<th colspan="2">Nowa wersja</th>
+		</tr>
+	</thead>
+<tbody>
+{% for an, a, bn, b, has_change in changes %}
+
+<tr class="{% if has_change %}change{% endif %}">
+<td>{{an}}</td>
+<td class="left">{{ a|safe }}&nbsp;</td>
+<td>{{bn}}</td>
+<td class="right">{{ b|safe }}&nbsp;</td>
+</tr>
+
+{% endfor %}
+</tbody>
+</table>
\ No newline at end of file
diff --git a/apps/wiki/templates/wiki/document_list.html b/apps/wiki/templates/wiki/document_list.html
index 183d6128..1b0662b3 100644
--- a/apps/wiki/templates/wiki/document_list.html
+++ b/apps/wiki/templates/wiki/document_list.html
@@ -29,19 +29,28 @@ $(function() {
 {% block maincontent %}
 <h1><img src="{{STATIC_URL}}/img/logo.png">Platforma Redakcyjna</h1>
 
-<div class="document-list">
-    <form action="#" method="GET">
-    <p>Filtr: <input autocomplete="off" name="filter" id="file-list-filter" type="text" size="60" />
-        <input type="submit" value="Znajdź" id="file-list-find-button"/>
-        <input type="reset" value="Wyczyść" id="file-list-reset-button"/>
-    </p>
-    </form>
-	 
+<div id="document-list">    	 
     <div id="file-list">
-    {% for file in document_list %}
-        <p><a href="{% url wiki.views.document_detail file|urlencode %}">{{ file }}</a></p>
-    {% endfor %}
+    	<form action="#" method="GET">
+    	<p>Filtr: <input autocomplete="off" name="filter" id="file-list-filter" type="text" size="60" />
+        	<input type="submit" value="Znajdź" id="file-list-find-button"/>
+        	<input type="reset" value="Wyczyść" id="file-list-reset-button"/>
+    	</p>
+    	</form>
+    	{% for file in document_list %}
+        	<p><a target="_blank" href="{% url wiki.views.document_detail file|urlencode %}">{{ file }}</a></p>
+    	{% endfor %}
     </div>
+	
+	<div id="last-edited-list">
+		<h2>Twoje ostatnio otwierane dokumenty:</h2>
+		<ol>
+			{% for	 name, date in last_docs %}
+			<li><a href="{% url wiki.views.document_detail name|urlencode %}"
+				target="_blank">{{ name }}</a><br/><span class="date">({{ date|date:"H:i:s, d/m/Y" }})</span></li>
+			{% endfor %}			
+		</ol>
+	</div>
 </div>
 
 </div>
diff --git a/apps/wiki/views.py b/apps/wiki/views.py
index ea6fa8bd..dd3991fe 100644
--- a/apps/wiki/views.py
+++ b/apps/wiki/views.py
@@ -10,8 +10,11 @@ from wiki.forms import DocumentForm
 from datetime import datetime
 
 # import google_diff
-import difflib
+# import difflib
+import nice_diff
+import operator
 
+MAX_LAST_DOCS = 10
 
 class DateTimeEncoder(json.JSONEncoder):
      def default(self, obj):
@@ -20,9 +23,12 @@ class DateTimeEncoder(json.JSONEncoder):
          return json.JSONEncoder.default(self, obj)
 
 def document_list(request, template_name = 'wiki/document_list.html'):
+    # TODO: find a way to cache "Storage All"
     return direct_to_template(request, template_name, extra_context = {
         'document_list': storage.all(),
-    })
+        'last_docs': sorted(request.session.get("wiki_last_docs", {}).items(), 
+                        key=operator.itemgetter(1), reverse = True)
+    })  
 
 
 def document_detail(request, name, template_name = 'wiki/document_details.html'):
@@ -31,12 +37,15 @@ def document_detail(request, name, template_name = 'wiki/document_details.html')
     except DocumentNotFound:        
         raise Http404
     
-#    access_time = datetime.ctime();
-#    last_documents = request.session.get("wiki_last_docs", [])
-#        
-#    if name not in last_documents:
-#        last_documents.insert(0, (name, access_time))        
-
+    access_time = datetime.now()
+    last_documents = request.session.get("wiki_last_docs", {})      
+    last_documents[name] = access_time
+    
+    if len(last_documents) > MAX_LAST_DOCS:
+        oldest_key = min(last_documents, key = last_documents.__getitem__)
+        del last_documents[oldest_key]        
+    request.session['wiki_last_docs'] = last_documents   
+                
     if request.method == 'POST':
         
         form = DocumentForm(request.POST, instance = document)
@@ -66,15 +75,13 @@ def document_gallery(request, directory):
 
         raise Http404
     
-def document_diff(request, name, revA, revB):
-    differ = difflib.HtmlDiff(wrapcolumn=60)
-     
+def document_diff(request, name, revA, revB):     
     docA = storage.get(name, int(revA))
-    docB = storage.get(name, int(revB))
-     
-    return HttpResponse(differ.make_table(
-                                docA.plain_text.splitlines(),
-                                docB.plain_text.splitlines() ) )                                           
+    docB = storage.get(name, int(revB)) 
+    
+    
+    return HttpResponse(nice_diff.html_diff_table(docA.plain_text.splitlines(), 
+                                         docB.plain_text.splitlines()) )                                           
     
     
 def document_history(reuqest, name):
diff --git a/platforma/settings.py b/platforma/settings.py
index 5d0479a8..628b4eff 100644
--- a/platforma/settings.py
+++ b/platforma/settings.py
@@ -84,8 +84,9 @@ MIDDLEWARE_CLASSES = (
     'django.contrib.auth.middleware.AuthenticationMiddleware',
     'django_cas.middleware.CASMiddleware',
 
-    'django.middleware.doc.XViewMiddleware',
+    'django.middleware.doc.XViewMiddleware',    
     'maintenancemode.middleware.MaintenanceModeMiddleware',
+    'debug_toolbar.middleware.DebugToolbarMiddleware'
 )
 
 AUTHENTICATION_BACKENDS = (
@@ -127,10 +128,10 @@ COMPRESS_CSS = {
 }
  
 COMPRESS_JS = {
-    # everything except codemirror
+    # everything except codemirror and jquery (which we take from google)
     'detail': {
         'source_filenames': (
-                'js/jquery-1.4.2.min.js', 
+                #'js/jquery-1.4.2.min.js', 
                 'js/jquery.autocomplete.js', 
                 'js/jquery.blockui.js',
                 'js/jquery.elastic.js',
@@ -143,13 +144,14 @@ COMPRESS_JS = {
      },
     'listing': {
         'source_filenames': (
-                'js/jquery-1.4.2.min.js', 
+                # 'js/jquery-1.4.2.min.js', 
                 'js/slugify.js',                
         ),             
         'output_filename': 'compressed/listing_scripts_?.js',
      }
 }
  
+COMPRESS = True
 COMPRESS_CSS_FILTERS = None
 COMPRESS_JS_FILTERS = None
 COMPRESS_AUTO = False
@@ -166,6 +168,8 @@ INSTALLED_APPS = (
     'django.contrib.admindocs',
 
     'django_nose',
+    'debug_toolbar',
+    
     'compress',
 
     'wiki',
diff --git a/platforma/static/css/filelist.css b/platforma/static/css/filelist.css
index 732ebd91..0afc954d 100644
--- a/platforma/static/css/filelist.css
+++ b/platforma/static/css/filelist.css
@@ -29,26 +29,40 @@ body {
 	font-family: sans-serif;
 }
 
-.document-list {
+#file-list {
 	overflow: visible;
 	float: left;
-	max-width: 60%;
+	max-width: 50%;
+	padding-right: 2%;
+	border-right: 1px dashed black;	
+	
 }
 
-.document-list .page-nav-wrap button {
-    width: 2.5em;
+#last-edited-list {
+	float: left;
+	max-width: 35%;
+	margin-left: 5%;
 }
 
-ul.file-tree-part {    
-    margin: 0.5em 1em;
-    padding: 0em;
+#last-edited-list ul {
+	margin: 0px;
+}	
+
+#last-edited-list li {
+	margin-bottom: 1em;
 }
 
-ul.file-tree-part li {
-    list-style: square;    
-    padding: 0em;    
+#last-edited-list .date {
+	font-size: 70%;
+	color: grey;
 }
 
-ul.file-tree-part a {
-    padding: 0em;
+a, a:visited, a:active {
+	color: blue;
+	text-decoration: none;	
+}
+
+a:hover {
+	text-decoration: underline;
+}
 }
diff --git a/platforma/static/css/master.css b/platforma/static/css/master.css
index 5fc28d00..619d3fdd 100644
--- a/platforma/static/css/master.css
+++ b/platforma/static/css/master.css
@@ -1,3 +1,8 @@
+a, a:visited, a:active {
+	color: gold;
+	text-decoration: none;	
+}
+
 body {
     margin: 0;
     overflow: hidden;
@@ -321,43 +326,71 @@ p { margin: 0;}
     line-height: 18px;
 	padding-right: 5px;
 }
+
+#history-view table {
+	width: 80%;
+	margin: 1em auto;	
+}
+
+#history-view .diff_table {
+	width: 90%;
+}
  
 /* DIFFS */
- #history-view table {
- 	width: 75%;
-	margin: 1em auto;
-	max-height: 400px;
-	overflow-y: auto;
- }
- 
- #diff-view {
- 	border: 2px solid black;
-	background: #fafafa;
-	margin: 1em auto;
-	width: 90%; 	
- }
- 
- td.diff_header {
- 	display: none;
- }
- 
- td.diff_next {
- 	display: none;
- }
+ .diff_table { 
+	border-width: 1px;
+	border-style: solid;
+	border-color: black;
+	empty-cells: show;
+	border-spacing: 0px;		
+}
+
+.diff_table td {
+	border-width: 0px 1px 1px 0px;
+	border-style: dotted;
+	border-color: grey;
+	
+	font-size: 10px;
+	line-height: 20px;
+	font-family: monospace;
+	padding: 0px;
+	white-space:pre-line;
+	/*word-wrap:break-word;
+	word-break:break-all; */
+}
 
- .diff_sub {
- 	background-color: #ef4040;
- }
- 
- .diff_chg {
- 	background-color: #efef40;
- }
+.diff_table th {
+	border-width: 0px 1px 1px 0px;
+	border-style: solid;
+	border-color: black;
+	background: #e5ffe5;		
+}
  
- .diff_add {
- 	background-color: #40ef40;
- }
+/* .diff_table td.left, .diff_table td.right {
+	width: 50%;
+}*/
+
+.diff_table tr.change {
+	background-color: #dcdcdc;
+	
+}
 
+.diff_mark {
+	display: inline-block;
+	padding: 2px;
+}
+
+.diff_mark_removed {
+	background-color: #ff9c94;
+}
 
+.diff_mark_added {
+	background-color: #90ee90;
+}
+
+.diff_mark_changed {
+	background-color: yellow;
+}
 /*
  * HTML Editor view
  */ 
diff --git a/platforma/static/js/main.js b/platforma/static/js/main.js
index 3f4d30c2..15099585 100644
--- a/platforma/static/js/main.js
+++ b/platforma/static/js/main.js
@@ -715,7 +715,7 @@ function refreshHistory(callback){
 					+'<td>'+ this[1]+'</td></tr>')			
 			});							
 			$.unblockUI();	
-			callback();
+			if(callback) callback();
 		}
 	});
 };
@@ -887,11 +887,11 @@ $(function() {
     $('.vsplitbar').click(function() {
         if ($('#sidebar').width() == 0) {
             $('#sidebar').width(480).css({right: 0}).show();
-            $('#source-editor, #simple-editor').css({right: 495});
+            $('#editor .editor').css({right: 495});
             $('.vsplitbar').css({right: 480}).addClass('active');
         } else {
             $('#sidebar').width(0).hide();
-            $('#source-editor, #simple-editor').css({right: 15});
+            $('#editor .editor').css({right: 15});
             $('.vsplitbar').css({right: 0}).removeClass('active');
         }
         $(window).resize();
diff --git a/platforma/templates/base.html b/platforma/templates/base.html
index 07c9f786..cb518a35 100644
--- a/platforma/templates/base.html
+++ b/platforma/templates/base.html
@@ -11,6 +11,8 @@
 	<div id="body-wrap">
         <div id="content">{% block maincontent %} {% endblock %}</div>
         </div>
+	<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js" 
+		type="text/javascript"></script>
     {% block extrabody %}{% endblock %}
     </body>
 </html>
diff --git a/requirements.txt b/requirements.txt
index 00c38ace..975ceb77 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -6,4 +6,6 @@ mercurial>=1.3.1
 librarian>=1.3.dev,<1.4
 PyYAML>=3.0
 # MySQL-python>=1.2,<2.0
+# debug stuff
 django-nose>=0.0.3
+django-debug-toolbar>=0.8
diff --git a/scripts/nice_diff.py b/scripts/nice_diff.py
deleted file mode 100644
index 3de9ad4e..00000000
--- a/scripts/nice_diff.py
+++ /dev/null
@@ -1,57 +0,0 @@
-import difflib
-import re
-
-def read_whole(path):
-	with open(path) as f:
-		return f.readlines()
-
-DIFF_RE = re.compile(r"""\x00([+^-])""" ,re.UNICODE)
-NAMES = { '+': 'added', '-': 'removed', '^': 'changed' }
-
-def diff_replace(match):
-	return """<span class="diff_mark diff_mark_%s">""" %  NAMES[match.group(1)]
-
-
-def filter_line(line):
-	return DIFF_RE.sub(diff_replace, line).replace('\x01', '</span>')
-
-def group(diffs):
-	group_a = []
-	group_b = []
-
-	a, b, _ = diffs.next()
-	group_a.append(a[1])
-	group_b.append(b[1])
-
-
-	for _ in range(10):
-		a, b, _ = diffs.next()
-		group_a.append(a[1])
-		group_b.append(b[1])	
-
-		while a[0] == '':
-			a, b, _ = diffs.next()
-			group_a.append(a[1])
-			group_b.append(b[1])
-
-		yield group_a[:-1], group_b[:-1]
-		group_a, group_b = group_a[-1:], group_b[-1:]
-
-def join_to_html(diffs):
-	for group_a, group_b in group(diffs):
-		yield """
-<div class="change_block">
-	<div class="old_block">%s</div>
-	<div class="new_block">%s</div>
-</div>""" % ( 
-	'\n'.join( filter_line(line) for line in group_a ),
-	'\n'.join( filter_line(line) for line in group_b ),
-	)
-
-fa = read_whole("file_a")
-fb = read_whole("file_b")
-
-print '\n'.join( repr(x) for x in  difflib._mdiff(fa, fb)  )
-print "**************************"
-print '\n'.join( join_to_html( difflib._mdiff(fa, fb) ) )
-# print '\n'.join( repr(x) for x in group(difflib._mdiff(fa, fb)) )
-- 
2.20.1