Bug #84777

Exception because of non existing form yaml after saving ContentElement

Added by Stefan Froemken over 1 year ago. Updated 6 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2018-04-18
Due date:
% Done:

0%

TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Sprint Focus:
Remote Sprint

Description

Hello Core-Team,

I just see that you have changed the default directory of stored form yaml files from 1:/user_upload/ to 1:/form_definitions/.
As all files in 1:/user_upload/ are read-only I have moved (not copied) all yaml files to the new location.
Of cause: All form plugins contains invalid entries now: "1:/user_upload/kontakt.yaml" (no read access).
Now I want to correct all of my form Plugins to use the new location and select: Kontaktformular (1:/form_definitions/kontakt.yaml)
When I save my plugin it will result in following error:

"Oops, an error occurred!
Object with identifier "1:/user_upload/kontakt.yaml" does not exist in storage
More information regarding this error might be available online."

The DataHandler loops through all fields before saving the record. While checking the field "categories" the RefereceIndexer was started which retrieves the "old" record from DB where we have, of cause, the old invalid yaml filename.
The problem seems to be in \TYPO3\CMS\Form\Hooks\SoftReferenceParserHook where you call retrieveFileOrFolderObject() without a try->catch construct.


Related issues

Duplicated by TYPO3 Core - Bug #84859: Exceptions from Form Configuration files are not caught Closed 2018-04-25

History

#1 Updated by Gerrit Code Review over 1 year 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/56719

#2 Updated by Ralf Zimmermann over 1 year ago

  • Status changed from Under Review to Needs Feedback
  • Assignee set to Stefan Froemken

Please abandoned this in favor of https://review.typo3.org/c/56028/

#3 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Needs Feedback to Under Review

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

#4 Updated by Gerrit Code Review over 1 year ago

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

#5 Updated by Gerrit Code Review over 1 year ago

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

#6 Updated by Gerrit Code Review over 1 year ago

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

#7 Updated by Gerrit Code Review over 1 year ago

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

#8 Updated by Gerrit Code Review over 1 year ago

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

#9 Updated by Gerrit Code Review over 1 year ago

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

#10 Updated by Benni Mack over 1 year ago

  • Duplicated by Bug #84859: Exceptions from Form Configuration files are not caught added

#11 Updated by Gerrit Code Review over 1 year ago

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

#12 Updated by Bjoern Jacob over 1 year ago

  • Sprint Focus set to On Location Sprint

#13 Updated by Bjoern Jacob over 1 year ago

  • Sprint Focus changed from On Location Sprint to Remote Sprint

#14 Updated by Patrick Gaumond over 1 year ago

  • Category changed from Form Framework to Linkvalidator

#15 Updated by Stefan Froemken over 1 year ago

"Changed to Linkvalidator" What? Why? It has nothing to to with LinkValidator.

#16 Updated by Bjoern Jacob over 1 year ago

  • Category changed from Linkvalidator to Form Framework
  • Assignee deleted (Stefan Froemken)

Why did you change the category? This belongs definitely to the form framework.

#17 Updated by Bjoern Jacob 12 months ago

  • Status changed from Under Review to New
  • Target version deleted (next-patchlevel)
  • Sprint Focus deleted (Remote Sprint)

#18 Updated by Bjoern Jacob 7 months ago

  • Sprint Focus set to Remote Sprint

#19 Updated by Ralf Zimmermann 6 months ago

  • Status changed from New to Closed

Also available in: Atom PDF