Project

General

Profile

Actions
Copy link

Feature #23589

closed

Popup wizard doesn't close when clicking "save & close"

Added by Soren Malling over 13 years ago. Updated almost 9 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
-
Category:
-
Target version:
7.4 (Backend)
Start date:
2010-09-24
Due date:
% Done:

0%

Estimated time:
PHP Version:
5.3
Tags:
Complexity:
Sprint Focus:

Description

Using below configuration in TCA.

The popup window doesn't close, but shows the "parent element" that "popped up" the window, since it's set in the returnUrl and you can't overwrite it.

A solution might be to overwrite the returnUrl, would this be accepted as a solution? 

'categories' => array(
                        'exclude' => 0,
                                'label'   => 'LLL:EXT:mediadatabase/Resources/Private/Language/locallang_db.xml:tx_mediadatabase_domain_model_media.categories',
                                'config'  => array(
                                       'type' => 'select',
                                        'foreign_table' => 'tx_mediadatabase_domain_model_category',
                                        'size' => 10,
                                        'maxitems' => 20,
                                        'wizards' => array(
                                                '_PADDING' => 2,
                                                '_VERTICAL' => 1,
                                                'add' => Array(
                                                        'type' => 'popup',
                                                        'title' => 'LLL:EXT:tt_news/locallang_tca.xml:tt_news_cat.createNewParentCategory',
                                                        'icon' => 'add.gif',
                                                        'params' => Array(
                                                                'table'=>'tx_mediadatabase_domain_model_category',
                                                                'pid' => '###CURRENT_PID###',
                                                                'setValue' => 'append',
                                                        ),
                                                        'script' => 'wizard_add.php',
                                                        'JSopenParams' => 'height=280,width=465,status=0,menubar=0,scrollbars=1',
                                                ),
                                        ),
                                ),
                ),

(issue imported from #M15782)

Actions
Copy link
 #1

Updated by Soren Malling over 13 years ago

Can anyone confirm this, I think this needs to get in before feature freeze! Or else, the popup possibility is useless!

Actions
Copy link
 #2

Updated by Ernesto Baschny over 13 years ago

The "wizard_add.php" isn't prepared to be called as a popup. So the solution is simply not to use a popup, but to call it with type='script', as all other extensions do. e.g.:

'add' => Array(
                                                'type' => 'script',
                                                'title' => 'Create new Test',
                                                'icon' => 'add.gif',
                                                'params' => Array(
                                                        'table'=>'tx_caretaker_test',
                                                        'pid' => '###CURRENT_PID###',
                                                        'setValue' => 'prepend'
                                                ),
                                                'script' => 'wizard_add.php',
                                        ),

Having a wizard_add which works with a popup would be nice, but I don't think it is possible right now. So this is kinda like a new feature. :)

Actions
Copy link
 #3

Updated by Xavier Perseguers almost 13 years ago

  • Target version changed from 4.6.0 to 4.6.0-beta1
Actions
Copy link
 #4

Updated by Xavier Perseguers over 12 years ago

  • Target version deleted (4.6.0-beta1)
Actions
Copy link
 #5

Updated by Alexander Opitz almost 11 years ago

  • Status changed from Needs Feedback to New
Actions
Copy link
 #6

Updated by Mathias Schreiber over 9 years ago

  • Target version set to 7.4 (Backend)
Actions
Copy link
 #7

Updated by Benni Mack almost 9 years ago

  • Status changed from New to Rejected

I think that the standard wizard for adding records should not be adapted here. Instead I opt for implementing a similar script in your own extension to provide this functionality. If you have further questions on this topic, feel free to contact me.

Actions
Copy link

Also available in: Atom PDF