Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Status: Active » Needs review
FileSize
4.88 KB

Okay, with some help from merlinofchaos I was able to get this working.

You can now add a relationship from a user to their profile edit form.
In addition to user/%user/edit, the panel will also be loaded for user/%user_category/edit/%.

tim.plunkett’s picture

Assigned: tim.plunkett » Unassigned
Status: Needs review » Fixed

I tested this in several different scenarios, and ended up committing http://drupalcode.org/project/ctools.git/commit/d58d900

Status: Fixed » Closed (fixed)

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

Alex Bukach’s picture

Issue summary: View changes
Status: Closed (fixed) » Needs review
FileSize
651 bytes

The patch above does not send category to the profile form in fallback case. Here's a patch to fix it.

Alex Bukach’s picture

Moreover, if fallback is not the case, ctools_context_create_user_edit_form() does not use the category. Current patch fixes this as well (not sure whether in the most natural way).

Alex Bukach’s picture

Was anybody able to check the patch?

  • tim.plunkett committed b8671ef on 8.x-2.x
    Issue #1892850 by tim.plunkett: Follow-up: ensure when the context is...
  • tim.plunkett committed d58d900 on 8.x-2.x
    Issue #1892850 by tim.plunkett: Allow specific user category profile...

Status: Needs review » Needs work

The last submitted patch, 5: ctools-pass_category_to_profile_form-1892850-5.patch, failed testing.

Alex Bukach’s picture

Status: Needs work » Needs review
FileSize
688 bytes

Here's re-rolled patch for 7.x-1.x-dev. The first part of the patch in #5 (i.e. the whole patch in #4) appear to have been committed.

  • tim.plunkett committed b8671ef on 8.x-3.x
    Issue #1892850 by tim.plunkett: Follow-up: ensure when the context is...
  • tim.plunkett committed d58d900 on 8.x-3.x
    Issue #1892850 by tim.plunkett: Allow specific user category profile...
Alex Bukach’s picture

@tim.plunkett is it possible to commit the patch from #10 to 7.x? Do you have any concerns?

Alex Bukach’s picture

Re-rolled against the current dev.

Konstantin Komelin’s picture

Vote for #13. Works for me

Chris Matthews’s picture

The 3 year old patch in #13 to user_edit_form.inc applied cleanly to the latest ctools 7.x-1.x-dev and (if still applicable) probably needs another review.

Checking patch plugins/contexts/user_edit_form.inc...
Applied patch plugins/contexts/user_edit_form.inc cleanly.