X-Git-Url: https://src.twobees.de/?p=tampermonkeyscripts.git;a=blobdiff_plain;f=AzureDevOpsCommentEnhancer.user.js;h=b42e8bec2a74c500f46c308e55514cb6f66595ea;hp=aa338109e0cbfd84efe0cb9687d93c36bb3ec0b2;hb=HEAD;hpb=585e782df064fcc0c31d584a9ea182245e060a89

diff --git a/AzureDevOpsCommentEnhancer.user.js b/AzureDevOpsCommentEnhancer.user.js
old mode 100755
new mode 100644
index aa33810..b42e8be
--- a/AzureDevOpsCommentEnhancer.user.js
+++ b/AzureDevOpsCommentEnhancer.user.js
@@ -1,24 +1,36 @@
 // ==UserScript==
 // @name         Fix ADS checkin comments in discussion and history of workitems
-// @version      0.7
+// @version      0.18
 // @author       Tobias Sachs
 //  ... in @match replace "ads" with the url of you Azure DevOps Server
 // @match        https://ads/*
-// @updateURL    https://github.com/tsheba/tampermonkeyscripts/raw/master/AzureDevOpsCommentEnhancer.user.js
-// @downloadURL  https://github.com/tsheba/tampermonkeyscripts/raw/master/AzureDevOpsCommentEnhancer.user.js
+// @updateURL    https://src.twobees.de/?p=tampermonkeyscripts.git;a=blob_plain;f=AzureDevOpsCommentEnhancer.user.js;hb=HEAD
+// @downloadURL  https://src.twobees.de/?p=tampermonkeyscripts.git;a=blob_plain;f=AzureDevOpsCommentEnhancer.user.js;hb=HEAD
 // @grant        none
 // @description
 // ==/UserScript==
 
-// 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 fixWorkitems = () => {
         let found = document.getElementsByClassName("comment-content");
         fixCommentContents(found);
 
@@ -26,81 +38,87 @@
         fixCommentContents(found);
 
         console.debug("observe...");
-    }
-    let fixCommentContents = (items) =>
-    {
-        if (items === null || items === undefined || items.length === 0)
-        {
+    };
+
+    let checkRegex = /^.*(Associated|Resolved).*[:.]/;
+    let fixCommentContents = (items) => {
+        if (items === null || items === undefined || items.length === 0) {
             return;
         }
         console.info("fixing '" + items.length +"' comments.");
-        for (var i = 0; i < items.length; i++){
+        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 = "😋";
             }
         }
     };
 
-    let fixVersionControl = () =>
-    {
+    let fixVersionControl = () => {
         let elToFix;
         let found = document.getElementsByClassName("changeset-version")[0];
-        if (found) {
-            // if opened from email notification it is the first span in div "changeset-version"
-            elToFix = found.querySelector("span");
-        }
-        else
-        {
-            // if opened from histrory in ads it is the span in div "changeset-id"
-            // elToFix = document.getElementsByClassName("changeset-id")[0];
-        }
-        if (!elToFix)
-        {
+        if (!found) {
             return;
         }
 
-        elToFix.innerHTML = elToFix.innerHTML.replace(/(Changeset )(\d+)/, "$1<a href='/HeBa/Entwicklung/_versionControl/changeset/$2'>$2</a>");
+        // if opened from email notification, it is the first span in div "changeset-version"
+        elToFix = found.querySelector("span");
 
-    }
+        if (!elToFix) {
+            return;
+        }
+
+        elToFix.innerHTML = elToFix.innerHTML
+            .replace(/(Changeset )(\d+)/,
+                "$1<a href='/HeBa/Entwicklung/_versionControl/changeset/$2'>$2</a>");
+    };
 
     let fixit = () => {
-        if (timerId){
+        if (timerId) {
             console.debug("fixit timerreset...");
             clearTimeout(timerId);
         }
 
         observer.disconnect();
 
-        timerId = setTimeout(function(){
+        timerId = setTimeout(function() {
             timerId = undefined;
 
             let url = window.location.href;
 
-            if (url.includes("/_versionControl"))
-            {
+            if (url.includes("/_versionControl")) {
                 fixVersionControl();
             }
-            else{
+            else {
                 // if (url.includes("/_workitems")){
-                // does not work since workitem are often shown in 
-                // diaolgs on random pages
+                // does not work since workitems are often shown in
+                // dialogs on random pages
                 fixWorkitems();
             }
 
-            // keep watching for changes.
+            // keep watching for changes
             observer.observe(document, { subtree: true, childList: true, characterData: true });
         }, 300);
     };
 
     const observer = new MutationObserver(function() {
-        console.debug('observertriggered...');
+        console.debug('observer was triggered...');
         fixit();
     });
 
     fixit();
 
 })();
+