Comments

wjaspers’s picture

Status: Active » Needs review
FileSize
2.62 KB
wjaspers’s picture

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

Status: Needs work » Needs review
FileSize
2.94 KB
wjaspers’s picture

Status: Needs review » Needs work

Noticed some cleanup required in the patch.

andypost’s picture

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

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

Moved into Masquerade Extras.