Closed (fixed)
Project:
Project
Version:
5.x-1.x-dev
Component:
Issues
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Jun 2006 at 10:25 UTC
Updated:
20 Sep 2007 at 17:55 UTC
Jump to comment: Most recent file
Comments
Comment #1
pwolanin commentedHere's a start - add some sanity to the page callbacks.
Comment #2
pwolanin commentedok, no project/issues//subscribe path since that clashes with a project...
Comment #3
dwwLet's move discussion (and any further patches) about the menu callback sanity over to http://drupal.org/node/163599.
Until that's committed, we'll call this postponed.
Thanks!
-Derek
Comment #4
dwwhttp://drupal.org/node/163599 is now committed to HEAD, so this is back to active.
Comment #5
hunmonk commentedpatch no longer applies.
couple of things from the code review:
$GLOBALS['user']usagesComment #6
hunmonk commentedcleanup for the menu callbacks, and some additional cleanup/bugfixes for the subscribe form. the subscription status should properly default to 'None' if there's no entry in the database, and i've added links to the project pages -- seems convenient to have if you want to check out a project before you decide to subscribe.
tested the subscription form extensively, and it appears to be working well. tested all the other paths briefly and they seem good as well -- all testing on postgres.
as far as the original bug report, i don't see a truly clean way to do this -- there's no way to tell currently that a user did, in fact, select any of the global options. we *could* do a slight hack, and, say, create a special nid of -1 for users who select a global option, then check for that and load all projects if it's found.
even if we did that, the upgrade path would be a nightmare, because of the aforementioned information deficiency.
Comment #7
dwwI'm confused by this patch. http://drupal.org/node/163599 ended up being a failure, and was rolled back. Most of these menu callbacks should just be replaced by project_views.inc and some default views. I don't really see what we gain by reorganizing them if we're just going to rip them all out in the near future.
Then, there's the problem that improvements to the subscription form (unrelated to menu callback sanity, IMHO) is mixed in to this patch, making it harder to ensure we're just fixing the subscribe problems.
Also, please see http://drupal.org/node/161850 for a broader perspective on this bug and possible solutions...
Setting to needs work since I'd rather this patch *just* made fixes to the subscribe form and behavior. I'm feeling bitten by attempts to reform the menu callback insanity -- I think we need a revolution, instead of incremental change. ;)
Comment #8
hunmonk commentedyou're confused by the patch b/c i was confused what the issue is here. i'd much rather close this extremely confusing thread and open two separate new issues (one for the issue described in the title, one for my subscriptions form fix. i'll open the subscription for issue right now...
Comment #9
hunmonk commentedclosing, please refer to http://drupal.org/node/177318 -- which basically restarts the issues w/o all the posting confusion.