Edgeを利用してMoodle 3.2のファイルをダウンロードした際に文字化けする件

# Moodleの文字化け
Moodleはその生まれのためか所謂CJK文字への対応があまり良くなく、不具合を報告してもスルーされることがある。IE利用時の文字化けについては対応されているが、EdgeはIEとは全く異なる扱いとなっているため文字化けしてしまう。

# 状況
修正前は以下のようにダウンロード時に文字化けしてしまう。ファイル名が化けるだけならまだしも、拡張子前のピリオドが失われてしまうために文字化けについて詳しくないユーザだとファイルを開くことができなくなる場合がある（ファイル内容自体は壊れていない）。

Moodle上の表示
<img width="237" alt="キャプチャ1.PNG" src="https://qiita-image-store.s3.amazonaws.com/0/63695/4d5edf79-4b87-b4fa-fcff-842c691e58ac.png">

ダウンロード時のポップアップ
<img width="723" alt="キャプチャ2.PNG" src="https://qiita-image-store.s3.amazonaws.com/0/63695/533fbb29-b549-c0a4-9cb0-b890ac5dd500.png">

# 修正方法
## 一般的なファイルのダウンロードの場合
ファイルダウンロード時にIE用にファイル名を処理している部分が lib/filelib.php 中に3箇所あるため、その部分を全て以下のように書き換える。
※Moodle 3.4では2箇所になっている。
※Moodle 3.5.1、3.4.4で[修正された模様](https://git.moodle.org/gw?p=moodle.git;a=commitdiff;h=8e6d95b9cf9f8f74c9f7e330dc965c10ce78feff)。

変更前

``` php:lib/filelib.php
    // if user is using IE, urlencode the filename so that multibyte file name will show up correctly on popup
    if (core_useragent::is_ie()) {
        $filename = rawurlencode($filename);
    }
```

変更後

``` php:lib/filelib.php
    // if user is using IE, urlencode the filename so that multibyte file name will show up correctly on popup
    if (core_useragent::is_ie() || core_useragent::is_edge()) {
        $filename = rawurlencode($filename);
    }
```

### 動作確認
修正後に動作確認すると文字化けしなくなっている。
<img width="724" alt="キャプチャ3.PNG" src="https://qiita-image-store.s3.amazonaws.com/0/63695/937c1ac9-83b5-09ea-03f4-8c1cd6d0cebd.png">

## mod_feedbackの回答結果をダウンロードする場合
この場合は lib/classes/dataformat/spout_base.php を書き換える必要がある。

```php:lib/classes/dataformat/spout_base.php
     public function send_http_headers() {
         $this->writer = \Box\Spout\Writer\WriterFactory::create($this->spouttype);
         if (method_exists($this->writer, 'setTempFolder')) {
             $this->writer->setTempFolder(make_request_directory());
         }
         $filename = $this->filename . $this->get_extension();
+
+        // if user is using IE, urlencode the filename so that multibyte file name will show up correctly
+        if (\core_useragent::is_ie() || \core_useragent::is_edge()) {
+            $filename = rawurlencode($filename);
+        }
+
         $this->writer->openToBrowser($filename);
```

# 備考
`core_useragent::is_edge()`は lib/classes/useragent.php 中で定義されている。このメソッドは3.0系にも存在するが、2.8系には存在しない。

``` php:lib/classes/useragent.php
    /**
     * Checks the user agent is Edge (of any version).
     *
     * @return bool true if Edge
     */
    public static function is_edge() {
        return self::check_edge_version();
    }

    /**
     * Check the User Agent for the version of Edge.
     *
     * @param string|int $version A version to check for, returns true if its equal to or greater than that specified.
     * @return bool
     */
    public static function check_edge_version($version = null) {
        $useragent = self::get_user_agent_string();

        if ($useragent === false) {
            // No User Agent found.
            return false;
        }

        if (strpos($useragent, 'Edge/') === false) {
            // Edge was not found in the UA - this is not Edge.
            return false;
        }

        if (empty($version)) {
            // No version to check.
            return true;
        }

        // Find the version.
        // Edge versions are always in the format:
        //      Edge/<version>.<OS build number>
        preg_match('%Edge/([\d]+)\.(.*)$%', $useragent, $matches);

        // Just to be safe, round the version being tested.
        // Edge only uses integer versions - the second component is the OS build number.
        $version = round($version);

        // Check whether the version specified is >= the version found.
        return version_compare($matches[1], $version, '>=');
    }
```