The attached patch makes login_destination work when the path to act upon is an alias.
For quick review also inline here:

--- a/login_destination.module
+++ b/login_destination.module
@@ -128,6 +128,9 @@ function login_destination_apply_redirect() {
     if (LOGIN_COND_PAGES == $mode) {
       $paths = split("[\n\r]", $cond);
       $result = in_array($_GET['q'], $paths);
+      if (!$result) {
+        $result = in_array(drupal_get_path_alias($_GET['q']), $paths);
+      }
     } elseif (LOGIN_COND_SNIPPET == $mode) {
       $destination = variable_get('ld_url_destination', 'user');
       $result = drupal_eval('<?php '. $cond. ' ?>');

should be ready for use ;-)

CommentFileSizeAuthor
login_destination-urlalias.patch745 bytesray007

Comments

ardas’s picture

Nice addition to the module! Thanks.

I have applied patch and commited changes for HEAD and 5.0 versions. They will be available in the development release.

ardas’s picture

Status: Needs review » Fixed

Nice addition to the module! Thanks.

I have applied patch and commited changes for HEAD and 5.0 versions. They will be available in the development release.

Anonymous’s picture

Status: Fixed » Closed (fixed)