Project

General

Profile

Actions
Copy link

Bug #51808

closed

ProcessedFiles need to be able to do file exists in processing folder without permission checks

Added by Helmut Hummel over 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
next-patchlevel
Start date:
2013-09-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Files

Download all files
1375955684.txt (8.01 KB) 1375955684.txt Helmut Hummel, 2013-09-06 12:20
1375955684_pagemodule_dmnkhhn.txt (6.92 KB) 1375955684_pagemodule_dmnkhhn.txt Helmut Hummel, 2013-09-06 12:20
1375955684_listmodule_dmnkhhn.txt (6.06 KB) 1375955684_listmodule_dmnkhhn.txt Helmut Hummel, 2013-09-06 12:20
Uncaught_TYPO3_Exception_tx_news_sysfolder.txt (7.96 KB) Uncaught_TYPO3_Exception_tx_news_sysfolder.txt Helmut Hummel, 2013-09-06 12:20
Uncaught_1376992145.txt (4.52 KB) Uncaught_1376992145.txt Andrea Böhm, 2013-09-06 20:34
Uncaught_1314354065.txt (7.9 KB) Uncaught_1314354065.txt Andrea Böhm, 2013-09-06 20:40

Related issues 3 (0 open — 3 closed)

Related to TYPO3 Core - Task #51807: RTE MagicImage should use the FAL processing API and not storage-0ClosedStanislas Rolland2013-09-06

Actions
Related to TYPO3 Core - Bug #51747: Fatal Error after upgrading to 6.0.9 and 6.1.4 and deleting typo3temp/_processed_Closed2013-09-04

Actions
Has duplicate TYPO3 Core - Bug #51858: Fatal error on objects using uploads/tx_...... after upgrading to 6.1.4Closed2013-09-09

Actions
Actions
Copy link
 #1

Updated by Helmut Hummel over 10 years ago

  • Status changed from New to Accepted
Actions
Copy link
 #2

Updated by Gerrit Code Review over 10 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23660

Actions
Copy link
 #3

Updated by Dominik Hahn over 10 years ago

I've applied the patch but the problems I reported earlier (=> http://forge.typo3.org/issues/51747#note-41) remain.

Does this patch and patch 6 (ticket #51747) work together or do I have to reset all files prior to applying this patch?

Actions
Copy link
 #4

Updated by Philipp Gampe over 10 years ago

@Dominik please report your test results directly in gerrit. You may set the verified state to -1.
Hint: Gerrit uses the same login data as for typo3.org or forge.

Actions
Copy link
 #5

Updated by Gerrit Code Review over 10 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23660

Actions
Copy link
 #6

Updated by Helmut Hummel over 10 years ago

Dominik Hahn wrote:

I've applied the patch but the problems I reported earlier (=> http://forge.typo3.org/issues/51747#note-41) remain.

Does this patch and patch 6 (ticket #51747) work together or do I have to reset all files prior to applying this patch?

OK, read permissions to files is still required. can you check out patch set 2?

Actions
Copy link
 #7

Updated by Gerrit Code Review over 10 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23660

Actions
Copy link
 #8

Updated by Gerrit Code Review over 10 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23660

Actions
Copy link
 #9

Updated by Gerrit Code Review over 10 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23660

Actions
Copy link
 #10

Updated by Helmut Hummel over 10 years ago

OK, for anybody having problems with the current release, please try out this

https://review.typo3.org/23660 (patch set 5)

It rolls back permission restrictions for storage 0 and should solve all problems with RTE and old extensions (like tt_news)

Actions
Copy link
 #11

Updated by Andrea Böhm over 10 years ago

With this patch set editors can insert images via RTE again.

Thanks Helmut!

Actions
Copy link
 #12

Updated by Ernesto Baschny over 10 years ago

  • Is Regression set to Yes
Actions
Copy link
 #13

Updated by Ernesto Baschny over 10 years ago

  • Category changed from 1394 to File Abstraction Layer (FAL)
  • Target version set to next-patchlevel
Actions
Copy link
 #14

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/23664

Actions
Copy link
 #15

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/23665

Actions
Copy link
 #16

Updated by Helmut Hummel over 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #17

Updated by Andrea Böhm over 10 years ago

while these patches resolve the RTE images issues they introduce a minor issue in the filelist module.

It is not a mayor bug but I want to mention it:
The filelist module now shows the processed folder. Clicking on this folder leads to an exception #1376992145 (see attachted file).

Update: see next post

Actions
Copy link
 #18

Updated by Andrea Böhm over 10 years ago

Sorry - I've attached the wrong file - exception #1314354065 is the correct one.

Actions
Copy link
 #19

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF