at http://drupal.org/cvs-application

The link points to http://www.gnu.org/copyleft/gpl.html which used to be the address of GPLv2, but it has been changed. It seems that the latest version of the GPL lives at that address. GPLv2 now lives at http://www.gnu.org/licenses/old-licenses/gpl-2.0.html

CommentFileSizeAuthor
#2 cvs.module.patch.txt854 bytesAjK

Comments

AjK’s picture

Project: Drupal.org site moderators » CVS integration
Version: » 5.x-1.0
Component: Broken link » User interface

Moving this issue to the CVSlog module as the string is hardcoded into that.

AjK’s picture

Status: Active » Needs review
StatusFileSize
new854 bytes

and a patch.....

gerhard killesreiter’s picture

For now, I've modified the install at drupal.org.

hunmonk’s picture

Version: 5.x-1.0 » 6.x-1.x-dev
Status: Needs review » Needs work

we need a define for this -- it shouldn't be buried in the code.

dww’s picture

Furthermore, we need a way for sites to customize this application form. It's crazy that these questions are hard-coded. But sure, a php constant for now is at least a small step in the right direction.

p.s. hunmonk, no crazy theme functions for this, ok? ;)

jpetso’s picture

In Version Control API's Account Status module, I provided a one-question default condition and error message, both of which can be customized in the admin settings form. Sites that absolutely cannot live without multiple No/Yes questions can implement hook_versioncontrol_account_status_conditions_alter() (yeah, the name is awfully long, but what can I do). Well-known example:

/**
 * Customize the list of necessary requirements that each applicant must agree
 * in order to get the application approved.
 * [left out the (obvious) parameter and return value docs]
 */
function versioncontrol_drupalorg_versioncontrol_account_status_conditions_alter(&$conditions) {
  $conditions = array(
    'maintain' => array(
      'description' => t('I want to maintain a module, theme or translation'),
      'error' => t('Your application cannot be considered: CVS accounts are only granted to users wishing to maintain a module, theme or translation.'),
    ),
    'license' => array(
      'description' => t('I will only commit code that is licensed under terms of the <a href="http://www.gnu.org/licenses/old-licenses/gpl-2.0.html">GNU public license, version 2.0</a>'),
      'error' => t('Your application cannot be considered: all contributions in the CVS repository must be licensed under the terms of the GPL.'),
    ),
  );
}

At least, that would be one way to do it. Don't know if you like it this way or not.

jpetso’s picture

On further thought, this could be made a replacement function than an altering one by placing it into a versioncontrol_account_status_local.inc file. I think I'll modify it this way.

hunmonk’s picture

Title: CVS application form links to GPLv3 » add defines for license version and link
Status: Needs work » Fixed

added the defines, deployed on d.o.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.