Interested, but for D6

There's a chance, but I don't have a use for it myself. I'd be glad to review any work to backport it to D6. If you've got time to dedicate to it, let me know and I can connect you with our contact at Hackpad to enable API access until they make it public.

As of yesterday, users are identified by their username. It probably needs to be updated with a theme('username') call, and then stripping out HTML, so real names can be used on sites that use them.

I am interested in working on a D6 backport for a project. I will ping you separately devientintegral for that contact.

Here is my initial effort for a D6 backport. Still a few issues to address most likely. But I was able to get this running on a local D6 install.

Let's try a version where the admin settings form isn't orphaned.

Archived attached. I'll ping deviantintegral in a bit to add this as a proper module.

Also, fyi, looks like the D7 version needs a patch. Will try to get to this later.

I did a quick read of the diff between the attachment and 7.x-1.x, and while I'm sure there are followups, it's far enough along I figured it was worth it to open a branch and at least have a proper version tag for the issue queue. I'll do a proper review and post comments soon.

This has already been committed; just posting again so I can review it with dreditor.

So here's a high level pass of the code. If we get these things fixed, I think we can close this issue and open up new ones for specific changes as you get them in place. I've avoided all code style issues, but I figure we can deal with those one the module is in a closer-to-working state.

Be sure to file any followup patches against the new branch now that it's been created.

+++ b/hackpad.info
@@ -1,6 +1,6 @@
+description = Hackpad integration for Drupal. A backport of the D7 version. This is in a pre-alpha state. Not intended for producton use.

Let's remove this disclaimer, and put it on the module home page instead.

+++ b/hackpad.info
@@ -1,6 +1,6 @@
+core = 6.x

We're missing a dependency on CCK (content.module).

+++ b/hackpad.install
@@ -34,6 +34,13 @@ function hackpad_field_schema($field) {
+  //Nothing to do

This doesn't match the docblock. But if we don't need it, let's just remove it entirely?

+++ b/hackpad.js
@@ -3,19 +3,18 @@
+  //attach: function(context) {

Lets remove all commented code that isn't needed anymore from D7.

+++ b/hackpad.js
@@ -3,19 +3,18 @@
+console.log(settings);

We should remove all console.log() calls as some browsers will throw errors if it's not defined.

+++ b/hackpad.module
@@ -50,51 +50,51 @@ function hackpad_menu() {
-  return variable_get('hackpad_enabled') && user_access($permission, $account);

Good catch. Submit a patch for the 7.x branch?

+++ b/hackpad.module
@@ -50,51 +50,51 @@ function hackpad_menu() {
+ * The documentation doesn't reference the #attached property, but this goes for the d7 version as well. ¶

#attached was added for D7, and doesn't work in D6. We might just have to be broader and include hackpad.js on every page.

+++ b/hackpad.module
@@ -50,51 +50,51 @@ function hackpad_menu() {
- * Implements hook_theme().

We should pull the hook_theme() implementation below up here so at least we start with it in the same place as the 7.x branch.

+++ b/hackpad.module
@@ -265,42 +423,60 @@ function hackpad_element_process($element, &$form_state, $complete_form) {
+      if(!class_exists('HackpadPad')){

Let's use Autoload here to keep some consistency with D7.

https://drupal.org/project/autoload

+++ b/hackpad.module
@@ -265,42 +423,60 @@ function hackpad_element_process($element, &$form_state, $complete_form) {
+          $pad = new HackpadPad($form_state['input']['hackpad_pad_id_' . $delta]); dpm('hackpad_widget multiple');

Remove dpm()'s from the code so it doesn't break for site installs that don't have devel enabled yet.

+++ b/hackpad.module
@@ -265,42 +423,60 @@ function hackpad_element_process($element, &$form_state, $complete_form) {
+        drupal_set_message('Could not connect to Hackpad service. Please verify your credentials in <a href="admin/config/services/hackpad">The settings form</a> and try again.','error');

I think this URL needs to be changed. As well, this text needs to be a t() call.

+++ b/includes/hackpad.admin.inc
@@ -18,14 +21,6 @@ function hackpad_settings_form($form, $form_state) {
-    ),

Agreed on removing these. It's just something nice for the admin UI, and it's not worth writing the custom code for a rarely used form.

+++ b/includes/hackpad.api.inc
@@ -14,12 +14,14 @@ class HackpadApi {
-    $api = &drupal_static('HACKPAD_API');

Let's depend on ctools and use ctools_static() which is identical to drupal_static in D7.

Thanks a bunch for the feedback deviantintegral! I'll get to work on these this week.

6.x-1.x Patch based on devientintegral's feedback. I a couple of issues to note:

-Since only the subdomain owner has permission to create pads in that subdomain, we will need to capture that user's email address, which is registered with the hackpad service in the global settings form so pads can be created on their behalf. Otherwise we get a HackpadRequestException thrown, which is not caught at the moment.

-Users keep being created in the hackpad service as "Anonymous" --For my own purposes, I created a separate patch that integrates with the realname module. There I called token_replace directly, but it may be nice to run usernames through theme_username(), to allow any rewriting. We will just need to filter out the html markup.

**Edit: I like to always include an install hook, even when there is no installation activity. This allows a module to be automatically installed in profiles.