$subscription->fields should not be manditory

I am writing a module that makes use of the notifications api. It provides a subscription that never has 'fields'.

However, when I go to save them I get:

warning: Invalid argument supplied for foreach() in notifications.module on line 379

Attached is a patch that makes the 'foreach fields as name => value' section conditional on there being fields

(sorry - patch cleans up line endings again) Important part is (line 189 of the patch):

-  foreach ($subscription->fields as $name => $value) {
-    db_query("INSERT INTO {notifications_fields} (sid, field, value) VALUES(%d, '%s', '%s')", $subscription->sid, $name, $value);
+  if (is_array($subscription->fields)) {
+    foreach ($subscription->fields as $name => $value) {
+      db_query("INSERT INTO {notifications_fields} (sid, field, value) VALUES(%d, '%s', '%s')", $subscription->sid, $name, $value);
+    }