Module "Call Me" (for Drupal 7) provides a block through which users can send a requests to call back. Requests is sent to the specified email address.
Sandbox: http://drupal.org/sandbox/Anonym/1676276
Git: Anonym@git.drupal.org:sandbox/Anonym/1676276.git

CommentFileSizeAuthor
callme.png6.54 KBAnonym

Comments

sanchi.girotra’s picture

1. Error report: http://ventral.org/pareview/httpgitdrupalorgsandboxanonym1676276git.
2. There should be proper comments in your code refer http://drupal.org/coding-standards.

Anonym’s picture

I fixed these issues.

dabblela’s picture

I will leave this for a member of the team to close, but is this project necessary? It seems to duplicate functionality that can already be produced with the Webform and Webform Phone modules.

dabblela’s picture

Status: Active » Needs review
Anonym’s picture

Webform is big and slow. My module is simple and fast. Many people don't need big monster such as webform, but need simple module "Call Me" )

vaibhavjain’s picture

To fasten the process, please look -- http://drupal.org/node/1410826

andypost’s picture

Status: Needs review » Needs work
$message['body'][] = t("Phone number: !phone", array('!phone' => $params['phone']));

there's asecurity issue with this code - phone send as is, use @phone and @name and do not use check_plain() here:

$values['name'] = check_plain($values['name']);

read more about escaping and text formating at format_string() and hook_mail()

Anonym’s picture

Status: Needs work » Needs review

andypost: Thank you. I'm fixed this issue.

a_thakur’s picture

Status: Needs review » Closed (duplicate)

Duplicate applications, refer to http://drupal.org/node/1702174#comment-6280390,

Please finish the oldest one first and then continue with other project applications or push them as full projects on your own as once you are done with the project application of your oldest module(http://drupal.org/node/1102520), you can create your own full projects.

Thanks,
Ashish.

avpaderno’s picture

Title: Call Me » [D7] Call Me
Issue summary: View changes
Related issues: +#1102520: [D7] Adminer