Feature #23589

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

Added by Soren Malling about 9 years ago. Updated over 4 years ago.

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

0%

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)

History

#1 Updated by Soren Malling about 9 years ago

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

#2 Updated by Ernesto Baschny almost 9 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. :)

#3 Updated by Xavier Perseguers over 8 years ago

  • Target version changed from 4.6.0 to 4.6.0-beta1

#4 Updated by Xavier Perseguers over 8 years ago

  • Target version deleted (4.6.0-beta1)

#5 Updated by Alexander Opitz over 6 years ago

  • Status changed from Needs Feedback to New

#6 Updated by Mathias Schreiber almost 5 years ago

  • Target version set to 7.4 (Backend)

#7 Updated by Benni Mack over 4 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.

Also available in: Atom PDF