I've just installed the module, added a connection in the settings page, and attempted to create my first filemaker node.

I'm given this error:

Fatal error: Class 'FileMaker' not found in mywebsite/sites/all/modules/filemaker/filemaker.module on line 1940 Call Stack: 0.0024 337748 1. {main}() mywebsite/index.php:0 0.2683 26567892 2. menu_execute_active_handler() mywebsite/index.php:17 0.2963 28774700 3. call_user_func_array() mywebsite/includes/menu.inc:349 0.2963 28774952 4. node_page_view() mywebsite/includes/menu.inc:0 0.2963 28775028 5. node_show() mywebsite/modules/node/node.module:1803 0.2963 28775160 6. node_view() mywebsite/modules/node/node.module:1107 0.2963 28775248 7. node_build_content() mywebsite/modules/node/node.module:1012 0.2964 28775556 8. node_invoke() mywebsite/modules/node/node.module:1088 0.2965 28775808 9. filemaker_view() mywebsite/modules/node/node.module:658 0.3001 28784876 10. filemaker_browse_tab() mywebsite/sites/all/modules/filemaker/filemaker.module:355 0.3001 28784964 11. filemaker_default_find() mywebsite/sites/all/modules/filemaker/filemaker.module:617 0.3001 28784964 12. filemaker_object() mywebsite/sites/all/modules/filemaker/filemaker.module:1396

Any idea what might cause this?

CommentFileSizeAuthor
#2 new_features-1169844-2.patch11.24 KBgrasmash
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grasmash’s picture

This issue occurred because I hadn't properly placed the filemaker api and it's filemaker.php file.

There should be a warning when these files are missing, rather than a crash.

grasmash’s picture

FileSize
11.24 KB

I've made a few changes to the module and created a patch:

  • Added a check and corresponding error message that is triggered when a user hasn't properly installed the filemaker api. I haven't tested this in all situations.
  • Added the ability to change the weight of each field
  • Made it so that users without the 'Filemaker Nodes' permission cannot see the Edit | Delete links.
  • Added "Show "Record #" column in results?" on node form.
  • Added classes to table cells for easier css styling
tedstein’s picture

Thanks. I will look at this patch tomorrow.

tedstein’s picture

The patch, by and large, looks good. There are a few minor things I need to fix (for example, you modified the schema instead of providing an update) but I will include almost everything in the next update. Which will be very soon.

Thanks again for the patch.

tedstein’s picture

Status: Active » Closed (fixed)

madmatter23, I applied the patch and kept some version of everything you added. I also added your name to the credits (if you would prefer I remove it, let me know).

You might want to use the coder module to check your syntax, as there are a few Drupal formatting gotchas that popped up a few times. Also, see what I did at the bottom of the .install file.

But, all in all, great work and thanks again.

Check out the 6.x-1.1 release.

grasmash’s picture

That's great! Thanks for including the changes. I'll be sure to use the coder module and create a new update for future patches. Thanks also for the shout-out.