When creating a user edit context from a user object (for example on the account view page), the user_category_edit_form_from_user relationship plugin fails to either provide a default category or allow one to be set on the relationship. Instead, it requires that one is set on the user object ($context->data->user_category), and if it is not, the relationship silently fails. This property will only ever be set by page manager on user/*/edit pages, meaning that this relationship will fail everywhere else.
Attached patch lets the relationship work on all pages (defaulting to the account category), and lets you override the category in a configuration form.
Comment | File | Size | Author |
---|---|---|---|
ctools-user_edit_relationship.patch | 1.79 KB | rbayliss |
Comments
Comment #1
Bram Tassyns CreditAttribution: Bram Tassyns commentedAny reason why this hasn't been commited so far?
Comment #3
asherry CreditAttribution: asherry commentedThis works great for me, the relationship is honestly pretty useless without this patch seeing as though you could just as easily override the profile form with the core url by using pm_existing_pages.
This patch was the only way I could put it on a custom url.
I'm going to go out on a limb and set this to RTBC.
Comment #4
rbayliss CreditAttribution: rbayliss at Last Call Media commentedI just hit this bug, googled it, and was surprised to find this old issue with my patch from above. Can we get this committed? It still applies.
Comment #5
mpgeek CreditAttribution: mpgeek commentedI'm using this patch in a builds today. Really close to 3 years reviewed and teted. Did this one get missed somehow?Comment #6
mpgeek CreditAttribution: mpgeek commentedI'm using this patch in builds today. Really close to 3 years reviewed and teted. Did this one get missed somehow?
Comment #7
rivimeyOriginal patch applies cleanly to 7.x-1.x.
Patch code looks good and has been tested by multiple people.
I realize there aren't many, but I've run the patch through the existing simpletests and it passes them all.
Comment #9
japerryEasy fix. Committed.