Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rburgundy’s picture

subscribing

rickvug’s picture

Issue tags: +CTools exportables

Adding the obligatory cTools Exportables tag.

okokokok’s picture

*bump*

I really want to be able to export user relationships through Features.

mrf’s picture

Status: Active » Postponed

I am postponing any 6.x feature requests that don't have a current applicable patch to help sort the large list of open features.

Please feel free to set to 'needs review' once a patch is included.

hefox’s picture

Title: Relationship Exportables » Features support to export user_relationships
Version: 6.x-1.0-beta10 » 7.x-1.x-dev
Status: Postponed » Needs review
FileSize
6.29 KB

Very basic. Easier to do features than ctools (and some project I've seen try to adept ctools to existing ended up with features).

Adds a machine name to free up name to be changed without effecting the feature export; also may want to later consider making name non-unique so could use the same name for different situations/permissions (unno).

Status: Needs review » Needs work

The last submitted patch, user_relationship-691034-features.patch, failed testing.

hefox’s picture

bug: Need to add dependency to features (Assuming I'll find other bugs so will update patch later).

DuaelFr’s picture

Is it always planned ?

mrded’s picture

Component: User interface » Code
Priority: Normal » Major
Status: Needs work » Needs review
FileSize
3.75 KB

Please, check out my patch.

mrded’s picture

New patch with machine name support.

Apply #1980610: Add machine name to user_relationships first.

Status: Needs review » Needs work

The last submitted patch, user_relationships-691034-10.patch, failed testing.

mrded’s picture

Update.
Once again, this patch required #1980610: Add machine name to user_relationships

mrded’s picture

Status: Needs work » Needs review
mrded’s picture

Priority: Major » Critical

Please! any feedback?

Andre-B’s picture

#12 just tested it - works like a charm! thanks a lot for that. I wont change the status here because I am not sure I covered everything

jhedstrom’s picture

Patch in #12 worked, but I needed to make this slight adjustment in order for it to work during a site install (the unset($relationship->rtid); was breaking the menu hook which requires the $rtid variable.

jhedstrom’s picture

Andre-B’s picture

applying #1980610: Add machine name to user_relationships first and https://drupal.org/comment/7998431#comment-7998431 afterwards works for me. I had to delete all of my old permission exports manually from the feature and database before I was able to create a new working export though.

Andre-B’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
heylookalive’s picture

Nudge, can we revive this and get it committed please?

heylookalive’s picture

heylookalive’s picture

Status: Reviewed & tested by the community » Fixed

With the machine name issue in, the patch in #17 has been committed.

@andre-b the permissions thing is an annoyance but one that people will need to go through should they wish to upgrade to the upcoming alpha6 release. We'll need to make this very clear in the release notes.

@jhedstrom, @mrded thanks again for your work.

mrded’s picture

Always welcome :)

Status: Fixed » Closed (fixed)

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