Is it possible to hide the settings of IM alerts for users who are not IM enabled?

Thanks

Comments

jvieille’s picture

jvieille commented

IM settings appear on all profiles. Should only appear for IM allowed roles

jvieille’s picture

jvieille commented

Is it possible to fix this small bug?
Thanks

pahariwalla’s picture

pahariwalla commented

Sorry for the delay. Trying to catch up now after 3 months of neglect.

Yes, I agree that this is a "bug". Currently IM uses a permission "IM Talk", and it makes sense to use this in determining whether the user can access the alert settings. Will do.

pahariwalla’s picture

pahariwalla commented
Issue tags: +sound, +quick

tagging quick

pahariwalla’s picture

pahariwalla commented

commited to rev 1.9.2.21 for im.module

I've moved the user customizable IM alert settings from being on their own "tab" in the profile to being on the main edit page of the user profile.

I did this because this tab is controlled by the menu system and i couldn't figure out how to make it sensitive to user access rights ...

pahariwalla’s picture

pahariwalla commented
Status: Active » Needs review

setting status

jvieille’s picture

jvieille commented

Installed the last dev: doesn't work

I've got a lot of warnings running update.php, and then after disabling IM, an error keeps showing on the Modules page
"warning: array_merge() [function.array-merge]: Argument #1 is not an array in /home/controlc/www/sites/all/modules/im/im.module on line 482."

pahariwalla’s picture

pahariwalla commented

sorry folks ... maybe i should start posting patches and let the community test them before i commit ... ;-) fix is in rev 1.9.2.22 of im.module