Closed (fixed)
Project:
CVS integration
Version:
6.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
21 Feb 2008 at 21:42 UTC
Updated:
16 Mar 2009 at 05:00 UTC
now that #209408: 6.x port is completed, we only have these pieces left to get the module running on 6.x completely -- all need to be tested once project/project_release are ported to 6.x:
- menu item for release node cvs access
- menu item for release node cvs access deletion
- function cvs_alter_project_project_form()
- function cvs_alter_project_release_form()
Comments
Comment #1
dww3. is working, everything else is definitely busted.
Comment #2
dwwCommitted a ton of fixes for (4).
Known bugs:
A) If you select version values for HEAD that already exist, what you already typed is lost when the form errors are displayed.
B) If the API compatibility term isn't required, page #2 for adding HEAD releases doesn't give you any real validation errors if you don't type in anything. [workaround: ensure the vocab is required, which is should be, anyway].
C) CSS for release node form isn't working exactly as we want.
Working on #1 and #2 now, since the above aren't critical for the release.
Comment #3
dwwCommitted fixes for 1. and 2. from comment #1 to HEAD.
Repurposing this issue for the remaining UI tweaks in #2.
Comment #4
dwwFixed (A) and committed yesterday.
Only (C) remains.
Comment #5
dww(C) was trivial (and an issue in project_release.css itself, not here). Committed to HEAD: http://drupal.org/cvs?commit=177876
Setting this back to the original scope of the issue and marking fixed.