Hello,

I've been going through the issue list and the supplied example files, and I'm wasn't sure if this has been addressed, but here goes:

Without popups, when you create a node, Drupal redirects you to the created node when you submit the node form. How do you do that with popups? I've gotten it to tell me the node's been created, it to refresh the current page, and do nothing on form submit, but I'm not sure how to get it to redirect to the created node.

Thanks,

Mark Trapp

Comments

mark trapp’s picture

mark trapp
he/him
Primary language English
commented

All right, now that I've figured out how the hook works, I'm halfway there (I think). It appears hijackDestination is part of it, but I can't seem to get popups to reload using that destination. Instead, it just loads the destination into the confirmation popup (so I see "Type: Foo has been created" followed by the node area part of the created node).

This is what I'm using for my hook:

 function foo_popups() {
  return array(
    'search/*' => array(
      'div.results-header ul.menu li a' => array(
        'hijackDestination' => FALSE,
        'width' => '600px',
        'updateMethod' => 'reload',
      ),
    ),
  );
}

The updateMethod doesn't seem to add anything to it: removing it produces the same effect.

mark trapp’s picture

mark trapp
he/him
Primary language English
commented
Category: support » feature

Finally figured it out, hopefully it'll be of use to someone else in the future. I don't think the solution is obvious: it required me looking at a few patches and reading popups.js to figure out the correct options.

Understandably, the module is still in alpha, but it'd be nice if this was a little more streamlined or documented. I think the main issue is that doneTest and updateSource are documented as being relevant only to multiform flows, but creating a node isn't a multiform (right?).

 function foo_popups() {
  return array(
    'search/*' => array(
      'div.results-header ul.menu li a' => array(
        'width' => '600px',
        'hijackDestination' => FALSE,
        'doneTest' => 'node\/\d*',
        'updateMethod' => 'reload',
        'updateSource' => 'final',
      ),
    ),
  );
}

This hook tells popups to not hijack the destination, check to see if the path returned by the form submission is equal to node/[nid], and to reload the page with the path the form submission gave (i.e. node/99).

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented

I don't see hijackDestination in the code anymore.

Kristen

mark trapp’s picture

mark trapp
he/him
Primary language English
commented

hijackDestination was added in 2.0: did you download the 1.3 release instead? Popups API is no longer maintained, and hasn't been updated since long before this request was placed.

CoreyMoore’s picture

CoreyMoore commented

This code is not working for me. I'm using Pathauto, so I am assuming that I need to use a different regexp for doneTest?

areynolds’s picture

areynolds commented

Yeah, I'm having some issues with this code too. It all makes sense according to the readme, but the popup seems to be ignoring my form's redirect path and instead displaying the popup's parent page inside of the popup. I'm assuming that it's a problem with my doneTest, so I tried dumbing it down just to see it work:

function customquestion_popups() {
  return array(
    '*' => array(
      '#ask-question a' => array(
        'width' => '600px',
        'hijackDestination' => FALSE,
        'doneTest' => '.*admin.*',
        'updateMethod' => 'reload',
        'updateSource' => 'final',
      ),
    ),
  );
}

I've set my form redirect in the submit function:

$form_state['redirect'] = 'admin';

The redirect is functioning correctly on normal pages.

Any ideas on how I'm screwing up here? Just general ignorance of regular expressions?

mark trapp’s picture

mark trapp
he/him
Primary language English
commented

I personally haven't used Popups API in at least a year, I don't know why your code isn't working. But bear in mind that I posted these code snippets in February, 2010 (Over a year ago): there's probably a good chance that the code no longer works due to the past year's worth of development.