Bug #44585
closedFAL: file properties are not updated
100%
Description
Hello Core-Team,
I have my own FAL-Extension: fal_dropbox: https://github.com/froemken/fal_dropbox/
I create (NOT upload!) a new empty file called: hallo.txt
The new file will be created and listed perfectly. The size of this file is 0 bytes. This information is also added to sys_file.
Now I edit this file and write something in there and save this file.
The file was saved successfully and we return to our filelist.
BUT: The filesize keeps being 0 bytes.
After a long time of debugging I found out following:
In File.php -> Method: mergeIndexRecord() you have:
$this->properties = array_merge($this->properties, $recordData);
In $this->properties we have the correct values (bytes = 141)
In $recordData we have the old values from database.
After merging, the correct values are overwritten with the old values from the database.
Changing the values soved the problem:
$this->properties = array_merge($recordData, $this->properties);
Stefan
Updated by Gerrit Code Review almost 12 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17569
Updated by Stefan Froemken almost 12 years ago
Funny:
Switching "extended view" to on: filesize is wrong
Switching "extended view" to off: filesize is correct
Updated by Gerrit Code Review almost 12 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17569
Updated by Camelia M almost 12 years ago
patch 2 works fine for me too on version 6.0.1
Updated by Benni Mack almost 12 years ago
Could you send it to the 6.0 branch as well?
Updated by Stefan Froemken almost 12 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 7d799fb3b6d6844095f68ec5ac385937fbb0b614.
Updated by Gerrit Code Review almost 12 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/18212
Updated by Stefan Froemken almost 12 years ago
- Status changed from Under Review to Resolved
Applied in changeset 90da916c4eb0003d9b7f16e91975c908739517ce.
Updated by Gerrit Code Review over 11 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch master_new has been pushed to the review server.
It is available at https://review.typo3.org/18633
Updated by Christian Kuhn over 11 years ago
- Status changed from Under Review to Resolved