Add in ability to link to python config files

Author: raggleton

README.md

@@ -1,21 +1,18 @@
 #github-linkify-cmssw
 
-This Chrome extension turns `#include X` in C/C++ files (and in the future `import X`/`from X import Y` in python configs) into hyperlinks when viewing CMSSW files on Github.com, so you can get to the includes easily. (I don't know why this hasn't been done already.)
+This Chrome extension turns `#include X` in C/C++ files, and `import X`/`from X import Y`/`process.load(XXX.YYY.ZZZ_cff)` in python config files, into hyperlinks when viewing CMSSW files on Github.com, so you can get to the includes easily. (I don't know why this hasn't been done already.)
 
 ## Instructions
 
 1) Download the latest release:
 
 https://github.com/raggleton/github-linkify-cmssw/releases/latest
 
-2) Then install me:
+2) Go to [chrome://extensions](chrome://extensions)
 
-    - Go to [chrome://extensions](chrome://extensions)
-    - Drag and drop the `github-linkify-cmssw.crx` package onto the window. Lcick OK when it asks permission to access github.com. This should install it.
+3) Drag and drop the `github-linkify-cmssw.crx` package onto the window. Click OK when it asks permission to access github.com. This should install it.
 
 ## TODO
-- [ ] Do for Python `import`/`from X import Y` statements
-- [ ] ignore STL/boost libraries (like `<iostream>`)
 - [ ] keep the colour of the links to their original colour
 
 Maybe I'll do a Firefox equivalent some day. If I ever get this one done.

contentscript.js

@@ -16,9 +16,9 @@ var cpp_pattern = /#include/;
 var cpp_header_pattern = /[<\"].*[>\"]/; // does "myheader.h" or <myheader.h>
 
 // pattern to do python matching
-var py_pattern1 = /import/;
-var py_pattern2 = /from\s.*\simport\s.*/; // might not need this one
-var py_pattern3 = /import.*as.*/;
+var py_pattern1 = /from\s.*\simport\s.*/;
+var py_pattern2 = /import.*as.*/;
+var py_pattern3 = /import/;
 var py_pattern4 = /process.load(.*)/;
 
 // loop through all rows <tr>
@@ -48,9 +48,34 @@ for (var i = 0; i < rows.length; i++) {
                 //     }
                 // }
             }
-            // test for python
-            else if (py_pattern1.test(line) || py_pattern3.test(line) || py_pattern4.test(line)) {
+            // tests for python imports/fragments
+            var config = "";
+            if (py_pattern1.test(line)) {
+                config = line.replace(/from\s/,"");
+                config = config.replace(/\simport.*/,"");
+            } else if (py_pattern2.test(line)) {
+                config = line.replace(/import\s/,"");
+                config = config.replace(/\sas\s.*/,"");
+            } else if (py_pattern3.test(line)) {
+                config = line.replace(/import\s/,"");
+            } else if (py_pattern4.test(line)) {
+                config = line.replace("process.load(","");
+                config = config.replace(")","");
+            }
+
+            // if we have a valid python config, turn it into a path
+            // ensure it has a / in it so it isn't a 3rd party library
+            if (config != "" && (/\./.test(config))) {
+                config = config.replace(/['"]/g,"");
+                var path = config.replace(/\./g,"/");
+                var parts = path.split("/");
+                path = path.replace(parts[parts.length-1],"python/"+parts[parts.length-1]+".py")
+                var link = rootURL.concat(path)
 
+                // let's replace the text with a link
+                // TODO keep same styling as before
+                var cell_html = cell.innerHTML;
+                cell.innerHTML = cell_html.replace(config, "<a href=\""+link+"\">"+config+"</a>");
             }
         }
     }

manifest.json

@@ -1,7 +1,7 @@
 {
     "name": "Github linkify for CMSSW",
-    "description": "Turns #include statments into links when browing CMSSW on Github.",
-    "version": "0.1",
+    "description": "Turns #include statments in C++ files, and import X/from X import Y/process.load(XXX.YYY.ZZZ_cff) in python config files, into links when browing CMSSW on Github.",
+    "version": "0.2.0",
     "content_scripts": [
         {
         "matches": [