the pages contain nothing
same happen with latest dev version

CommentFileSizeAuthor
#3 role_change_notify.gif26.36 KBGiorgosK
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rfay’s picture

Status: Active » Postponed (maintainer needs more info)

The relevant configurations are on admin/build/trigger/user. Please take a look on the "user" tab.

It may be that I just need you to explain more carefully what you did and how you got there.

That said, I'm not sure many people have tested this, so your attention to it is appreciated.

rfay’s picture

Is it possible you're saying that there are no actions available to configure under "Trigger: When a role is added to an account" on admin/build/trigger/user?

Please enable the token_actions module, which will provide some useful actions.

GiorgosK’s picture

Status: Postponed (maintainer needs more info) » Active
FileSize
26.36 KB

on the the triggers page there is two extra tabs which have no label
and lead to admin/build/trigger/role_removed
and admin/build/trigger/role_added
which contain nothing

take a look at the screenshot

on admin/build/trigger/user there does not exist any available actions even though token_actions are enabled

rfay’s picture

That's mighty strange. I can't duplicate that.

What browser are you using? Would it be possible to send me a database dump? randy at randyfay.com

GiorgosK’s picture

Status: Active » Postponed (maintainer needs more info)

I have started to disable modules in order to send you a dump with minimal modules
and when I leave the installation with the bare minimum + core modules it behaves correctly

I will have to investigate which module might be at fault

rfay’s picture

It may still be role change notify, but your investigation will really help. Thanks for doing this.

If it is possible, it will probably be best for you to send me a tarball and dump, although a db dump may be adequate.

Thanks very much.

-Randy

rfay’s picture

Actually, when you nail the module that interacts with role_change_notify to cause this, you can just tell me and I'll probably be able to recreate it.

rfay’s picture

Ping - Hoping to hear from you on this, as I do want to get it straightened out. Thanks!

GiorgosK’s picture

I am not sure I have the time to debug it right now ... when I get the time I will look into it ...

heltem’s picture

@rfay : I think your patch for #736086: Cannot use object of type stdClass as array, Remove extra hook invocation embeds the fix for this issue too. Right ?

rfay’s picture

@heltem: I was never able to recreate this one, so I don't know. But I wouldn't be surprised.

heltem’s picture

I was thinking about this patch part from #736086: Cannot use object of type stdClass as array, Remove extra hook invocation

 function role_change_notify_hook_info() {
   return array(
-    'role_added' => array(
+    'user' => array(
       'user' => array(
         'role_added' => array(
           'runs when' => t("When a role is added to an account"),
         ),
-      )
-    ),
-    'role_removed' => array(
-      'user' => array(
         'role_removed' => array(
           'runs when' => t("When a role is removed from an account"),
         ),
-      )
+      ),
     ),
-
   );
 }
rfay’s picture

Status: Postponed (maintainer needs more info) » Fixed

OK, calling this fixed.

Status: Fixed » Closed (fixed)

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