When trying to use the module I get:

Fatal error: Call to undefined function content_access() in editablefields.module on line 84

Replacing content_access with content_permissions_field_access seems to fix the problem for me, I've attached a patch that checks if you have content_permssions enables and uses that if you do.

CommentFileSizeAuthor
editablefields-fix-content-access.diff614 bytesasciikewl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

psynaptic’s picture

Status: Needs review » Reviewed & tested by the community

I have tested this and it works fine. Please commit. :)

markfoodyburton’s picture

YOu shoudl update to the latest (dev) version of CCK, this patch is, I think, only valid on older versions of CCK.

Cheers

Mark.

tim.plunkett’s picture

+++ editablefields.module	2009-02-22 17:53:32.000000000 +0200
@@ -81,7 +81,8 @@
+  if (!node_access('update',$node) || ¶

Trailing whitespace here, but that could be fixed on commit

+++ editablefields.module	2009-02-22 17:53:32.000000000 +0200
@@ -81,7 +81,8 @@
+     (module_exists('content_permissions') && !content_permissions_field_access('edit', $field))) {

Thats indented 5 spaces, kinda odd. It could just go on one line, or 6 spaces in.

stewart.adam’s picture

Status: Reviewed & tested by the community » Needs work

This patch calls a hook implementation directly (content_permissions_field_access is an implementation of hook_field_access) and will therefore ignore any permissions defined through this hook in other modules. It should use module_load_include() to include the file content_access() is defined in (i.e. content.module) and call content_access() to check for permissions it as before.

joelpittet’s picture

Issue summary: View changes
Status: Needs work » Closed (outdated)

Last update was 2012, going to close as outdated because it's against 6.x branch.