Adding ability to embed/link to single file within a gist (from #936740)

Author: Jake Bell

gist_filter.module

@@ -33,10 +33,10 @@ function gist_filter_filter_info() {
  */
 function gist_filter_gist_filter_process($text, $filter, $format) {
   $display = $filter->settings['gist_filter_display_method'];
-  return preg_replace('@\[gist\:(\d+)\]@', _gist_display('$1', $display), $text);
+  $callback = '_gist_display_' . $display;
+  return preg_replace_callback('@\[gist\:(\d+)\:?(.*)?\]@', $callback, $text);
 }
 
-
 /**
  * Settings callback for gist_filter.
  */
@@ -63,14 +63,19 @@ function gist_filter_filter_tips($filter, $format, $long = FALSE) {
 }
 
 /**
- * Replace the text with embedded script or link.
+ * Replace the text with embedded script.
+ */
+function _gist_display_embed($matches) {
+  $gist_url = 'http://gist.github.com/' . $matches[1];
+  $gist_url = isset($matches[2]) && !empty($matches[2]) ? $gist_url . '.js?file=' . $matches[2] : $gist_url . '.js';
+  return '<script src="' . $gist_url . '"></script>';
+}
+
+/**
+ * Replace the text with a link.
  */
-function _gist_display($gist, $display) {
-  $url = 'http://gist.github.com/' . $gist;
-  switch ($display) {
-    case 'embed':
-      return '<script src="' . $url . '.js"></script>';
-    case 'link':
-      return l($url, $url);
-  }
+function _gist_display_link($matches) {
+  $gist_url = 'http://gist.github.com/' . $matches[1];
+  $gist_url = isset($matches[2]) && !empty($matches[2])? $gist_url . '#file_' . $matches[2] : $gist_url;
+  return l($gist_url, $gist_url);
 }

gist_filter.test

@@ -58,6 +58,31 @@ class GistFilterTestCase extends DrupalWebTestCase {
 
   }
 
+  /**
+   * Testing the embedded gist option with a file parameter.
+   */
+  function testEmbedStyleWithFile() {
+
+    // Turn on our input filter and set the option to embed.
+    $edit = array(
+      'filters[gist_filter][status]' => 1,
+      'filters[gist_filter][settings][gist_filter_display_method]' => 'embed',
+    );
+    $this->drupalPost('admin/config/content/formats/plain_text', $edit, t('Save configuration'));
+
+    // Create a test node
+    $langcode = LANGUAGE_NONE;
+    $edit = array(
+      "title" => $this->randomName(),
+      "body[$langcode][0][value]" => 'Hello! [gist:865412:php_file.php]',
+    );
+    $result = $this->drupalPost('node/add/' . $this->contentType->type, $edit, t('Save'));
+    $this->assertResponse(200);
+    $this->assertRaw("Hello! ");
+    $this->assertRaw('<script src="http://gist.github.com/865412.js?file=php_file.php"></script>');
+
+  }
+
   /**
    * Testing the link option. 
    */
@@ -82,4 +107,28 @@ class GistFilterTestCase extends DrupalWebTestCase {
 
   }
 
+  /**
+   * Testing the link option. 
+   */
+  function testLinkStyleWithFile() {
+
+    // Turn on our input filter and set the option to link.
+    $edit = array(
+      'filters[gist_filter][status]' => 1,
+      'filters[gist_filter][settings][gist_filter_display_method]' => 'link',
+    );
+    $this->drupalPost('admin/config/content/formats/plain_text', $edit, t('Save configuration'));
+
+    // Create a test node
+    $langcode = LANGUAGE_NONE;
+    $edit = array(
+      "title" => $this->randomName(),
+      "body[$langcode][0][value]" => 'Hello! [gist:865412:php_file.php]',
+    );
+    $result = $this->drupalPost('node/add/' . $this->contentType->type, $edit, t('Save'));
+    $this->assertResponse(200);
+    $this->assertRaw('Hello! <a href="http://gist.github.com/865412#file_php_file.php">http://gist.github.com/865412#file_php_file.php</a>');
+
+  }
+
 }