I am experiencing a problem when try to specify the redirect page as "current".
When do this, give me a error: 'Incorrect path, Please Enter valid Path".

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lucass created an issue. See original summary.

lucass’s picture

Issue summary: View changes
lucass’s picture

Issue summary: View changes
lucass’s picture

Follow the patch to fix this problem.

izmeez’s picture

Status: Needs review » Reviewed & tested by the community

Same problem here, redirect to <current> fails with incorrect path.

Patch works perfectly, thanks.

It's a simple fix but I'm almost inclined to call the bug major since it breaks a major use for this module.

  • DmitryDrozdik committed 9ccc2e3 on 7.x-1.x
    Issue #2617156 by lucass, DmitryDrozdik: Incorrect path - <current>
    
ddrozdik’s picture

Status: Reviewed & tested by the community » Fixed
izmeez’s picture

Thanks for the commit. Do you think a new release tag is in order? 7.x-1.1 is almost three years old.

ddrozdik’s picture

Yeah, new version will be released soon. Just want to finish with a few other bugs before next release.

izmeez’s picture

Priority: Normal » Critical
Status: Fixed » Needs work

The commit #9ccc2e3 is not correct, it is not the same as the patch and with the commit enabling the module now causes a WSOD. Also on closer examination the permissions on the files are not correct.

ddrozdik’s picture

Status: Needs work » Postponed

@izmeez, I have added your patch to the new structure of the code, because your patch has been added after applying changes in other issue.
But you still can see that if you will add or destination then function will finish work.

+  $available_urls = array('<current>', '<front>');
+  if (empty($destination) || $destination_type != 0 || in_array($destination, $available_urls)) {
     return;
   }

Also I have checked module on the real sites before committing the changes, and did not saw any WSOD. Please check, that you have installed latest version of the module, and if module was installed earlier you have done update.php. If error still appear please provide log messages from your site.

izmeez’s picture

ok I will take another look and try to test it again.
Also the patch is from @lucass

Thanks.

ddrozdik’s picture

@izmeez any news?

izmeez’s picture

Status: Postponed » Closed (fixed)

Sorry to not get back earlier.
Versions 7.x-1.3 and the latest 7.x-1.x-dev work fine.
I've changed the status to Closed (fixed).
Thanks.