I know OG isn't ready for D6 yet, but is there a port planned for this module once it's ready?

Comments

pwolanin’s picture

planned in a general sense, but help would be appreciated.

darren.ferguson’s picture

Today we took the D5 port and made a straight port to D6.

It is attached in the tar ball below.

Should be exactly similar to the D5 distribution for the module.

For some reason cannot get it to attach so link to it is below:

http://apollo.openband.net/drupal-modules/og_mandatory_group/og_mandator...

darren.ferguson’s picture

pwolanin’s picture

can you roll as a patch?

darren.ferguson’s picture

Sure, will make it a patch against the D5 version.

darren.ferguson’s picture

StatusFileSize
new406 bytes

Patch files for .info and .module against the D5 version are attached.

darren.ferguson’s picture

Posting the module file now
Can't for some reason. module diff and info diff as well as tar ball can be found at the following:

http://apollo.openband.net/drupal-modules/og_mandatory_group/

geodaniel’s picture

Status: Active » Needs review
StatusFileSize
new3.87 KB

(changing status, attempting to attach module patch from Darren's link)

darren.ferguson’s picture

StatusFileSize
new757 bytes

Patch for fixing the old use of the url in the d6 version for the og_mandatory_groups module

darren.ferguson’s picture

StatusFileSize
new8.14 KB

Updated the patch to fix the d6 api for the drupal_mail function.

This patch includes all other patches above and works on D6.

geodaniel’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new9.16 KB

Attached is an updated patch that includes both the info and module patch files rolled into one. It applies cleanly to the DRUPAL-5 branch. Users joining the site are assigned to the group specified in the settings, as expected.

I didn't receive email notifications still, but it could be related to my local setup.

pwolanin’s picture

Status: Reviewed & tested by the community » Fixed
StatusFileSize
new10.17 KB

there were some column name changes your patch missed, I think. Committed the attached, adn making an alpha release.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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