From 6dcf3602c0c04eb63ee4cf46d3482b9b5993d2f8 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Tim=20D=C3=BCsterhus?= <duesterhus@woltlab.com>
Date: Mon, 7 Jul 2014 19:52:05 +0200
Subject: [PATCH] Show nice diff instead of unix diff in editHistory.tpl

---
 com.woltlab.wcf/templates/editHistory.tpl | 39 +++++++++++++++++++++--
 1 file changed, 37 insertions(+), 2 deletions(-)

diff --git a/com.woltlab.wcf/templates/editHistory.tpl b/com.woltlab.wcf/templates/editHistory.tpl
index d3c40fe44c..d14134a54d 100644
--- a/com.woltlab.wcf/templates/editHistory.tpl
+++ b/com.woltlab.wcf/templates/editHistory.tpl
@@ -16,7 +16,42 @@
 
 {include file='userNotice'}
 
-<pre>{$diff}</pre>
+{if $diff}
+<div><div>
+{assign var='prevType' value=''}
+{foreach from=$diff->getRawDiff() item='line'}
+{if $line[0] !== $prevType}
+	</div>
+	
+	{* unmodified, after deletion needs a "fake" insertion *}
+	{if $line[0] === ' ' && $prevType === '-'}<div class="containerPadding"></div>{/if}
+	
+	{* unmodified and deleted start a new container *}
+	{if $line[0] === ' ' || $line[0] === '-'}</div>{/if}
+	
+	{* adding, without deleting needs a "fake" deletion *}
+	{if $line[0] === '+' && $prevType !== '-'}
+		</div>
+		<div class="container marginTop sideBySide">
+			<div class="containerPadding">
+			</div>
+	{/if}
+	
+	{if $line[0] === ' '}
+		<div class="container marginTop">
+	{/if}
+	{if $line[0] === '-'}
+		<div class="container marginTop sideBySide">
+	{/if}
+	<div class="containerPadding"{if $line[0] === '+'} style="color: green;"{elseif $line[0] === '-'} style="color: red;"{/if}>
+{/if}
+{if $line[0] === ' '}{$line[1]}<br />{/if}
+{if $line[0] === '-'}{$line[1]}<br />{/if}
+{if $line[0] === '+'}{$line[1]}<br />{/if}
+{assign var='prevType' value=$line[0]}
+{/foreach}
+</div></div>
+{/if}
 
 <form action="{link controller='EditHistory'}{/link}" method="get">
 	<div class="tabularBox tabularBoxTitle marginTop">
@@ -40,7 +75,7 @@
 				<tr>
 					<td class="columnIcon">
 						<span class="icon icon16 icon-undo disabled"></span>
-						<input type="radio" name="oldID" value="current"{if $oldID == 'current'} checked="checked"{/if} /> <input type="radio" name="newID" value="current"{if $newID == 'current'} checked="checked"{/if} />
+						<input type="radio" name="oldID" value="current"{if $oldID === 'current'} checked="checked"{/if} /> <input type="radio" name="newID" value="current"{if $newID === 'current'} checked="checked"{/if} />
 						{event name='rowButtons'}
 					</td>
 					<td class="columnID"><strong>{lang}wcf.edit.currentVersion{/lang}</strong></td>
-- 
2.20.1