Do not play with smtp_url too early.

smtp_url can be None, so do not treat it as string, allow to take different
actions (like doing nothing).

bt5/erp5_promise/SkinTemplateItem/portal_skins/erp5_promise/Alarm_checkPromiseMailServer.xml

@@ -54,7 +54,6 @@
 \n
 portal = context.getPortalObject()\n
 mailhost = portal.MailHost\n
-promise_url = portal.getPromiseParameter(\'external_service\', \'smtp_url\').rstrip(\'/\')\n
 if getattr(mailhost, \'getMessageList\', None) is not None:\n
   context.newActiveProcess().postResult(ActiveResult(\n
     severity=1,\n
@@ -63,10 +62,12 @@ if getattr(mailhost, \'getMessageList\', None) is not None:\n
   ))\n
   return\n
 \n
+promise_url = portal.getPromiseParameter(\'external_service\', \'smtp_url\')\n
 \n
 if promise_url is None:\n
   return\n
 \n
+promise_url = promise_url.rstrip(\'/\')\n
 if mailhost.force_tls:\n
   protocol = \'smtps\'\n
 else:\n

bt5/erp5_promise/SkinTemplateItem/portal_skins/erp5_promise/Alarm_installMailServer.xml

@@ -57,6 +57,9 @@ if getattr(mailhost, \'getMessageList\', None) is not None:\n
   return\n
 promise_url = portal.getPromiseParameter(\'external_service\', \'smtp_url\')\n
 \n
+if promise_url is None:\n
+  return\n
+\n
 protocol, promise_url = promise_url.split(\'://\', 1)\n
 \n
 if protocol == \'smtps\':\n

bt5/erp5_promise/bt/revision

-25
+26
\ No newline at end of file