CVS edit link for supersaas

Hi there,
I've created a small module for Drupal that acts as a bridge to SuperSaaS that I would like to contribute.

SuperSaaS.com is a flexible online appointment scheduling system that works with many different businesses and is available in 14 languages. The basic version is free, a paid version is available for large users and commercial uses.

The module displays a "Book now" button that automatically logs the user into a SuperSaaS schedule using his Drupal user name. It passes the user's information along, creating or updating the user's information on SuperSaaS as needed. This saves users from having to log in twice.

More information, and a download link to the module are here:
http://www.supersaas.com/tutorials/drupal_appointment_scheduling

CommentFileSizeAuthor
#1 supersaas_drupal.zip19.43 KBsupersaas

Comments

supersaas’s picture

Title: supersaas [supersaas] » supersaas appointment scheduling [supersaas]
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new19.43 KB
avpaderno’s picture

Title: supersaas appointment scheduling [supersaas] » supersaas [supersaas]
supersaas’s picture

Hi, this one has been at "postponed (maintainer needs more info)" for about a month. If you can tell me what additional info is required I would be happy to supply it.
Cheers,
-j

avpaderno’s picture

The status is needs review, not postponed (maintainer needs more info); no other information is required from you.

avpaderno’s picture

Status: Needs review » Needs work
Issue tags: +Module review
  • The points reported in this review are not in order or importance / relevance.
  • Most of the times I report the code that present an issue. In such cases, the same error can be present in other parts of the code; the fact I don't report the same issue more than once doesn't mean the same issue is not present in different places.
  • Not all the reported points are application blockers; some of the points I report are simple suggestions to who applies for a CVS account. For a list of what is considered a blocker for the application approval, see CVS applications review, what to expect. Keep in mind the list is still under construction, and can be changed to adapt it to what has been found out during code review, or to make the list clearer to who applies for a CVS account.
  1. License files cannot be committed in Drupal.org repository. Projects committed in Drupal.org repository have the same license used by Drupal.
  2. See http://drupal.org/coding-standards to understand how a module should be written. In particular, see how the code should be formatted.
  3.       $out = '<form method="post" action="http://' . ($domain ? $domain : t('www.supersaas.com')) . '/api/users"><div>';
          $out.= '<input type="hidden" name="account" value="' . str_replace(' ', '_', $account) . '"/>';
          $out.= '<input type="hidden" name="id" value="' . $user->uid . 'fk"/>';
          $out.= '<input type="hidden" name="user[name]" value="' . htmlspecialchars($user->name) . '"/>';
          $out.= '<input type="hidden" name="user[email]" value="' . htmlspecialchars($user->mail) . '"/>';
          $out.= '<input type="hidden" name="checksum" value="' . md5("$account$password$user->name") . '"/>';
          $out.= '<input type="hidden" name="after" value="' . htmlspecialchars(str_replace(' ', '_', $after)) . '"/>';
    
    

    The module should use the form API, to generate an HTML form.
    A URL is not translatable.

  4. The module doesn't implement hook_uninstall().
avpaderno’s picture

Status: Needs work » Closed (won't fix)
edumdq’s picture

Component: Miscellaneous » miscellaneous

Hi, I'm creating a site that uses supersaas scheduling, I need to open the schedule inside the page the user is in (inside an iframe maybe) so the user stays inside my website. Does anyone know how to do that? maybe changing supersaas.module. I'm stuck with this and I need to get it right ASAP.

Regards,
Eduardo.

avpaderno’s picture

Issue summary: View changes

Settings comments as read-only.