diff --git a/core/modules/file/file.module b/core/modules/file/file.module
index 687a62c..8262e39 100644
--- a/core/modules/file/file.module
+++ b/core/modules/file/file.module
@@ -674,8 +674,82 @@ function file_cron() {
}
/**
+ * Saves form file uploads.
+ *
+ * The files will be added to the {file_managed} table as temporary files.
+ * Temporary files are periodically cleaned. Use the 'file.usage' service to
+ * register the usage of the file which will automatically mark it as permanent.
+ *
+ * @param array $element
+ * The FAPI element whose values are being saved.
+ * @param \Drupal\Core\Form\FormStateInterface $form_state
+ * The current state of the form.
+ * @param null|int $delta
+ * (optional) The delta of the file to return the file entity.
+ * Defaults to NULL.
+ * @param int $replace
+ * (optional) The replace behavior when the destination file already exists.
+ * Possible values include:
+ * - FILE_EXISTS_REPLACE: Replace the existing file.
+ * - FILE_EXISTS_RENAME: (default) Append _{incrementing number} until the
+ * filename is unique.
+ * - FILE_EXISTS_ERROR: Do nothing and return FALSE.
+ *
+ * @return array|\Drupal\file\FileInterface|null|false
+ * An array of file entities or a single file entity if $delta != NULL. Each
+ * array element contains the file entity if the upload succeeded or FALSE if
+ * there was an error. Function returns NULL if no file was uploaded.
+ */
+function file_save_upload_from_form($element, FormStateInterface $form_state, $delta = NULL, $replace = FILE_EXISTS_RENAME) {
+ $destination = isset($element['#upload_location']) ? $element['#upload_location'] : FALSE;
+ $validators = isset($element['#upload_validators']) ? $element['#upload_validators'] : array();
+ $upload_name = implode('_', $element['#parents']);
+ $errors_before = drupal_get_messages('error');
+
+ $files = file_save_upload($upload_name, $validators, $destination, $delta, $replace);
+
+ // Get all possible new errors that are generated while trying to save the
+ // upload.
+ $new_errors = array_diff(drupal_get_messages('error'), $errors_before);
+
+ if (!empty($new_errors['error'])) {
+ $new_errors = $new_errors['error'];
+
+ if (count($new_errors) > 1) {
+ // Render multiple errors into a single message.
+ $render_array = array(
+ 'error' => array(
+ '#markup' => t('One or more files could not be uploaded.'),
+ ),
+ 'item_list' => array(
+ '#theme' => 'item_list',
+ '#items' => $new_errors,
+ ),
+ );
+ $error_message = \Drupal::service('renderer')->renderPlain($render_array);
+ }
+ else {
+ $error_message = reset($new_errors);
+ }
+
+ $form_state->setError($element, $error_message);
+
+ // Empty the old queue and put the errors back as before.
+ drupal_get_messages('error', TRUE);
+ foreach ($errors_before as $error) {
+ drupal_set_message($error, 'error');
+ }
+ }
+ return $files;
+}
+
+/**
* Saves file uploads to a new location.
*
+ * @deprecated in Drupal 8.3.0, intended to be removed in Drupal 9.0.0.
+ * This function should not be used during form validation, use
+ * file_save_upload_from_form() instead.
+ *
* The files will be added to the {file_managed} table as temporary files.
* Temporary files are periodically cleaned. Use the 'file.usage' service to
* register the usage of the file which will automatically mark it as permanent.
@@ -1192,9 +1266,8 @@ function file_managed_file_save_upload($element, FormStateInterface $form_state)
$files_uploaded = $element['#multiple'] && count(array_filter($file_upload)) > 0;
$files_uploaded |= !$element['#multiple'] && !empty($file_upload);
if ($files_uploaded) {
- if (!$files = file_save_upload($upload_name, $element['#upload_validators'], $destination)) {
+ if (!$files = file_save_upload_from_form($element, $form_state)) {
\Drupal::logger('file')->notice('The file upload failed. %upload', array('%upload' => $upload_name));
- $form_state->setError($element, t('Files in the @name field were unable to be uploaded.', array('@name' => $element['#title'])));
return array();
}
diff --git a/core/modules/file/src/Tests/SaveUploadFormTest.php b/core/modules/file/src/Tests/SaveUploadFormTest.php
new file mode 100644
index 0000000..5d7f89f
--- /dev/null
+++ b/core/modules/file/src/Tests/SaveUploadFormTest.php
@@ -0,0 +1,365 @@
+drupalCreateUser(array('access site reports'));
+ $this->drupalLogin($account);
+
+ $image_files = $this->drupalGetTestFiles('image');
+ $this->image = File::create((array) current($image_files));
+
+ list(, $this->imageExtension) = explode('.', $this->image->getFilename());
+ $this->assertTrue(is_file($this->image->getFileUri()), "The image file we're going to upload exists.");
+
+ $this->phpfile = current($this->drupalGetTestFiles('php'));
+ $this->assertTrue(is_file($this->phpfile->uri), 'The PHP file we are going to upload exists.');
+
+ $this->maxFidBefore = db_query('SELECT MAX(fid) AS fid FROM {file_managed}')->fetchField();
+
+ // Upload with replace to guarantee there's something there.
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_REPLACE,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri()),
+ );
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called then clean out the hook
+ // counters.
+ $this->assertFileHooksCalled(array('validate', 'insert'));
+ file_test_reset();
+ }
+
+ /**
+ * Test the file_save_upload() function.
+ */
+ public function testNormal() {
+ $max_fid_after = db_query('SELECT MAX(fid) AS fid FROM {file_managed}')->fetchField();
+ $this->assertTrue($max_fid_after > $this->maxFidBefore, 'A new file was created.');
+ $file1 = File::load($max_fid_after);
+ $this->assertTrue($file1, 'Loaded the file.');
+ // MIME type of the uploaded image may be either image/jpeg or image/png.
+ $this->assertEqual(substr($file1->getMimeType(), 0, 5), 'image', 'A MIME type was set.');
+
+ // Reset the hook counters to get rid of the 'load' we just called.
+ file_test_reset();
+
+ // Upload a second file.
+ $image2 = current($this->drupalGetTestFiles('image'));
+ $edit = array('files[file_test_upload]' => drupal_realpath($image2->uri));
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('You WIN!'));
+ $max_fid_after = db_query('SELECT MAX(fid) AS fid FROM {file_managed}')->fetchField();
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'insert'));
+
+ $file2 = File::load($max_fid_after);
+ $this->assertTrue($file2, 'Loaded the file');
+ // MIME type of the uploaded image may be either image/jpeg or image/png.
+ $this->assertEqual(substr($file2->getMimeType(), 0, 5), 'image', 'A MIME type was set.');
+
+ // Load both files using File::loadMultiple().
+ $files = File::loadMultiple(array($file1->id(), $file2->id()));
+ $this->assertTrue(isset($files[$file1->id()]), 'File was loaded successfully');
+ $this->assertTrue(isset($files[$file2->id()]), 'File was loaded successfully');
+
+ // Upload a third file to a subdirectory.
+ $image3 = current($this->drupalGetTestFiles('image'));
+ $image3_realpath = drupal_realpath($image3->uri);
+ $dir = $this->randomMachineName();
+ $edit = array(
+ 'files[file_test_upload]' => $image3_realpath,
+ 'file_subdir' => $dir,
+ );
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('You WIN!'));
+ $this->assertTrue(is_file('temporary://' . $dir . '/' . trim(drupal_basename($image3_realpath))));
+ }
+
+ /**
+ * Test extension handling.
+ */
+ public function testHandleExtension() {
+ // The file being tested is a .gif which is in the default safe list
+ // of extensions to allow when the extension validator isn't used. This is
+ // implicitly tested at the testNormal() test. Here we tell
+ // file_save_upload() to only allow ".foo".
+ $extensions = 'foo';
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_REPLACE,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri()),
+ 'extensions' => $extensions,
+ );
+
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $message = t('Only files with the following extensions are allowed:') . ' ' . $extensions . '';
+ $this->assertRaw($message, 'Cannot upload a disallowed extension');
+ $this->assertRaw(t('Epic upload FAIL!'), 'Found the failure message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate'));
+
+ // Reset the hook counters.
+ file_test_reset();
+
+ $extensions = 'foo ' . $this->imageExtension;
+ // Now tell file_save_upload() to allow the extension of our test image.
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_REPLACE,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri()),
+ 'extensions' => $extensions,
+ );
+
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertNoRaw(t('Only files with the following extensions are allowed:'), 'Can upload an allowed extension.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'load', 'update'));
+
+ // Reset the hook counters.
+ file_test_reset();
+
+ // Now tell file_save_upload() to allow any extension.
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_REPLACE,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri()),
+ 'allow_all_extensions' => TRUE,
+ );
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertNoRaw(t('Only files with the following extensions are allowed:'), 'Can upload any extension.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'load', 'update'));
+ }
+
+ /**
+ * Test dangerous file handling.
+ */
+ public function testHandleDangerousFile() {
+ $config = $this->config('system.file');
+ // Allow the .php extension and make sure it gets renamed to .txt for
+ // safety. Also check to make sure its MIME type was changed.
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_REPLACE,
+ 'files[file_test_upload]' => drupal_realpath($this->phpfile->uri),
+ 'is_image_file' => FALSE,
+ 'extensions' => 'php',
+ );
+
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $message = t('For security reasons, your upload has been renamed to') . ' ' . $this->phpfile->filename . '.txt' . '';
+ $this->assertRaw($message, 'Dangerous file was renamed.');
+ $this->assertRaw(t('File MIME type is text/plain.'), "Dangerous file's MIME type was changed.");
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'insert'));
+
+ // Ensure dangerous files are not renamed when insecure uploads is TRUE.
+ // Turn on insecure uploads.
+ $config->set('allow_insecure_uploads', 1)->save();
+ // Reset the hook counters.
+ file_test_reset();
+
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertNoRaw(t('For security reasons, your upload has been renamed'), 'Found no security message.');
+ $this->assertRaw(t('File name is @filename', array('@filename' => $this->phpfile->filename)), 'Dangerous file was not renamed when insecure uploads is TRUE.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'insert'));
+
+ // Turn off insecure uploads.
+ $config->set('allow_insecure_uploads', 0)->save();
+ }
+
+ /**
+ * Test file munge handling.
+ */
+ public function testHandleFileMunge() {
+ // Ensure insecure uploads are disabled for this test.
+ $this->config('system.file')->set('allow_insecure_uploads', 0)->save();
+ $this->image = file_move($this->image, $this->image->getFileUri() . '.foo.' . $this->imageExtension);
+
+ // Reset the hook counters to get rid of the 'move' we just called.
+ file_test_reset();
+
+ $extensions = $this->imageExtension;
+ $edit = array(
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri()),
+ 'extensions' => $extensions,
+ );
+
+ $munged_filename = $this->image->getFilename();
+ $munged_filename = substr($munged_filename, 0, strrpos($munged_filename, '.'));
+ $munged_filename .= '_.' . $this->imageExtension;
+
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('For security reasons, your upload has been renamed'), 'Found security message.');
+ $this->assertRaw(t('File name is @filename', array('@filename' => $munged_filename)), 'File was successfully munged.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'insert'));
+
+ // Ensure we don't munge files if we're allowing any extension.
+ // Reset the hook counters.
+ file_test_reset();
+
+ $edit = array(
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri()),
+ 'allow_all_extensions' => TRUE,
+ );
+
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertNoRaw(t('For security reasons, your upload has been renamed'), 'Found no security message.');
+ $this->assertRaw(t('File name is @filename', array('@filename' => $this->image->getFilename())), 'File was not munged when allowing any extension.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'insert'));
+ }
+
+ /**
+ * Test renaming when uploading over a file that already exists.
+ */
+ public function testExistingRename() {
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_RENAME,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri())
+ );
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'insert'));
+ }
+
+ /**
+ * Test replacement when uploading over a file that already exists.
+ */
+ public function testExistingReplace() {
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_REPLACE,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri())
+ );
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('You WIN!'), 'Found the success message.');
+
+ // Check that the correct hooks were called.
+ $this->assertFileHooksCalled(array('validate', 'load', 'update'));
+ }
+
+ /**
+ * Test for failure when uploading over a file that already exists.
+ */
+ public function testExistingError() {
+ $edit = array(
+ 'file_test_replace' => FILE_EXISTS_ERROR,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri())
+ );
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('Epic upload FAIL!'), 'Found the failure message.');
+
+ // Check that the no hooks were called while failing.
+ $this->assertFileHooksCalled(array());
+ }
+
+ /**
+ * Test for no failures when not uploading a file.
+ */
+ public function testNoUpload() {
+ $this->drupalPostForm('file-test/save_upload_from_form_test', array(), t('Submit'));
+ $this->assertNoRaw(t('Epic upload FAIL!'), 'Failure message not found.');
+ }
+
+ /**
+ * Tests for log entry on failing destination.
+ */
+ public function testDrupalMovingUploadedFileError() {
+ // Create a directory and make it not writable.
+ $test_directory = 'test_drupal_move_uploaded_file_fail';
+ drupal_mkdir('temporary://' . $test_directory, 0000);
+ $this->assertTrue(is_dir('temporary://' . $test_directory));
+
+ $edit = array(
+ 'file_subdir' => $test_directory,
+ 'files[file_test_upload]' => drupal_realpath($this->image->getFileUri())
+ );
+
+ \Drupal::state()->set('file_test.disable_error_collection', TRUE);
+ $this->drupalPostForm('file-test/save_upload_from_form_test', $edit, t('Submit'));
+ $this->assertResponse(200, 'Received a 200 response for posted test file.');
+ $this->assertRaw(t('File upload error. Could not move uploaded file.'), 'Found the failure message.');
+ $this->assertRaw(t('Epic upload FAIL!'), 'Found the failure message.');
+
+ // Uploading failed. Now check the log.
+ $this->drupalGet('admin/reports/dblog');
+ $this->assertResponse(200);
+ $this->assertRaw(t('Upload error. Could not move uploaded file @file to destination @destination.', array(
+ '@file' => $this->image->getFilename(),
+ '@destination' => 'temporary://' . $test_directory . '/' . $this->image->getFilename()
+ )), 'Found upload error log entry.');
+ }
+
+}
diff --git a/core/modules/file/tests/file_test/file_test.routing.yml b/core/modules/file/tests/file_test/file_test.routing.yml
index cdbf08e..1c5a763 100644
--- a/core/modules/file/tests/file_test/file_test.routing.yml
+++ b/core/modules/file/tests/file_test/file_test.routing.yml
@@ -4,3 +4,9 @@ file.test:
_form: 'Drupal\file_test\Form\FileTestForm'
requirements:
_access: 'TRUE'
+file.save_upload_from_form_test:
+ path: '/file-test/save_upload_from_form_test'
+ defaults:
+ _form: 'Drupal\file_test\Form\FileTestSaveUploadFromForm'
+ requirements:
+ _access: 'TRUE'
diff --git a/core/modules/file/tests/file_test/src/Form/FileTestSaveUploadFromForm.php b/core/modules/file/tests/file_test/src/Form/FileTestSaveUploadFromForm.php
new file mode 100644
index 0000000..8aa39b4
--- /dev/null
+++ b/core/modules/file/tests/file_test/src/Form/FileTestSaveUploadFromForm.php
@@ -0,0 +1,124 @@
+ 'file',
+ '#title' => t('Upload a file'),
+ );
+ $form['file_test_replace'] = array(
+ '#type' => 'select',
+ '#title' => t('Replace existing image'),
+ '#options' => array(
+ FILE_EXISTS_RENAME => t('Appends number until name is unique'),
+ FILE_EXISTS_REPLACE => t('Replace the existing file'),
+ FILE_EXISTS_ERROR => t('Fail with an error'),
+ ),
+ '#default_value' => FILE_EXISTS_RENAME,
+ );
+ $form['file_subdir'] = array(
+ '#type' => 'textfield',
+ '#title' => t('Subdirectory for test file'),
+ '#default_value' => '',
+ );
+
+ $form['extensions'] = array(
+ '#type' => 'textfield',
+ '#title' => t('Allowed extensions.'),
+ '#default_value' => '',
+ );
+
+ $form['allow_all_extensions'] = array(
+ '#type' => 'checkbox',
+ '#title' => t('Allow all extensions?'),
+ '#default_value' => FALSE,
+ );
+
+ $form['is_image_file'] = array(
+ '#type' => 'checkbox',
+ '#title' => t('Is this an image file?'),
+ '#default_value' => TRUE,
+ );
+
+ $form['submit'] = array(
+ '#type' => 'submit',
+ '#value' => t('Submit'),
+ );
+ return $form;
+ }
+
+ /**
+ * {@inheritdoc}
+ */
+ public function validateForm(array &$form, FormStateInterface $form_state) {
+ // Process the upload and perform validation. Note: we're using the
+ // form value for the $replace parameter.
+ if (!$form_state->isValueEmpty('file_subdir')) {
+ $destination = 'temporary://' . $form_state->getValue('file_subdir');
+ file_prepare_directory($destination, FILE_CREATE_DIRECTORY);
+ }
+ else {
+ $destination = FALSE;
+ }
+
+ // Setup validators.
+ $validators = array();
+ if ($form_state->getValue('is_image_file')) {
+ $validators['file_validate_is_image'] = array();
+ }
+
+ if ($form_state->getValue('allow_all_extensions')) {
+ $validators['file_validate_extensions'] = array();
+ }
+ elseif (!$form_state->isValueEmpty('extensions')) {
+ $validators['file_validate_extensions'] = array($form_state->getValue('extensions'));
+ }
+
+ // The test for drupal_move_uploaded_file() triggering a warning is
+ // unavoidable. We're interested in what happens afterwards in
+ // file_save_upload().
+ if (\Drupal::state()->get('file_test.disable_error_collection')) {
+ define('SIMPLETEST_COLLECT_ERRORS', FALSE);
+ }
+
+ $form['file_test_upload']['#upload_validators'] = $validators;
+ $form['file_test_upload']['#upload_location'] = $destination;
+
+ $file = file_save_upload_from_form($form['file_test_upload'], $form_state, 0, $form_state->getValue('file_test_replace'));
+ if ($file) {
+ $form_state->setValue('file_test_upload', $file);
+ drupal_set_message(t('File @filepath was uploaded.', array('@filepath' => $file->getFileUri())));
+ drupal_set_message(t('File name is @filename.', array('@filename' => $file->getFilename())));
+ drupal_set_message(t('File MIME type is @mimetype.', array('@mimetype' => $file->getMimeType())));
+ drupal_set_message(t('You WIN!'));
+ }
+ elseif ($file === FALSE) {
+ drupal_set_message(t('Epic upload FAIL!'), 'error');
+ }
+ }
+
+ /**
+ * {@inheritdoc}
+ */
+ public function submitForm(array &$form, FormStateInterface $form_state) { }
+
+}
diff --git a/core/modules/locale/src/Form/ImportForm.php b/core/modules/locale/src/Form/ImportForm.php
index 0ea7180..60b2d50 100644
--- a/core/modules/locale/src/Form/ImportForm.php
+++ b/core/modules/locale/src/Form/ImportForm.php
@@ -108,6 +108,7 @@ public function buildForm(array $form, FormStateInterface $form_state) {
),
'#size' => 50,
'#upload_validators' => $validators,
+ '#upload_location' => 'translations://',
'#attributes' => array('class' => array('file-import-input')),
);
$form['langcode'] = array(
@@ -154,7 +155,7 @@ public function buildForm(array $form, FormStateInterface $form_state) {
* {@inheritdoc}
*/
public function validateForm(array &$form, FormStateInterface $form_state) {
- $this->file = file_save_upload('file', $form['file']['#upload_validators'], 'translations://', 0);
+ $this->file = file_save_upload_from_form($form['file'], $form_state, 0);
// Ensure we have the file uploaded.
if (!$this->file) {
diff --git a/core/modules/system/src/Form/ThemeSettingsForm.php b/core/modules/system/src/Form/ThemeSettingsForm.php
index 37b5ea1..5dbc411 100644
--- a/core/modules/system/src/Form/ThemeSettingsForm.php
+++ b/core/modules/system/src/Form/ThemeSettingsForm.php
@@ -212,7 +212,10 @@ public function buildForm(array $form, FormStateInterface $form_state, $theme =
'#type' => 'file',
'#title' => t('Upload logo image'),
'#maxlength' => 40,
- '#description' => t("If you don't have direct file access to the server, use this field to upload your logo.")
+ '#description' => t("If you don't have direct file access to the server, use this field to upload your logo."),
+ '#upload_validators' => array(
+ 'file_validate_is_image' => array(),
+ ),
);
}
@@ -252,7 +255,12 @@ public function buildForm(array $form, FormStateInterface $form_state, $theme =
$form['favicon']['settings']['favicon_upload'] = array(
'#type' => 'file',
'#title' => t('Upload favicon image'),
- '#description' => t("If you don't have direct file access to the server, use this field to upload your shortcut icon.")
+ '#description' => t("If you don't have direct file access to the server, use this field to upload your shortcut icon."),
+ '#upload_validators' => array(
+ 'file_validate_extensions' => array(
+ 'ico png gif jpg jpeg apng svg',
+ ),
+ ),
);
}
@@ -355,37 +363,18 @@ public function validateForm(array &$form, FormStateInterface $form_state) {
parent::validateForm($form, $form_state);
if ($this->moduleHandler->moduleExists('file')) {
- // Handle file uploads.
- $validators = array('file_validate_is_image' => array());
-
// Check for a new uploaded logo.
- $file = file_save_upload('logo_upload', $validators, FALSE, 0);
- if (isset($file)) {
- // File upload was attempted.
- if ($file) {
- // Put the temporary file in form_values so we can save it on submit.
- $form_state->setValue('logo_upload', $file);
- }
- else {
- // File upload failed.
- $form_state->setErrorByName('logo_upload', $this->t('The logo could not be uploaded.'));
- }
+ $file = file_save_upload_from_form($form['logo']['settings']['logo_upload'], $form_state, 0);
+ if ($file) {
+ // Put the temporary file in form_values so we can save it on submit.
+ $form_state->setValue('logo_upload', $file);
}
- $validators = array('file_validate_extensions' => array('ico png gif jpg jpeg apng svg'));
-
// Check for a new uploaded favicon.
- $file = file_save_upload('favicon_upload', $validators, FALSE, 0);
- if (isset($file)) {
- // File upload was attempted.
- if ($file) {
- // Put the temporary file in form_values so we can save it on submit.
- $form_state->setValue('favicon_upload', $file);
- }
- else {
- // File upload failed.
- $form_state->setErrorByName('favicon_upload', $this->t('The favicon could not be uploaded.'));
- }
+ $file = file_save_upload_from_form($form['favicon']['settings']['favicon_upload'], $form_state, 0);
+ if ($file) {
+ // Put the temporary file in form_values so we can save it on submit.
+ $form_state->setValue('favicon_upload', $file);
}
// When intending to use the default logo, unset the logo_path.