Bug #51808

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

Added by Helmut Hummel almost 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
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

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

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
#1

Updated by Helmut Hummel almost 8 years ago

  • Status changed from New to Accepted
#2

Updated by Gerrit Code Review almost 8 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

#3

Updated by Dominik Hahn almost 8 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?

#4

Updated by Philipp Gampe almost 8 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.

#5

Updated by Gerrit Code Review almost 8 years ago

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

#6

Updated by Helmut Hummel almost 8 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?

#7

Updated by Gerrit Code Review almost 8 years ago

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

#8

Updated by Gerrit Code Review almost 8 years ago

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

#9

Updated by Gerrit Code Review almost 8 years ago

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

#10

Updated by Helmut Hummel almost 8 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)

#11

Updated by Andrea Böhm almost 8 years ago

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

Thanks Helmut!

#12

Updated by Ernesto Baschny almost 8 years ago

  • Is Regression set to Yes
#13

Updated by Ernesto Baschny almost 8 years ago

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

Updated by Gerrit Code Review almost 8 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

#15

Updated by Gerrit Code Review almost 8 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

#16

Updated by Helmut Hummel almost 8 years ago

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

Updated by Andrea Böhm almost 8 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

#18

Updated by Andrea Böhm almost 8 years ago

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

#19

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF