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.

CommentFileSizeAuthor
#10 drupalnet.patch23.56 KBbennybobw
#3 phone_home_only_6x_2.patch15.79 KBpwolanin
#1 phone_home_only_6x_1.patch12.22 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Version: x.y.z » 6.x-dev
Status: Active » Needs review
StatusFileSize
new phone_home_only_6x_1.patch12.22 KB

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...

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

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.

pwolanin’s picture

pwolanin commented
StatusFileSize
new phone_home_only_6x_2.patch15.79 KB

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

gopherspidey’s picture

gopherspidey commented

What ever happen to this?

dries’s picture

dries commented

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.

moshe weitzman’s picture

moshe weitzman commented

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

moshe weitzman’s picture

moshe weitzman commented

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

robertdouglass’s picture

robertdouglass commented

subscribe.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

Subscribing.

+1 for removing this from core.

bennybobw’s picture

bennybobw commented
Status: Needs review » Needs work
StatusFileSize
new drupalnet.patch23.56 KB

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.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Needs work » Closed (duplicate)

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

senpai’s picture

senpai commented
Status: Closed (duplicate) » Fixed

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

drupalorg.module is now a contrib.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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