Fixed #4382

craftcms · Jul 11, 2019 · 84d9645 · 84d9645
1 parent 3ff1333
commit 84d9645
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGELOG-v3.md b/CHANGELOG-v3.md
@@ -17,6 +17,7 @@
 - Fixed a bug where Assets fields set to restrict uploads to a single folder could have empty selector modals. ([#4522](https://github.com/craftcms/cms/issues/4522))
 - Fixed an error that could occur if a template was accessing the deprecated `locale` property of an element query, but `siteId` wasn’t set to an integer. ([#4531](https://github.com/craftcms/cms/issues/4531))
 - Fixed a bug where users without the “Publish live changes” permission for a section weren’t able to create new entries. ([#4528](https://github.com/craftcms/cms/issues/4529))
+- Fixed a PHP error that would occur when uploading a file to an Asset field using frontend. ([#4382](https://github.com/craftcms/cms/issues/4382))
 
 ## 3.2.0 - 2019-07-09
 

diff --git a/src/fields/Assets.php b/src/fields/Assets.php
@@ -578,6 +578,10 @@ private function _getUploadedFiles(ElementInterface $element): array
         /** @var Element $element */
         $uploadedFiles = [];
 
+        if (ElementHelper::rootElement($element)->getIsRevision()) {
+            return $uploadedFiles;
+        }
+
         // Grab data strings
         if (isset($this->_uploadedDataFiles['data']) && is_array($this->_uploadedDataFiles['data'])) {
             foreach ($this->_uploadedDataFiles['data'] as $index => $dataString) {