Closed (won't fix)
Project:
Project issue tracking
Version:
5.x-2.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
1 Aug 2007 at 05:33 UTC
Updated:
26 Feb 2009 at 23:11 UTC
Jump to comment: Most recent file
Comments
Comment #1
pwolanin commentedsubscribe
Comment #2
dwwReviewed and tested heavily. All is well. Committed to HEAD. Thanks!
Comment #3
dwwUgh. This broke something that escaped my testing, but which wwwebernet reported in IRC.
If you went to project/issues, selected a given project in the dropdown and pressed "Search", it would redirect you to project/issues/user/.... :(
So, I just temporarily reverted the change on d.o. But, we need to fix this ASAP, if possible. Otherwise, I should just back out the commit from CVS entirely, but that'd be a shame, since everything else seemed to be an improvement...
Comment #4
pwolanin commentedahh, alive again...
Here's a follow-on patch.
the problem was that the default value for the argument was 0, and (0 == 'user') is TRUE (oddly enough) since PHP casts to int most strings as zero
Comment #5
dwwThanks, Peter. Even more local testing, then deployed on d.o, cleared the menu cache, and tested on the live site. As far as I can tell, everything's now happy. Phew. ;)
Comment #6
dwwNope, still not right. :( See http://drupal.org/node/163874. Ugh, this is what I get for trying to deal with dozens of bugs and problems simultaneously, I get hasty and don't do as thorough a job as usual. Alas. Reverted to the revisions before either of these patches on d.o again, pending a better patch here. I'm not going to commit and deploy onto d.o anything else in here until someone else reviews and tests it, since I'm clearly not upto my usual abilities... ;)
Comment #7
pwolanin commentedsorry - will try to get back to it.
Comment #8
hunmonk commentedthis is not a bug, but a task IMO -- and one that we can more effectively solve in 5.x-2.x
Comment #9
dwwAll this nasty is gone in D6 now that we're using views. project_issue_page() no longer exists. ;) I have no interest in cleaning up the insanity in the D5 branch.