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
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | ignoring_suppress.patch | 439 bytes | greggles |
Comments
Comment #1
gregglesI 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
Comment #2
gregglesHeh - attach the patch...
Comment #3
creatorsdream commentedThis problem is also in the latest cvs. I actually added it and did not seem to make any difference.
Comment #4
Renirtor commentedIt works for me.
@ creatorsdream
It seems that U forgot to erase the line with the minus in the patch
Comment #5
creatorsdream commentedOkay, 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!
Comment #6
kowalke commentedThanks for the patch. It seems to work like a charm on my 4.6 system.
Comment #7
gregglesThis 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.
Comment #8
kowalke commentedSince 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.
Comment #9
kowalke commentedIgnore my last comment---something else broek flexinode. I wrote too soon.
Comment #10
sinker commentedI'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
Comment #11
whatistocome commentedPatch worked like a charm on my 4.7 system
Comment #12
Hayakawa commentedThanks Greggles. do you have plans of new pathauto version?
Comment #13
gregglesNo 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.
Comment #14
gregglesIssue 79460 has better formatted code so I'm marking this issue as a duplicate of 79460
http://drupal.org/node/79460