User story #2782
closedUse a real diff viewing for Directive modification event log
Description
In EventLog screen, when we see a Directive modification event, we are flooded by a lot of not really meaningfull XML.
A first solution is to display a diff of the before/after XML.
I see two solution:
1/ create a unified diff of the two XML, and display it with something like http://torisugari.hostei.com/development/UnifiedDiffToHTMLTable.html
(it's really easy to create diff in Java server-side, with for ex. http://code.google.com/p/java-diff-utils/)
2/ directly use a Javascript diff like https://github.com/kpdecker/jsdiff
The second one seems the easier.
In each case, we are making strong assomption about the structure of the XML (not two much different) witch should be OK if the XML document is correctly pretty printed.
Files
Updated by Jonathan CLARKE over 12 years ago
- Assignee deleted (
Jean VILVER) - Target version changed from 2.4.0~beta4 to 24
Updated by Jean VILVER over 12 years ago
- Status changed from 2 to Pending technical review
- % Done changed from 0 to 100
Applied in changeset 5f978248d0fc42e32203f502acd83ae3754fc792.
Updated by Jean VILVER over 12 years ago
The result is in the attached image.
Updated by Jonathan CLARKE over 12 years ago
- Assignee set to Jean VILVER
- Target version changed from 24 to 2.4.0~beta5
Updated by Jean VILVER over 12 years ago
The result :
Updated by François ARMAND over 12 years ago
- Status changed from Pending technical review to 10
That seems ok.
Jon, I let you see with Jean for the look'n'feel you want for the result.
Updated by Jonathan CLARKE over 12 years ago
- Status changed from 10 to Released
Looks good to me! Thanks Jean.
Updated by Nicolas CHARLES over 12 years ago
- Status changed from Released to Discussion
Jean, you did not include the license file in this commit ( https://github.com/kpdecker/jsdiff/blob/master/LICENSE )
Could you add it please ?
Updated by Jean VILVER over 12 years ago
- Status changed from Discussion to Pending technical review
Applied in changeset bf9fdd7a856558010719a7db498b65c6ae4da24d.
Updated by Nicolas CHARLES over 12 years ago
- Status changed from Pending technical review to 10
This looks valid, thank you Jean
Updated by Jonathan CLARKE about 12 years ago
- Status changed from 10 to Released