Closed (fixed)
Project:
Chaos Tool Suite (ctools)
Version:
7.x-1.x-dev
Component:
Page Manager
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Sep 2011 at 10:13 UTC
Updated:
11 Apr 2012 at 09:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
pcambraHere's a patch to fix this
Comment #2
merlinofchaos commentedComment #3
merlinofchaos commentedEr. Forgot to say committed. :)
Comment #5
merlinofchaos commentedI'm an idiot for committing this patch.
This needs ot be reverted.
Comment #6
pcambraWhat's wrong with this patch?
Comment #7
Letharion commentedApparently it's the cause of #1461510: General form not available on user%user/edit anymore.
Comment #8
anonThis patch will revert the one in #1
Comment #9
pcambraYeah, it will, of course but the issue in #0 will remain unfixed, no user context in an user edit form panel.
Comment #10
merlinofchaos commentedThat would be easily fixed by editing user_edit_form.inc and adding 'user', 'entity:user' to the keywords in
That would then allow the user edit form context to double as a user.
Comment #11
Letharion commentedPatch based on #10.
Comment #12
merlinofchaos commentedFantastic.
Letharion: If you properly format your commit messages (Issue #xxxxxx by yyyyy: message) then I can just use git am.
Comment #13
tim.plunkettSo the patch in #11 uses
'entity:user', whereas the patch in #1 usesentity_id:user, and neither actually work so far.I didn't spend any time debugging this, but reverting #1 and keeping #11 fixes #1461510: General form not available on user%user/edit anymore.
Comment #14
jherencia commentedI can confirm that #13 works.
But there is something broken once applied with Panels everywhere, I'm trying to figure out what's the problem but don't have enough experience with CTools architecture to find it.Maybe it would be better to open another issue but I'll do it if no one test this. It's possible that something in RC2 don't work with PE, if anyone can try user/%user/edit rendered with a PE layout it would help.Edit: Somehow now works...
Comment #15
merlinofchaos commentedSomeday this saga will be over with. Committed #13.
Comment #16
merlinofchaos commentedCan you talk more about what's broken with PE?
Comment #17
jherencia commented@merlinofchaos After clearing the caches three times it finally worked, so don't know what happened and didn't have the time to investigate further.
I use PE everyday so, in case I see the problem I open an issue, thank you.
Comment #18.0
(not verified) commentedFix image links