Issue #2679239 by gaurav.bajpai: Editing the MimeTypes of a file type is buggy

Author: gaurav-bajpai

file_entity.install

@@ -88,3 +88,13 @@ function file_entity_uninstall() {
   // Remove the added column to the core {file_managed} table.
   db_drop_field('file_managed', 'type');
 }
+/**
+ * Update existing bogus entries for File types Document.
+ */
+function file_entity_update_8002(&$sandbox) {
+  foreach (\Drupal::configFactory()->listAll('file_entity') as $config_name) {
+    $config = \Drupal::configFactory()->getEditable($config_name);
+    $config->set('mimetypes', str_replace("\r", "", $config->get('mimetypes')));
+    $config->save();
+  }
+}

src/Form/FileTypeForm.php

@@ -131,7 +131,7 @@ protected function copyFormValuesToEntity(EntityInterface $entity, array $form,
     // Convert multi-line string to array before copying.
     // This may be called multiple times and exectued only if it is a string.
     if (is_string($form_state->getValue('mimetypes'))) {
-      $form_state->setValue('mimetypes', explode("\n", $form_state->getValue('mimetypes')));
+      $form_state->setValue('mimetypes', explode("\n", str_replace("\r", "", $form_state->getValue('mimetypes'))));
     }
     parent::copyFormValuesToEntity($entity, $form, $form_state);
   }