0.18: make '@@DEPENDENCIES' blue

[tampermonkeyscripts.git] / AzureDevOpsCommentEnhancer.user.js

diff --git a/AzureDevOpsCommentEnhancer.user.js b/AzureDevOpsCommentEnhancer.user.js
old mode 100755 (executable)
new mode 100644 (file)
index 8d06776..b42e8be
--- a/AzureDevOpsCommentEnhancer.user.js
+++ b/AzureDevOpsCommentEnhancer.user.js
@@ -1,6 +1,6 @@
 // ==UserScript==
 // @name         Fix ADS checkin comments in discussion and history of workitems
-// @version      0.9
+// @version      0.18
 // @author       Tobias Sachs
 //  ... in @match replace "ads" with the url of you Azure DevOps Server
 // @match        https://ads/*
@@ -10,14 +10,25 @@
 // @description
 // ==/UserScript==
 
-// 0.9: update download/update URLs
-// 0.8: fix typos/formatting
-// 0.7: fix work item tampering
-// 0.6: Add link to Changeset in diff view
-
+// 0.18: make '@@DEPENDENCIES' blue
+// 0.17: search caseless for '@@CUST'
+// 0.16: do not modify a comment more than once
+// 0.15: fix links to work items found in commitmessages (e.g. #1235)
+// 0.14: also check html property of comments
+// 0.13: Allow to manually insert changeset comments, which where not associated with the item during checking.
+//       Just copy changecomment into the commentsection and prefix with "Associated with changeset CHANGESET_NUMBER:"
+// 0.12: also fix "Resolved with changeset" comments
+// 0.11: fix regexp for later changesets attached.
+// 0.10: #Bugnumber to links, highlight comments for customors in checkins
+// 0.09: updateq download/update URLs
+// 0.08: fix typregexp for later changesets attachedos/formatting
+// 0.07: fix work itlsem tampering
+// 0.06: Add link to Changeset in diff view
+
+/* jshint esversion:6 */
 (function() {
     'use strict';
-    let timerId = undefined;
+    let timerId;
 
     let fixWorkitems = () => {
         let found = document.getElementsByClassName("comment-content");
@@ -29,6 +40,7 @@
         console.debug("observe...");
     };
 
+    let checkRegex = /^.*(Associated|Resolved).*[:.]/;
     let fixCommentContents = (items) => {
         if (items === null || items === undefined || items.length === 0) {
             return;
@@ -36,10 +48,20 @@
         console.info("fixing '" + items.length +"' comments.");
         for (var i = 0; i < items.length; i++) {
             let el = items[i];
+            if(el.wasTampered) { continue; }
+
             let html = el.innerHTML;
-            if (html.startsWith("Associated")) {
-                html = html.replace(/(Associated with changeset )(\d*):/, "<b>$1<a href='/HeBa/Entwicklung/_versionControl/changeset/$2'>$2</a></b>:<br />");
-                el.innerHTML = html.replace(/\n/gi, "<br />");
+            if (    checkRegex.test(html)
+                || (el.textContext && checkRegex.test(el.textContext))
+               ) {
+                html = html.replace(/((Associated|Resolved) with changeset )(\d*)([:.])/,
+                    "<b>$1<a href='/HeBa/Entwicklung/_versionControl/changeset/$3'>$3</a></b>$4<br />");
+                html = html.replace(/#(\d+)/g, "<a href='/HeBa/Entwicklung/_workitems/edit/$1'>#$1</a>");
+                html = html.replace(/\n/gi, "<br />");
+                html = html.replace(/(@@CUST.*)/is, "<br><span style=\"color: green; font-style:italic;\">$1</span>");
+                html = html.replace(/(@@Dependencies.*)/is, "<span style=\"color: blue; font-style:italic;\">$1</span>");
+                el.innerHTML = html;
+                el.wasTampered = "😋";
             }
         }
     };
@@ -58,8 +80,10 @@
             return;
         }
 
-        elToFix.innerHTML = elToFix.innerHTML.replace(/(Changeset )(\d+)/, "$1<a href='/HeBa/Entwicklung/_versionControl/changeset/$2'>$2</a>");
-    }
+        elToFix.innerHTML = elToFix.innerHTML
+            .replace(/(Changeset )(\d+)/,
+                "$1<a href='/HeBa/Entwicklung/_versionControl/changeset/$2'>$2</a>");
+    };
 
     let fixit = () => {
         if (timerId) {
@@ -79,7 +103,7 @@
             }
             else {
                 // if (url.includes("/_workitems")){
-                // does not work since workitems are often shown in 
+                // does not work since workitems are often shown in
                 // dialogs on random pages
                 fixWorkitems();
             }
@@ -97,3 +121,4 @@
     fixit();
 
 })();
+