Clean up case titles

Author: adam3smith

Old Bailey Online.js

@@ -9,7 +9,7 @@
 	"inRepository": true,
 	"translatorType": 4,
 	"browserSupport": "gcsibv",
-	"lastUpdated": "2018-10-08 14:15:17"
+	"lastUpdated": "2018-10-12 23:37:03"
 }
 
 /*
@@ -32,7 +32,7 @@
 	along with Zotero.  If not, see <http://www.gnu.org/licenses/>.
 	
 	***** END LICENSE BLOCK *****
-*/ 	
+*/
 // URLs
 // trial: div=t18000115-12
 // session: name=16900430
@@ -46,7 +46,7 @@ function detectWeb(doc, url) {
 		return "case";
 	} else if ( url.includes("search.jsp") &&  getSearchResults(doc, true)) {
 		return "multiple";
-	} 
+	}
 }
 
 // to do:  not trials...
@@ -91,20 +91,25 @@ if (url.includes('browse.jsp')  && ( url.includes('div=OA') || url.includes('nam
 
 	var sessDate = ZU.xpathText(doc, '//div[@class="sessionsPaper"]/div[@class="sessions-paper-date"]'); // add session date, as the date is now in a gettable node
 
-	newItem.date = ZU.strToISO(sessDate); 
+	newItem.date = ZU.strToISO(sessDate);
 
 	if (newItem.itemType == "case" && trialTitle && trialTitle == trialTitle.toUpperCase()) {
 		newItem.title = ZU.capitalizeTitle(trialTitle, true);  // todo tidying this up - sometimes no name, messy punctuation
 	} else if (newItem.itemType == "book") {
 		newItem.title = trialTitle + " " + sessDate;
 	}
+
+	newItem.title = newItem.title.trim().replace(/[,.]+$/, "");
+	if (!newItem.title) {
+		newItem.title = "[no title]";
+	}
 
 	var referenceNo = ZU.xpathText(doc, '//div[@class="ob-panel"][1]/table[@class="ob-info-table"][1]/tbody/tr[th[contains(text(),"Reference")]]/td').trim(); // changed fetching Reference number
 
 	newItem.extra = "Reference Number: " + referenceNo; // putting the ref number in the Extra field had a particular function, was it for Voyant? or the defunct DMCI plugin? retain it at least for now (non trials will want it anyway)
 
 	if (newItem.itemType == "case") {
-		newItem.docketNumber = referenceNo; 
+		newItem.docketNumber = referenceNo;
 	}
 
 	if (newItem.itemType == "book") {
@@ -144,7 +149,7 @@ if (newItem.itemType == "case") {
 
 // use print-friendly URLs for snapshots
 
-	var attachmentUrl = "https://www.oldbaileyonline.org/print.jsp?div=" + referenceNo;  
+	var attachmentUrl = "https://www.oldbaileyonline.org/print.jsp?div=" + referenceNo;
 	newItem.attachments.push({ url  : attachmentUrl,    title : "OBO Snapshot",    mimeType : "text/html" });
 
 	newItem.complete();
@@ -166,7 +171,7 @@ function doWeb(doc, url) {
 			}
 			ZU.processDocuments(articles, scrape);
 		});
-	} else if (url.includes('browse.jsp') && ( url.includes('div=') || url.includes('name=') ) )  {  
+	} else if (url.includes('browse.jsp') && ( url.includes('div=') || url.includes('name=') ) )  {
 		scrape(doc, url);
 	}
 }
@@ -186,7 +191,7 @@ var testCases = [
 		"items": [
 			{
 				"itemType": "case",
-				"caseName": "Peter Asterbawd, Andrew Forsman.",
+				"caseName": "Peter Asterbawd, Andrew Forsman",
 				"creators": [],
 				"dateDecided": "1800-01-15",
 				"docketNumber": "t18000115-12",
@@ -233,7 +238,7 @@ var testCases = [
 				"creators": [],
 				"date": "1711-04-21",
 				"extra": "Reference Number: OA17110421",
-				"libraryCatalog": "Old Bailey Online 201809",
+				"libraryCatalog": "Old Bailey Online",
 				"place": "London",
 				"url": "https://www.oldbaileyonline.org/browse.jsp?div=OA17110421",
 				"attachments": [
@@ -258,7 +263,7 @@ var testCases = [
 				"creators": [],
 				"date": "1711-04-21",
 				"extra": "Reference Number: OA17110421",
-				"libraryCatalog": "Old Bailey Online 201809",
+				"libraryCatalog": "Old Bailey Online",
 				"place": "London",
 				"url": "https://www.oldbaileyonline.org/browse.jsp?name=OA17110421",
 				"attachments": [
@@ -283,7 +288,7 @@ var testCases = [
 				"creators": [],
 				"date": "1710-04-18",
 				"extra": "Reference Number: 17100418",
-				"libraryCatalog": "Old Bailey Online 201809",
+				"libraryCatalog": "Old Bailey Online",
 				"place": "London",
 				"url": "https://www.oldbaileyonline.org/browse.jsp?name=17100418",
 				"attachments": [
@@ -304,7 +309,7 @@ var testCases = [
 		"items": [
 			{
 				"itemType": "case",
-				"caseName": ".",
+				"caseName": "[no title]",
 				"creators": [],
 				"dateDecided": "1678-08-28",
 				"docketNumber": "t16780828-12",