From cfd9743eaa576d1a2dad5bc3213d33b770fb6de2 Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Mon, 28 Jul 2014 18:49:53 -0700
Subject: [PATCH] remove extra period from end of summary

---
 plugins/summarize.js            |  4 ++--
 plugins/test/specs/summarize.js | 10 ++++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/plugins/summarize.js b/plugins/summarize.js
index 13810f8b..da72f738 100644
--- a/plugins/summarize.js
+++ b/plugins/summarize.js
@@ -18,8 +18,8 @@ exports.handlers = {
         // If the summary is missing, grab the first sentence from the description
         // and use that.
         if (e.doclet && !e.doclet.summary && e.doclet.description) {
-            // The summary may end with `. ` or with `.<` (a period followed by an HTML tag).
-            e.doclet.summary = e.doclet.description.split(/\.\s|\.</)[0];
+            // The summary may end with `.$`, `. `, or `.<` (a period followed by an HTML tag).
+            e.doclet.summary = e.doclet.description.split(/\.$|\.\s|\.</)[0];
             // Append `.` as it was removed in both cases, or is possibly missing.
             e.doclet.summary += '.';
 
diff --git a/plugins/test/specs/summarize.js b/plugins/test/specs/summarize.js
index c27452b1..9fb5ab80 100644
--- a/plugins/test/specs/summarize.js
+++ b/plugins/test/specs/summarize.js
@@ -51,6 +51,16 @@ describe('summarize', function() {
             expect(doclet.summary).toBe('This sentence is the summary.');
         });
 
+        it('should not add an extra period if there is only one sentence in the description',
+            function() {
+            var doclet = {
+                description: 'This description has only one sentence.'
+            };
+            handler({ doclet: doclet });
+
+            expect(doclet.summary).toBe('This description has only one sentence.');
+        });
+
         it('should use the entire description, plus a period, as the summary if the description ' +
             'does not contain a period', function() {
             var doclet = {