Had a user complain about seeing this when registering. Searched, found someone else had this problem too:
http://drupal.org/node/38267

Agreed with them that it was inappropriate. Wondered why this is being sent to the browser anyway:
drupal_set_message("Ignoring alias $dst");

Seems like it should just be commented out by default.

Mike

CommentFileSizeAuthor
#2 ignoring_suppress.patch439 bytesgreggles

Comments

greggles’s picture

Status: Active » Needs review

I don't know about commented out...but if verbose is off then I can understand why this should not be shown.

Here is a patch that attempts to check that and only displays the message if verbose is on. It's for 4.6.x version of pathauto

greggles’s picture

StatusFileSize
new439 bytes

Heh - attach the patch...

creatorsdream’s picture

This problem is also in the latest cvs. I actually added it and did not seem to make any difference.

function _pathauto_set_alias($src, $dst, $pid = NULL, $verbose = FALSE, $oldalias = NULL) {
  // Special case - don't let them screw themselves by creating
  // certain aliases
  switch ($dst) {
    case 'user':
    case 'blog':
    case 'admin':
      drupal_set_message("Ignoring alias $dst");
      if (variable_get('pathauto_verbose', FALSE) == 1):
        drupal_set_message("Ignoring alias $dst");
      endif;
    return;
    default:
  break;
  }
Renirtor’s picture

Version: 4.6.x-1.x-dev » 6.x-1.x-dev

It works for me.

@ creatorsdream

It seems that U forgot to erase the line with the minus in the patch

creatorsdream’s picture

Okay, you're right Renirtor. I thought I could just run the patch and it would fix the module. I guess I didn't have the correct version of the module because it did not delete the line that it was suppose to. So I did it manually and it seems to have worked. I'm running drupal 4.7 beta 6.

Thanks for the clarification!

kowalke’s picture

Thanks for the patch. It seems to work like a charm on my 4.6 system.

greggles’s picture

Status: Needs review » Reviewed & tested by the community

This patch still applies to the 4.7 version of pathauto and seemed to do the fix.

I contacted mikeryan about these issues and he mentioned that he is no longer interested in maintaining pathauto. He gave me CVS access. I'm moving this to RTC and will post on the devel list about issues in this queue that I am thinking of committing.

kowalke’s picture

Since applying the patch, I've been having trouble with Flexinode. See http://drupal.org/node/68634 .

If I revert to the pre-patched pathauto, Flexinode works fine again.

kowalke’s picture

Ignore my last comment---something else broek flexinode. I wrote too soon.

sinker’s picture

I'm still experiencing the "ignoring alias user" problem, even after applying the patch (manually). This is what I've plugged in:

function _pathauto_set_alias($src, $dst, $pid = NULL, $verbose = FALSE, $oldalias = NULL) {
// Specia

whatistocome’s picture

Patch worked like a charm on my 4.7 system

Hayakawa’s picture

Thanks Greggles. do you have plans of new pathauto version?

greggles’s picture

No real "plans". Just fixing bugs that I care about and reviewing/applying patches that others provide.

Also, stefan is the maintainer. I have cvs commit access and we are coordinating before committing anything.

greggles’s picture

Status: Reviewed & tested by the community » Closed (duplicate)

Issue 79460 has better formatted code so I'm marking this issue as a duplicate of 79460

http://drupal.org/node/79460