Mark lack of jcarousel as warning, not error

Here's a patch that will check if any field is using a carousel pager, and only run the requirement check if it is in use.

Works for me. Thanks.

Thanks for the patch 10oclock, but querying the database directly for this information is just the wrong way to go, instead we should be using the Field info API, probably field_info_instances.

Thanks for the pointer. Here's an updated patch using field_info_instances.

Awesome work! Looking good except for one thing:

+++ b/field_slideshow.install
@@ -32,25 +32,39 @@ function field_slideshow_requirements($phase) {
+    foreach (field_info_instances('node') as $type) {
+      foreach ($type as $fields) {
+        foreach ($fields['display'] as $display) {
+          if ($display['type'] == 'slideshow' && $display['settings']['slideshow_pager'] == 'carousel') {
+            $carousel_used = TRUE;
+            break;
+          }
+        }
+      }

You can apply a field slideshow to any entity, not just a node, so this needs another foreach in here, so it looks something like:

    foreach (field_info_instances() as $bundles) {
      foreach ($bundles as $instances) {
        foreach ($instances as $field) {
          foreach ($field['display'] as $display) {
            if ($display['type'] == 'slideshow' && $display['settings']['slideshow_pager'] == 'carousel') {
              $carousel_used = TRUE;
              break 4;
            }
          }
        }
      }
    }

Patch updated to reflect the comments in #6.

Great stuff, this patch is good to go!

Thanks guys! Commited and pushed to DEV.