Add Services support

This is awesome stuff. Since I've never used services I can't comment on if it works (but I know it does after talking to dave) but I'd still like someone else to review. Most of the stuff is great. Just a few small comments.

+++ b/bean_service/bean_service.incundefined
@@ -0,0 +1,157 @@
+ *   Unique identifier for this bean.
+ * @param $bean

It helps you add the object reference itself to the @param @param $bean Bean

This helps IDE's also do some fancy autocomplete and doxygen to do some awesome linkage.

+++ b/bean_service/bean_service.incundefined
@@ -0,0 +1,157 @@
+function bean_service_update($bid, $bean) {
+  $bean['bid'] = $bid;
+
+  $old_bean = bean_load($bid);
+  if (empty($old_bean)) {
+    return services_error(t('Bean @bid not found.', array('@bid' => $bid)), 404);
+  }
+
+  // Setup form_state.
+  $form_state = array();
+  $form_state['values'] = $bean;
+  $form_state['values']['op'] = t('Save');
+  $form_state['bean'] = $old_bean;
+
+  drupal_form_submit('bean_form', $form_state, $old_bean);
+
+  if ($errors = form_get_errors()) {
+    return services_error(implode(" ", $errors), 406, array('form_errors' => $errors));
+  }
+
+  return $bid;

Do we need both $bid and $bean passed in? You can get the bid from $bean->identifier()

+++ b/bean_service/bean_service.incundefined
@@ -0,0 +1,157 @@
+function bean_service_delete($bid) {
+  $bean = bean_load($bid);
+  if (empty($bean)) {
+    return services_error(t('There is no bean found with id @bid.', array('@bid' => $bid)), 404);
+  }
+
+  // Delete bean.
+  bean_delete($bean);
+
+  // Clear the cache so an anonymous user sees that his bean was deleted.
+  cache_clear_all();
+  return TRUE;

Use bean_reset() instead of cache_clear_all()

Updated version of the patch.

Changelog
* Move functionality into bean.module as there is little point in a sub module given this doesn't add any hard dependencies and it is only enabled if the user enables the specific resources in services
* Deal with the issues noted above
* Improve access controls
* Update to use new "view bean page" permission
* Rerolled so it applies cleanly against current 7.x-1.x HEAD
* Various bug fixes

The Bean argument is needed for bean_services_update as the REST call PUTs to the specific bean id - such as http://example.com/api/bean/123.json

Looks good.

http://drupal.org/commitlog/commit/22232/93d5c0c60faaef1a8301553be1c2160...