Project

General

Profile

Actions

Task #51807

closed

Epic #55070: Workpackages

Epic #54260: WP: FAL Missing Issues / Features / API

Story #54272: As an Editor I expect FAL to work with the RTE

RTE MagicImage should use the FAL processing API and not storage-0

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

Status:
Closed
Priority:
Should have
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2013-09-06
Due date:
% Done:

100%

Estimated time:
20.00 h
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

RTE MagicImageService relies on storage 0 and does create "magic" images by using ImageMagick functions
directly instead of using the FAL processing API.

This leads to several problems after adding restrictions to storage 0


Files


Related issues 3 (0 open3 closed)

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
Related to TYPO3 Core - Bug #51808: ProcessedFiles need to be able to do file exists in processing folder without permission checksClosed2013-09-06

Actions
Has duplicate TYPO3 Core - Task #51778: FE showing TYPO3 Exception: #1375955684: You are not allowed to access the given folder (RTE Magic Images)ClosedStanislas Rolland2013-09-05

Actions
Actions #1

Updated by Helmut Hummel over 10 years ago

  • Category changed from File Abstraction Layer (FAL) to 1394
Actions #5

Updated by Ernesto Baschny over 10 years ago

  • Target version set to next-patchlevel

This is a dupe of #51778, right, Helmut? Could we continue on the older one only?

Actions #6

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

Updated by Ernesto Baschny over 10 years ago

  • Is Regression set to Yes
Actions #8

Updated by Ernesto Baschny over 10 years ago

  • Category changed from 1394 to RTE (rtehtmlarea + ckeditor)
Actions #9

Updated by Jürgen König over 10 years ago

Thank you, this works for me. I can insert images in RTE again and open tx_news news.

The ResourceStorage.php does not work in this version. It throws the isMissing-Error mentioned in http://forge.typo3.org/issues/51747.

I used the original file and changed the three lines and it worked.

Thank you Thank you Thank you...

Actions #10

Updated by Ernesto Baschny over 10 years ago

  • Tracker changed from Bug to Task
  • Subject changed from RTE Images do not work because exceptions are thrown to RTE MagicImage should use the FAL processing API and not storage-0
  • Target version deleted (next-patchlevel)

The bug is already solved by #51808, but anyway we want to keep this open to rework rtehtmlarea to make proper use of the FAL API in future.

I changed the subject acordingly and made it a Task. I asked Stan if he could work on that.

Actions #11

Updated by Gerrit Code Review over 10 years ago

  • Status changed from Accepted to Under Review

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

Actions #12

Updated by Ernesto Baschny over 10 years ago

  • Tracker changed from Task to Bug
Actions #13

Updated by Ernesto Baschny over 10 years ago

  • Is Regression changed from Yes to No
Actions #14

Updated by Ernesto Baschny over 10 years ago

  • Tracker changed from Bug to Task
Actions #15

Updated by Ernesto Baschny over 10 years ago

  • Parent task set to #54272
Actions #16

Updated by Stanislas Rolland about 10 years ago

  • Estimated time set to 16.00 h
  • Translation missing: en.field_remaining_hours set to 16.0
Actions #17

Updated by Stanislas Rolland about 10 years ago

  • Assignee set to Stanislas Rolland
Actions #18

Updated by Gerrit Code Review about 10 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23663

Actions #19

Updated by Gerrit Code Review about 10 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23663

Actions #20

Updated by Gerrit Code Review about 10 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23663

Actions #21

Updated by Gerrit Code Review about 10 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23663

Actions #22

Updated by Gerrit Code Review about 10 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23663

Actions #23

Updated by Gerrit Code Review about 10 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23663

Actions #24

Updated by Stanislas Rolland about 10 years ago

  • % Done changed from 0 to 80
  • Estimated time changed from 16.00 h to 20.00 h
Actions #25

Updated by Gerrit Code Review about 10 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23663

Actions #26

Updated by Stanislas Rolland about 10 years ago

  • % Done changed from 80 to 100
Actions #27

Updated by Stanislas Rolland about 10 years ago

  • Status changed from Under Review to Resolved
Actions #28

Updated by Stanislas Rolland about 10 years ago

  • Estimated time changed from 20.00 h to 21.00 h
Actions #29

Updated by Stanislas Rolland about 10 years ago

  • Estimated time changed from 21.00 h to 20.00 h
Actions #30

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF