Extended from: http://drupal.org/node/1167744

CommentFileSizeAuthor
#3 masquerade-add-context-support-1305668-3.patch2.94 KBwjaspers
#1 masquerade-add-context-module-support-1305668-1.patch2.62 KBwjaspers
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wjaspers’s picture

wjaspers CreditAttribution: wjaspers commented
Status: Active » Needs review
FileSize
masquerade-add-context-module-support-1305668-1.patch2.62 KB
wjaspers’s picture

wjaspers CreditAttribution: wjaspers commented
Status: Needs review » Needs work

Realized this support should be moved into a sub-module, despite how dated this post is. Will fix when I have time.

wjaspers’s picture

wjaspers CreditAttribution: wjaspers commented
Status: Needs work » Needs review
FileSize
masquerade-add-context-support-1305668-3.patch2.94 KB
wjaspers’s picture

wjaspers CreditAttribution: wjaspers commented
Status: Needs review » Needs work

Noticed some cleanup required in the patch.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Status: Needs work » Needs review

Looks good let's wait for other's maintainers feedback. Probably this and ctools integration could live in separate module(project) masquerade_extras for example

+++ b/masquerade_context/plugins/condition/masquerade_is_masquerading.inc
@@ -0,0 +1,26 @@
+  /** ¶
+   * For whatever reason, custom plugins' settings get ignored when the context
+   * editor saves values. ¶
+   */

trailing whitespace

wjaspers’s picture

wjaspers CreditAttribution: wjaspers commented
Status: Needs review » Closed (won't fix)

Moved into Masquerade Extras.