Project

General

Profile

Actions
Copy link

Bug #66171

closed

Wizard element browser crashes with invalid identifier

Added by Xavier Perseguers over 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Michael Oehlhof
Category:
Backend API
Target version:
7 LTS
Start date:
2015-03-31
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.6
Tags:
Complexity:
hard
Is Regression:
No
Sprint Focus:

Description

Problem

In FlexForms or TCA, when you have such definition:

<path>
    <TCEforms>
        <exclude>0</exclude>
        <label>LLL:EXT:restdoc/Resources/Private/Language/locallang_db.xml:pi_flexform.path</label>
        <config>
            <type>input</type>
            <size>48</size>
            <eval>required,trim</eval>
            <wizards type="array">
                <_PADDING>2</_PADDING>
                <link type="array">
                    <type>popup</type>
                    <title>LLL:EXT:restdoc/Resources/Private/Language/locallang_db.xml:pi_flexform.path.wizard</title>
                    <icon>link_popup.gif</icon>
                    <module type="array">
                        <name>wizard_element_browser</name>
                        <urlParameters type="array">
                            <mode>wizard</mode>
                            <act>file</act>
                        </urlParameters>
                    </module>
                    <params type="array">
                        <blindLinkOptions>page,url,mail,spec,file</blindLinkOptions>
                    </params>
                    <JSopenParams>height=500,width=400,status=0,menubar=0,scrollbars=1</JSopenParams>
                </link>
            </wizards>
        </config>
    </TCEforms>
</path>

If you put an invalid identifier in the input text (such as file:1:/invalid/directory/) and try to open the wizard with , it crashes with

#1329647780: Object with identifier "1:/invalid/directory/" does not exist in storage (More information)

Expected output

Just like before FAL: the tree without any node (directory) selected and no exception at all, as this is not user friendly and may easily happen.

Files

wizard.png (50.8 KB) wizard.png Xavier Perseguers, 2015-03-31 09:41
Actions
Copy link
 #1

Updated by Xavier Perseguers over 9 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Xavier Perseguers over 9 years ago

  • Description updated (diff)
Actions
Copy link
 #3

Updated by Benni Mack over 9 years ago

  • Status changed from New to Needs Feedback

Please recheck, I fear we just fixed this one today :)

Actions
Copy link
 #4

Updated by Xavier Perseguers over 9 years ago

No, it still isn't fixed:

#1329647780: Object with identifier "1:/invalid/directory/" does not exist in storage
Actions
Copy link
 #5

Updated by Alexander Opitz over 9 years ago

  • Status changed from Needs Feedback to New
  • Target version changed from next-patchlevel to 7.5
Actions
Copy link
 #6

Updated by Benni Mack about 9 years ago

  • Target version changed from 7.5 to 7 LTS
Actions
Copy link
 #7

Updated by Michael Oehlhof almost 9 years ago

  • Status changed from New to In Progress
  • Assignee set to Michael Oehlhof
Actions
Copy link
 #8

Updated by Gerrit Code Review almost 9 years ago

  • Status changed from In Progress 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/44884

Actions
Copy link
 #9

Updated by Gerrit Code Review almost 9 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/44884

Actions
Copy link
 #10

Updated by Gerrit Code Review almost 9 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/44884

Actions
Copy link
 #11

Updated by Gerrit Code Review almost 9 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/44884

Actions
Copy link
 #12

Updated by Gerrit Code Review almost 9 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/44884

Actions
Copy link
 #13

Updated by Gerrit Code Review over 8 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/44884

Actions
Copy link
 #14

Updated by Gerrit Code Review over 8 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/44884

Actions
Copy link
 #15

Updated by Gerrit Code Review over 8 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/44884

Actions
Copy link
 #16

Updated by Gerrit Code Review over 8 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/44884

Actions
Copy link
 #17

Updated by Gerrit Code Review over 8 years ago

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

Actions
Copy link
 #18

Updated by Gerrit Code Review over 8 years ago

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

Actions
Copy link
 #19

Updated by Markus Klein over 7 years ago

  • Category set to Backend API
  • Complexity set to hard
  • Sprint Focus set to On Location Sprint
Actions
Copy link
 #20

Updated by Anja Leichsenring about 7 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions
Copy link
 #21

Updated by Gerrit Code Review almost 7 years ago

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

Actions
Copy link
 #22

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #23

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #24

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #25

Updated by Benni Mack about 6 years ago

  • Status changed from Under Review to Closed

Dear Xavier,

I tried to reproduce this issue exactly as you mentioned and I cannot reproduce this issue anymore in v8 and v9. Sorry for the long wait.

If you still can reproduce this, feel free to re-open the ticket.

Actions
Copy link

Also available in: Atom PDF