Closed (fixed)
Project:
SMS Framework
Version:
6.x-2.x-dev
Component:
SMS User
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
25 Jul 2010 at 19:45 UTC
Updated:
12 Jun 2014 at 02:21 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
shushu commentedComment #2
grub3 commentedAn updated version of the patch.
Comment #3
grub3 commentedThis patch works great, many thanks. This can solve spam issues. When a user validates the mobile phone number, he is assigned a group "Foo". Then I filter on "Foo" to allow posting messages. Great.
Setting to reviewed from the community.
Please apply ASAP.
Comment #4
grub3 commentedWhat happens when a validated mobile phone is removed from user settings? Is there a way to trigger another action to remove user from role?
Comment #5
grub3 commentedOkay, quite simple, I added the following code:
AND:
Comment #6
grub3 commentedI would like to discuss the creation of a user right to delete a mobile phone number. To fight against SPAM one important issue is to make sure that users do not create multiple accounts. So binding a mobile phone against ONE single account is important. I will get back with some code. If we can discuss and work on these issues, this would be nice.
Comment #7
univate commentedSounds like a great feature, lets have new developments against 2.x and get the additional rule for removing sms in there as well to complete the feature.
Comment #8
grub3 commentedPlease find attached an updated patch, which invokes two rules:
* one upon SMS validation,
* one upon deletion.
You only have to edit the corresponding rules and apply a group, for example "sms".
People with group "sms" may have special rights, i.e. post in forums, write documentation, etc ...
Also this patch adds a "Delete mobile" right, to avoid spam.
e.i. people registering a mobile phone cannot delete their own mobile phone without explicit right.
This avoids people sending 100 sms to spam and ruin you.
Comment #9
grub3 commentedThis patch has been running for 2 months on a production server without noticable problem.
Comment #10
BenK commentedSubscribing
Comment #11
univate commentedI have committed this. I have also moved the sms_user rules that are incorrectly part of sms.module to the sms_user module.
Comment #13
pareen commentedIs this patch in core module? Is it working?
Can anyone confirm?
Comment #14
shushu commentedpareen, this patch seems to be in the 6.x-2.x-dev release.
If you encounter any problem with it, you are welcome to contact me.
Regards,
Shushu