From c102620a298ffd5e2b3ac4a2893a6966551169e1 Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Fri, 3 Mar 2017 03:41:31 -0800
Subject: [PATCH] Lock `files.video-mime-types` config option for consistency

Summary:
This is a consistency change to make this option consistent with `audio-mime-types`, `image-mime-types` and `icon-mime-types`, all of which are locked.

(They're locked because SVG is definitely dangerous, and other types might be dangerous or might become dangerous in the future, although I'm not aware of any actual dangers from video types today.)

Test Plan: Viewed `files.video-mime-types` in Config, saw it was locked.

Reviewers: chad

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D17454
---
 src/applications/files/config/PhabricatorFilesConfigOptions.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/applications/files/config/PhabricatorFilesConfigOptions.php b/src/applications/files/config/PhabricatorFilesConfigOptions.php
index b16e07c148..0f13a2fb4f 100644
--- a/src/applications/files/config/PhabricatorFilesConfigOptions.php
+++ b/src/applications/files/config/PhabricatorFilesConfigOptions.php
@@ -155,6 +155,7 @@ final class PhabricatorFilesConfigOptions
             'List of MIME types which can be rendered with an `%s` tag.',
             '<audio />')),
       $this->newOption('files.video-mime-types', 'set', $video_default)
+        ->setLocked(true)
         ->setSummary(pht('Configure which MIME types are video.'))
         ->setDescription(
           pht(