Commit bd4c5692 authored by dockx thibault's avatar dockx thibault
Browse files

Merge branch...

Merge branch '6306-notifications-the-attributes-list-should-not-be-mandatory-only-supann-resource' into 'dev'

Resolve "[Notifications] - The Attributes list should not be mandatory - only supann resource."

See merge request fusiondirectory/fd-plugins!1069
Showing with 6 additions and 1 deletion
+6 -1
...@@ -56,7 +56,7 @@ class tasksNotifications extends simplePlugin ...@@ -56,7 +56,7 @@ class tasksNotifications extends simplePlugin
new SetAttribute( new SetAttribute(
new SelectAttribute( new SelectAttribute(
'Attributes', _('Any changes listed to set attributes will trigger a notification'), 'Attributes', _('Any changes listed to set attributes will trigger a notification'),
'fdTasksNotificationsAttributes', TRUE, ['None'] 'fdTasksNotificationsAttributes', TRUE
), ),
), ),
] ]
...@@ -189,6 +189,11 @@ class tasksNotifications extends simplePlugin ...@@ -189,6 +189,11 @@ class tasksNotifications extends simplePlugin
} }
} }
// Ensure 'None' is in the array
if (!in_array('None', $result, TRUE)) {
array_unshift($result, 'None'); // Add 'None' at the beginning
}
return $result; return $result;
} }
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment