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 4804e43..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.12
+// @version      0.18
 // @author       Tobias Sachs
 //  ... in @match replace "ads" with the url of you Azure DevOps Server
 // @match        https://ads/*
@@ -10,6 +10,13 @@
 // @description
 // ==/UserScript==
 
+// 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
@@ -33,6 +40,7 @@
         console.debug("observe...");
     };
 
+    let checkRegex = /^.*(Associated|Resolved).*[:.]/;
     let fixCommentContents = (items) => {
         if (items === null || items === undefined || items.length === 0) {
             return;
@@ -40,15 +48,20 @@
         console.info("fixing '" + items.length +"' comments.");
         for (var i = 0; i < items.length; i++) {
             let el = items[i];
-            let html = el.innerHTML;
+            if(el.wasTampered) { continue; }
 
-            if (html.startsWith("Associated") || html.startsWith("Resolved")) {
+            let html = el.innerHTML;
+            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/_versionControl/changeset/$1'>#$1</a>");
+                html = html.replace(/#(\d+)/g, "<a href='/HeBa/Entwicklung/_workitems/edit/$1'>#$1</a>");
                 html = html.replace(/\n/gi, "<br />");
-                html = html.replace(/(@@CUST.*)/s, "<br><span style=\"color: green; font-style:italic;\">$1</span>");
+                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 = "😋";
             }
         }
     };