Remarkup code blocks: guess language from "name="

Summary:
The file name is a sufficient source of information, if available.

Closes T15729

Test Plan: The new unit test is green. Old unit tests are green.

Reviewers: O1 Blessed Committers, avivey

Reviewed By: O1 Blessed Committers, avivey

Subscribers: avivey, tobiaswiese, Matthew, Cigaryno

Maniphest Tasks: T15729

Differential Revision: https://we.phorge.it/D25560

Author: valerio-bozzolan

src/infrastructure/markup/blockrule/PhutilRemarkupCodeBlockRule.php

@@ -153,6 +153,11 @@ public function markupText($text, $children) {
       return implode("\n", $out);
     }
 
+    // The name is usually a sufficient source of information for file ext.
+    if (empty($options['lang']) && isset($options['name'])) {
+      $options['lang'] = $this->guessFilenameExtension($options['name']);
+    }
+
     if (empty($options['lang'])) {
       // If the user hasn't specified "lang=..." explicitly, try to guess the
       // language. If we fail, fall back to configured defaults.
@@ -343,4 +348,18 @@ private static function knownLanguageCodes() {
     return $map;
   }
 
+  /**
+   * Get the extension from a filename.
+   * @param  string "/path/to/something.name"
+   * @return null|string ".name"
+   */
+  private function guessFilenameExtension($name) {
+    $name = basename($name);
+    $pos = strrpos($name, '.');
+    if ($pos !== false) {
+      return substr($name, $pos + 1);
+    }
+    return null;
+  }
+
 }

src/infrastructure/markup/remarkup/__tests__/remarkup/code-block-guess-from-name.txt

@@ -0,0 +1,7 @@
+  name=/etc/phpmyadmin/config.txt
+  $lol = 1;
+~~~~~~~~~~
+<div class="remarkup-code-block" data-code-lang="txt" data-sigil="remarkup-code-block"><div class="remarkup-code-header">/etc/phpmyadmin/config.txt</div><pre class="remarkup-code">$lol = 1;</pre></div>
+~~~~~~~~~~
+name=/etc/phpmyadmin/config.txt
+  $lol = 1;