I've had me head in Panels for the last few months so I decided to take on UR's panels integration.

The attached is only a very rough start, there may be the need for a larger rewrite to make this work correctly with Panels 3.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrf’s picture

Title: Update Panels Integration for7.x-1.x » Update Panels Integration for 7.x-1.x
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, panels-visibility-port.patch, failed testing.

Berdir’s picture

No idea what's wrong with the testbot...

--- /dev/null
+++ b/user_relationship_panels/plugins/access/ur_relationship.incundefined

Thinking about the name. I think we should start to standardize on user_relationship*s* as the module prefix, because that's the project name is and what the base module is called. And while most are the other way round right now, I think we shold name new modules correctly.

+++ b/user_relationship_panels/user_relationship_panels.infoundefined
@@ -0,0 +1,8 @@
+files[] = plugins/ur_relationship.inc

Does panels.module actually use this? Because files[] usually only has an effect if classes are inside that file and should only be used then.

Powered by Dreditor.

Shadlington’s picture

Subbing

mrf’s picture

Status: Needs work » Needs review
FileSize
9.21 KB

Updated the naming conventions for the panels submodule, and moved the implementation a little further along.

Visibility option is now loading as expected, but it still doesn't look like everything is working correctly.

Status: Needs review » Needs work

The last submitted patch, ur-panels-visibility.patch, failed testing.

Anonymous’s picture

subscribe...

hedley’s picture

Status: Needs work » Needs review
FileSize
1.19 KB

Couple of small changes, the 'core' was set to 6.x and the $form wasn't being returned in the settings form.

This sub-module seems to work for me with those edits now.

Berdir’s picture

Why not use and test mrf's patch from #5?

mrf’s picture

The rename made this way to hard to evaluate, will do that as a separate step. I think this is almost there but I know there were bugs when I last looked at this.

Status: Needs review » Needs work

The last submitted patch, user_relationships_panels_visibility-d7-port-1159638-10.patch, failed testing.

mrf’s picture

Status: Needs work » Needs review
FileSize
2.69 KB

Left in some of the remaining noise from the rename.

The missing $form return was keeping everything from working. As far as I can tell its working the same as d6 but would love some testing of this patch.

hedley’s picture

Thanks, was just figuring out what was going on with the other patches.

@Berdir - didn't realise those patches hadn't made it into dev

Will test this now

mrf’s picture

Forgot to mention that I posted #1469496: Rename user_relationships_panels_visibility to cover the renaming aspect.

hedley’s picture

Tested #12 and it is working well for me, using with the user profile page template.

One thing which I need is for the logged in user to be able to see the same thing when viewing their own profile as a user with a relationship can see. I've attached a patch which provides and option to do this, dependent on the patch in #12.

Status: Needs review » Needs work

The last submitted patch, user_relationships_panels_visibility-d7-port-1159638-15.patch, failed testing.

Berdir’s picture

@hedley

Patches always need to contain all changes, partial patches are not supported by the testbot.

Please provide a combined patch and I'll commit this asap. I'm not currently using this submodule but we can deal with additional stuff later on once the initial port is commited.

hedley’s picture

Status: Needs work » Needs review
FileSize
3.48 KB

Ok, still learning :) - here's a patch which combines #12 & #15

Berdir’s picture

Status: Needs review » Fixed

Ok, commited and pushed. Thanks for working on this.

Status: Fixed » Closed (fixed)

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