The D6 version used a custom table for storing its settings while the D7 version uses variables. There needs to be an upgrade script to migrate the settings between the two versions.

CommentFileSizeAuthor
#13 autoassignrole-n1672630-13.patch2.61 KBDamienMcKenna
#11 autoassignrole-n1672630-11.patch2.61 KBDamienMcKenna
#6 autoassignrole-n1672630-6.patch2.39 KBDamienMcKenna
#3 autoassignrole-n1672630.patch2 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: After D6->D7 upgrade the settings are lost » Build D6->D7 upgrade path

Clarifying the title.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Also related: #1672704: No hook_uninstall implementation, settings remain after module removed

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
autoassignrole-n1672630.patch2 KB

This should handle all of the base settings.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

This will need to be updated to handle #971320: Administrator configures registration paths, each with customised role assignments.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Needs work

After a little testing I've realized that I need to unserialize the data before saving it, updated patch due RSN.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review
FileSize
autoassignrole-n1672630-6.patch2.39 KB

This patch fixes the problem with serialized data.

steven-spencer’s picture

steven-spencer
Location West Palm Beach Florida
CreditAttribution: steven-spencer commented

#971320: Administrator configures registration paths, each with customised role assignments.

This is a great feature in the D6 release. No progress in D7 yet?

RaulMuroc’s picture

RaulMuroc CreditAttribution: RaulMuroc commented
Status: Needs review » Active

This should be commited to a newer versio. This project didn't move since September of 2011!

Reviewed and working for me, someone else has tried this patch to confirm this?

RaulMuroc’s picture

RaulMuroc CreditAttribution: RaulMuroc commented
Status: Active » Closed (fixed)

The issue's title/objective has been long-term accomplished. So no more sense for this issue to be opened.

Anyway, the module has been marked as "Obsolote" + "abandoned". Really sad, this was a really useful module. Is there some alternative?

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Closed (fixed) » Active

The patch in #8 is still needed in order for the module to work, and it needs a new maintainer; lets leave this open for now.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
autoassignrole-n1672630-11.patch2.61 KB

Updated to use strcmp() instead of just an equality check.

Status: Needs review » Needs work

The last submitted patch, autoassignrole-n1672630-11.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review
FileSize
autoassignrole-n1672630-13.patch2.61 KB

o.O

Status: Needs review » Needs work

The last submitted patch, autoassignrole-n1672630-13.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review

#13: autoassignrole-n1672630-13.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, autoassignrole-n1672630-13.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review

Upgrading the per-page functionality will be handled in #971320: Administrator configures registration paths, each with customised role assignments, this is for the main features.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

#13: autoassignrole-n1672630-13.patch queued for re-testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Fixed

Yay, it worked! Commited.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Fixed » Closed (fixed)

Closing this to clean up the issue queue as 7.x-1.0-beta1 was released.