Project

General

Profile

Actions

Bug #85570

closed

removal of form-configuration is not possible if there is a recycler inside the folder-structure

Added by Markus Volkmer over 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Must have
Category:
Form Framework
Start date:
2018-07-16
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.1
Tags:
.form.yaml, recycler, delete
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

Hi,

with the current release of 8.7.17 the newly introduced ".form.yaml"-extension and the FilePersistenceSlot disallows removing a form-configuration if there is a recycler bin in your folder structure.

This use-case leads to:

#1530281202: Persisting form definition "1:/user_upload/_recycler_/testform.form.yaml" is denied

Current workflow in ext:form on FormManagerController::deleteAction is calling ResourceStorage::deleteFile through FormPersistenceManager::delete.
If there is a "nearestRecyclerFolder" deleteFile will delegate to moveFile but in filePersistenceSLot::onPreMoveFile -> assertFile there is no invocation allowed for this combination (none defined and also none allowed).

A quick (and really dirty) workaround is calling defineInvocation inside of FilePersistenceSlot::onPreFileMove - but this allows moving ".form.yaml" always and should not be used on production environment.

Possibly also a bug in 9.x - not yet tested

Actions #1

Updated by Ralf Zimmermann over 6 years ago

  • Is Regression set to Yes
Actions #2

Updated by Ralf Zimmermann over 6 years ago

  • Assignee set to Ralf Zimmermann
Actions #3

Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

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

Actions #4

Updated by Gerrit Code Review over 6 years ago

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

Actions #5

Updated by Oliver Hader over 6 years ago

  • Target version changed from 8.7.19 to 7.6.31 & 8.7.18 & 9.3.3
Actions #6

Updated by Gerrit Code Review over 6 years ago

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

Actions #7

Updated by Oliver Hader over 6 years ago

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

Updated by Gerrit Code Review over 6 years ago

  • Status changed from Resolved to Under Review

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

Actions #9

Updated by Oliver Hader over 6 years ago

  • Status changed from Under Review to Resolved
Actions #10

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF