Feature #23589

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

Added by Soren Malling almost 9 years ago. Updated about 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 almost 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 over 8 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 about 8 years ago

  • Target version changed from 4.6.0 to 4.6.0-beta1

#4 Updated by Xavier Perseguers about 8 years ago

  • Target version deleted (4.6.0-beta1)

#5 Updated by Alexander Opitz about 6 years ago

  • Status changed from Needs Feedback to New

#6 Updated by Mathias Schreiber over 4 years ago

  • Target version set to 7.4 (Backend)

#7 Updated by Benni Mack about 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