From 1350bba16b1b864dcd4b335083e70517ac069061 Mon Sep 17 00:00:00 2001 From: Sebastian Sdorra Date: Wed, 30 Oct 2013 09:02:38 +0100 Subject: [PATCH 1/2] use full message instead of short message for git commits --- .../sonia/scm/repository/GitChangesetConverter.java | 11 ++++++++--- .../webapp/resources/js/override/ext.util.format.js | 7 +++++++ .../sonia.repository.changesetviewergrid.js | 10 +++++++--- .../js/repository/sonia.repository.commitpanel.js | 2 +- 4 files changed, 23 insertions(+), 7 deletions(-) diff --git a/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/GitChangesetConverter.java b/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/GitChangesetConverter.java index 27accf504f..ab588816ce 100644 --- a/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/GitChangesetConverter.java +++ b/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/GitChangesetConverter.java @@ -83,7 +83,6 @@ public class GitChangesetConverter implements Closeable * * * @param repository - * @param idLength */ public GitChangesetConverter(org.eclipse.jgit.lib.Repository repository) { @@ -96,7 +95,6 @@ public class GitChangesetConverter implements Closeable * * @param repository * @param revWalk - * @param idLength */ public GitChangesetConverter(org.eclipse.jgit.lib.Repository repository, RevWalk revWalk) @@ -195,7 +193,13 @@ public class GitChangesetConverter implements Closeable PersonIdent authorIndent = commit.getAuthorIdent(); Person author = new Person(authorIndent.getName(), authorIndent.getEmailAddress()); - String message = commit.getShortMessage(); + String message = commit.getFullMessage(); + + if (message != null) + { + message = message.trim(); + } + Changeset changeset = new Changeset(id, date, author, message); if (parentList != null) @@ -214,6 +218,7 @@ public class GitChangesetConverter implements Closeable if (Util.isNotEmpty(tagCollection)) { + // create a copy of the tag collection to reduce memory on caching changeset.getTags().addAll(Lists.newArrayList(tagCollection)); } diff --git a/scm-webapp/src/main/webapp/resources/js/override/ext.util.format.js b/scm-webapp/src/main/webapp/resources/js/override/ext.util.format.js index 1a13c3dfa7..258b489bac 100644 --- a/scm-webapp/src/main/webapp/resources/js/override/ext.util.format.js +++ b/scm-webapp/src/main/webapp/resources/js/override/ext.util.format.js @@ -54,6 +54,13 @@ Ext.apply(Ext.util.Format, { id: function(value){ return this.substr(value, 0, 12); + }, + + convertLineBreaks: function(value){ + if (value){ + value = value.replace(/(\r\n|\n|\r)/gm, "
"); + } + return value; } }); diff --git a/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.changesetviewergrid.js b/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.changesetviewergrid.js index 22ee81471f..0939d69b17 100644 --- a/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.changesetviewergrid.js +++ b/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.changesetviewergrid.js @@ -174,9 +174,13 @@ Sonia.repository.ChangesetViewerGrid = Ext.extend(Ext.grid.GridPanel, { } } var description = record.data.description; - // if ( description ){ - // description = Ext.util.Format.htmlEncode(description); - // } + if ( description ){ + // description = Ext.util.Format.htmlEncode(description); + var index = description.indexOf('\n'); + if ( index > 0 ){ + description = description.substring(0, index) + " ..."; + } + } var date = record.data.date; if ( date ){ date = Ext.util.Format.formatTimestamp(date); diff --git a/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.commitpanel.js b/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.commitpanel.js index b3456e6fae..b711a477d3 100644 --- a/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.commitpanel.js +++ b/scm-webapp/src/main/webapp/resources/js/repository/sonia.repository.commitpanel.js @@ -45,7 +45,7 @@ Sonia.repository.CommitPanel = Ext.extend(Ext.Panel, { \n\
\n\

Commit {id}

\n\ -

{description}

\n\ +

{description:convertLineBreaks}

\n\

\n\ \n\ {name} <{mail}>\n\ From d9efa9c888997b0d8ae1dd75756764b9a62d01f2 Mon Sep 17 00:00:00 2001 From: Sebastian Sdorra Date: Fri, 13 Dec 2013 08:07:40 +0100 Subject: [PATCH 2/2] close branch issue-474