Getting "Incorrect path, Please Enter valid Path" error when trying to add a new path that does work

There is no patch here so this is just "active" status. But I get this as well.

Because that isn't a valid path... maybe this will do the trick.

If you want to make this more complicated than it is you have to create a pseudo menu link to like the way that <front> is created in core, or use some third party module like menu_token...

Did you test the patch to determine it needs work?

To be more specific you need to define a menu route in the menu_router table, likely have it callback to a 404 like <nolink> or <separator> are done for special_menu_items module, which is done through the menu_links table.

I think it's overkill because it's used as a placeholder not an actual menu item, that's why I went with the easier approach here.

Hi DarrellDuane,

Adding patch that may solve your problem
but i don't think it is bug according to drupal, administration can access all pages

Looks like some upstream changes are requiring this to be re-rolled.

Here is the rerolled patch.

+++ b/login_destination.admin.inc
@@ -504,13 +504,36 @@ function login_destination_edit_form_validate($form, &$form_state) {
+    if (!url_is_external($destination)) {
+      if (!_login_destination_path_exist($destination)) {
...
+      }
     }

Any thoughts on using && instead of nesting the checks?

Hey guys,
Good Idea, but I suppose we should to do more flexible validation, and we should check, that role selected in the rule, has access to entered path. Provided patch does not check this, and exist situation that after login/logout user will be redirected to a page with forbidden access for that role.

I have prepared a patch, with validation by role. This patch validates entered path by selected role. If role does not have access to a path, then validation function return an error.

@DmitryDrozdik requested that I reroll the last patch to fix the english language in the msgs.
Here is a new version with no code logic changes from #13, just language.

I attach a word diff screenshot so we see what I changed (new words are in green).

Looks good for me.

I was still getting this error for valid aliased paths so added $destination = drupal_get_normal_path($destination); above
$item = menu_get_item($destination); which is on line 526. Seems to have sorted it out.

I am having the same issue of "Incorrect path. Please enter a valid path" even after installing 7.x-1.x-dev. I then tried what njogz did in #19 and the error then changed to Role "role name" does not have access to the path. In the first case, the path is, in fact, a valid path and in the second case, the Role does, in fact have access to the path.

Suggestions anyone?

Thanks.

@njogz and @Bill Redman please provide examples of configuration that I could reproduce this bug. I need information about used roles, used paths, and other list of settings.

@ddrozdik I just tried the module on a clean install of Drupal 7.41 and got the error. The path I am redirecting to is a basic page with the URL alias set. See screenshots for settings.

@Bill Brendan I also got the same error and still looking into it. Commenting out the foreach loop on line 537-545 removes the error and the redirection works. This probably creates a security vulnerability so maybe someone can suggest an alternative?

@njogz do not need close ticket if you still have a problem.

I will review this bug today, and will prepare a patch if it will work, we will add it to the module.

As I understood you set an alias as destination path in your example, am I right?

@njogz and @Bill Redman, I have found a problem in the previous patch. I have added needed changes. Please apply this patch and check it.

@ddrozdik, sorry for not responding to your earlier request. Fortunately, @njogz did. Anyway, your modified patch has solved the problem. I was able to add my new rule successfully. Thank you.

Patch has been committed. Thanks guys.