Project

General

Profile

Actions

Bug #58330

closed

No FolderSelection anymore after creating a new Folder in browse_links.php?mode= wizard with 'blindlinkOptions' in TCA

Added by Alexander Kontos over 10 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2014-04-29
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Scenario:
  • Typo3 6.2.1
  • own extension using TCA with only Files being allowed to be added (tried --> 'blindLinkOptions' => 'mail, page, spec, url,media' or 'blindLinkOptions' => 'folder,mail, page, spec, url,media'
  • Firefox 28.0

After adding a new Folder the popupwindow reloads correct but you can't access any folder anymore. Besides this fact all the possibilities that were written in blindLinkOptions are visible but still not accessible.


Files

testissue_0.0.1_201404291758.zip (12.2 KB) testissue_0.0.1_201404291758.zip Alexander Kontos, 2014-04-29 20:00
Actions #1

Updated by Markus Klein over 10 years ago

Do I understand that correctly that these are 2 issues?
One with the blindLinkOptions and one with the folder?

If so please create a separate issue for the blindLinkOptions problem.
I'll rephrase this one then to only contain the folder problem.
Thank you.

Actions #2

Updated by Alexander Kontos over 10 years ago

hi,
no blindlinkoptions works as expected. The first load of the Popup is absolutely correct.
Only after uploading a file or creating a folder and after the reload of the window the folder/files show up, but you can't choose a file or a folder and all previously with blindlinkoptions removed possibilities show up.
i thank you!

I Should post the hole TCA:
'product_download' => array(
'exclude' => 0,
'label' => 'LLL:EXT:product/Resources/Private/Language/locallang.xlf:tca.product_download',
'config' => array(
'type' => 'input',
'size' => '50',
'max' => '256',
'eval' => 'trim',
'wizards' => array(
'_PADDING' => 2,
'link' => array(
'type' => 'popup',
'title' => 'LLL:EXT:cms/locallang_ttc.xml:header_link_formlabel',
'icon' => 'link_popup.gif',
'script' => 'browse_links.php?mode=wizard',
'params' => array(
'blindLinkOptions' => 'mail, page, spec, url,media'
),
'JSopenParams' => 'height=300,width=500,status=0,menubar=0,scrollbars=1'
)
),
'softref' => 'typolink'
)
),

when i use another Setting like suggested for TCA Type:'group' --> http://docs.typo3.org/typo3cms/TCAReference/Reference/Columns/Group/Index.html#columns-group-examples-fal
everything works perfect in the popup

Actions #3

Updated by Markus Klein over 10 years ago

Do you have a test extension at hand that runs into this bug?
(If you can provide a small test-extension here, reviews will be much happier as they don't need to create a demo ext on their own)

Actions #4

Updated by Alexander Kontos over 10 years ago

very good idea. i am very new in reporting. Any hint is welcome. I really appreciate your will to help and want to make it as easy as possible. i will write a small extension later with this and load it up here.

greetz

Actions #5

Updated by Markus Klein over 10 years ago

Thank you!

Actions #6

Updated by Alexander Kontos over 10 years ago

quick and dirty. i hope it is ok like that
greetz

Actions #7

Updated by Markus Klein over 10 years ago

Ok first some feedback to your extension.

I highly recommend to check out state of the art extension architecture. TCA is usually now placed in Configuration/TCA/<table>.php
Overrides are placed in Configuration/TCA/Overrides/<table>.php
Checkout some of the sysexts

Furthermore your ext_emconf.php holds lots of old elements and misses the important ones. dependencies array for instance

Regarding the bug. I investigated this. My question: Has this ever been working in 6.x? Has it been working in 4.x?

Actions #8

Updated by Alexander Kontos over 10 years ago

Thank you for your reply to my extension. i will keep it in mind.

to your question:
good question. i never tried it because i never ran into this special need. For the new Project i directly used ver 6.2. But shouldn't it work anyways?

Actions #9

Updated by Markus Klein over 10 years ago

Of course this should work. I just wanted to find out if this is a regression to some patch or not.

The problem is in ElementBrowser::createFolder() line 2526 and ElementBrowser::uploadForm() line 2462.
Both places fail to forward most of the GET parameters necessary to the hidden "redirect" form field.

Actions #10

Updated by Markus Klein over 10 years ago

  • Category changed from Form Framework to Backend User Interface
  • Status changed from New to Accepted
Actions #11

Updated by Markus Klein over 10 years ago

First investigation tells me that the GET parameters "formName" and "itemName" are missing, but maybe there's more.

Actions #12

Updated by Alexander Kontos over 10 years ago

would it be helpful if i try it in ver 6.x and 4.x to spare you some time?

Actions #13

Updated by Markus Klein over 10 years ago

No thanks. I already found out it never worked in 6.x and I don't care about 4.x in this case. So we have to find out which additional parameters need to passed.

Actions #14

Updated by Mathias Schreiber about 9 years ago

  • Status changed from Accepted to Closed

works on master

Actions

Also available in: Atom PDF