Download & Extend

Rename and refactor drupal module

Project:Drupal core
Version:6.x-dev
Component:drupal.module
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

The core 'drupal' module is useful, but too hard-coded to drupal.org. It is designed to enable some basic linking and exchange between a central site and associated sites, and could be used, e.g., in a network or multi-office organization where there is a need to pool certain data. But on the surface it looks like it's just a way to get listed on drupal.org.

We should rename the module to something like "network", and change the help messages etc. so that drupal.org is only an example. This would help generecize this module and move towards broader usability.

Comments

#1

Version:x.y.z» 6.x-dev
Status:active» needs review

patch attached renames functions to drupalnet (per: http://drupal.org/node/5243) and removes all authentication code.

I'm not sure how to make a patch for the file and directory renaming...

AttachmentSizeStatusTest resultOperations
phone_home_only_6x_1.patch12.22 KBIgnored: Check issue status.NoneNone

#2

I submitted a bug report more than three years ago, which dealt with the same issue. Now since here we have a patch (with the previously decided name "drupalnet.module"), the discussion will probably keep going here. I need to note that renaming Drupal module also removes the confusion that Drupal base functions have "drupal_" prefixes and the Drupal module also has "drupal_" prefixes for functions and these two namespaces are mixed.

#3

Here's a patch to add the all 3 files (.ino, .install, .module) to the modules/drupalnet based on the patch abvoe to the module file

AttachmentSizeStatusTest resultOperations
phone_home_only_6x_2.patch15.79 KBIgnored: Check issue status.NoneNone

#4

What ever happen to this?

#5

I'm not really sold on the name 'drupalnet', although it is not bad. Might just take some time getting used to so feel free to keep it for now.

Moshe was working on a related patch: http://drupal.org/node/148419. Moshe's patch should probably go in first -- as it also blocks openid.

#6

subscribe ... after this patch, we should talk about moving distributed auth out to Contrib.

#7

my patch went in. time to refocus on this one. anyone available?

#8

subscribe.

#9

Subscribing.

+1 for removing this from core.

#10

Status:needs review» needs work

This patch renames all functions, variables, etc. to drupalnet. I didn't move distributed authentication out, that's the next step. I also didn't do anything with the documentation.

What about renaming this module to the "affiliate" module instead of drupalnet?

I didn't know how to roll the patch to rename the drupal directory and files. It only creates a new directory called drupalnet with the modified files.

AttachmentSizeStatusTest resultOperations
drupalnet.patch23.56 KBIgnored: Check issue status.NoneNone

#11

Status:needs work» closed (duplicate)

See "Move drupal.module to the contributions repository" at http://drupal.org/node/178768

#12

Status:closed (duplicate)» fixed

Completely fixed by http://drupal.org/node/178768

drupalorg.module is now a contrib.

#13

Status:fixed» closed (fixed)

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

nobody click here