[Resuelto] cred_success_redirect breaks the redirect of other forms

[Autor]

Mensajes

[Valeriia]

Hi,

I'm using the following code to redirect one of the forms:

add_filter('cred_success_redirect', 'custom_redirect', 10, 3);
function custom_redirect( $url, $post_id, $form_data ) {  
    if ($form_data['id']==49 || $form_data['id']==529) {
        $return_id = $_GET['returnid']; 
        $unit_id = $_GET['unitid']; 
        return get_permalink($return_id) . '?unitid=' . $unit_id;        
    } 
}

It works fine, but I noticed that the forms that are not listed in this snippet don't redirect according to their settings after I added this code. Removing it makes other forms redirect properly.

Is it supposed to be like that? Should I use cred_success_redirect_[form_id] for each form when I need to redirect only some of the them?

Thanks.

[Nigel]

Hi Valeria

A filter must always return the first parameter (whether it has been modified or not).

Your function only returns anything for forms 49 and 529, for other forms it returns nothing.

You just need to add a line to return $url at the end.

[Valeriia]

That's right... Thanks, working now.

[Zayne]

Could you please post the final version of this code? I'm struggling to figure out where the final

return $url

should go.

Here's my snippet:

<?php

toolset_snippet_security_check() or die( 'Direct access is not allowed' );

add_filter('cred_success_redirect_2873', 'redirect_to_parent', 10, 3);
function redirect_to_parent($url, $post_id, $form_data) {
  if ($form_data['id']==2873 || $form_data['id']==2896) {
    $parent_id = toolset_get_related_post($post_id, 'work-recording');
  	return get_permalink( $parent_id );
  }
  return $url
}