From b1affc52672f7b9b61fb28a3e6791ccdd86a8b68 Mon Sep 17 00:00:00 2001
From: Guus der Kinderen <guus.der.kinderen@gmail.com>
Date: Fri, 15 Jan 2021 20:03:23 +0100
Subject: [PATCH] fixes #51: Add new offline setting: redirect to chatroom

This introduces a new configurable 'offline setting': redirect to a chat room, adding this option to the database and the admin console.

The embedded converse client is modified to no longer hardcoded falling back to a chatroom, but to do this only when the offline settings instruct it to do so. This routine needs a lot more work (see #49).
---
 changelog.html                                |  1 +
 classes/ofmeet.js                             | 53 +++++++++++++++++--
 src/database/fastpath_db2.sql                 |  1 +
 src/database/fastpath_hsqldb.sql              |  1 +
 src/database/fastpath_mysql.sql               |  1 +
 src/database/fastpath_oracle.sql              |  1 +
 src/database/fastpath_postgresql.sql          |  1 +
 src/database/fastpath_sqlserver.sql           |  1 +
 src/database/upgrade/1/fastpath_db2.sql       |  1 +
 src/database/upgrade/1/fastpath_hsqldb.sql    |  1 +
 src/database/upgrade/1/fastpath_mysql.sql     |  1 +
 src/database/upgrade/1/fastpath_oracle.sql    |  1 +
 .../upgrade/1/fastpath_postgresql.sql         |  1 +
 src/database/upgrade/1/fastpath_sqlserver.sql |  1 +
 .../settings/offline/OfflineSettings.java     | 17 +++++-
 .../offline/OfflineSettingsManager.java       | 30 +++++++----
 .../offline/OfflineSettingsProvider.java      |  3 ++
 .../xmpp/workgroup/WorkgroupIQHandler.java    |  3 +-
 src/web/workgroup-offline.jsp                 | 47 +++++++++++++---
 19 files changed, 143 insertions(+), 23 deletions(-)

diff --git a/changelog.html b/changelog.html
index 8315e9d79..2d69270c2 100644
--- a/changelog.html
+++ b/changelog.html
@@ -46,6 +46,7 @@ <h1>
 
 <p><b>4.5.0</b> -- tbd</p>
 <ul>
+    <li>[<a href="https://github.com/igniterealtime/openfire-fastpath-plugin/issues/51">#51</a>] - Add new offline setting: redirect to chat room.</li>
     <li>[<a href="https://github.com/igniterealtime/openfire-fastpath-plugin/issues/48">#48</a>] - Converse end-user UI does not get notified of rejection.</li>
     <li>[<a href="https://github.com/igniterealtime/openfire-fastpath-plugin/issues/47">#47</a>] - Fix ChatBot's rejection message.</li>
     <li>[<a href="https://github.com/igniterealtime/openfire-fastpath-plugin/issues/46">#46</a>] - Allow workgroup's chatbot to be configured in the admin console.</li>
diff --git a/classes/ofmeet.js b/classes/ofmeet.js
index 6bf0b0fde..ccf0a8c31 100644
--- a/classes/ofmeet.js
+++ b/classes/ofmeet.js
@@ -22,7 +22,6 @@ var ofmeet = (function(of)
     if (!conversation) conversation = "false";
 
     const fastpath = workgroup + '@workgroup.' + domain;
-    const roomJid = workgroup + '@conference.' + domain;
     const boshUri = 'https://' + server + '/http-bind/';
     const wsUri = 'wss://' + server + '/ws/';
 
@@ -111,7 +110,7 @@ var ofmeet = (function(of)
 
             }
             else {
-                _converse.api.rooms.open(roomJid, {nick: getNick()});
+                performOfflineAction();
             }
         });
     }
@@ -143,9 +142,7 @@ var ofmeet = (function(of)
         _converse.connection.addHandler(function(message) {
             if (message.querySelector('depart-queue')) {
                 console.info("joining workgroup failed - departed the queue")
-                document.getElementById("chatloader").style.visibility = "hidden";
-                document.getElementById("chatbutton").style.visibility = "visible";
-                // TODO: give some kind of feedback to the end-user, explaining why this has happened.
+                performOfflineAction();
 
                 // Got a 'depart-queue' message: remove this handler by returning 'false'.
                 return false; // TODO this handler should also be removed after successfully joining a room.
@@ -182,6 +179,52 @@ var ofmeet = (function(of)
         });
     }
 
+    function performOfflineAction(callback) {
+        // Obtain the offline settings options.
+        const iq = converse.env.$iq({type: 'get', to: fastpath}).c('offline-settings', {xmlns: 'http://jivesoftware.com/protocol/workgroup'});
+
+        _converse.connection.sendIQ(iq, function(resp)
+        {
+            console.debug("Received Offline Settings", resp);
+
+            if (resp.querySelector('redirectPage')) {
+                const redirectPage = resp.querySelector('redirectPage').innerHTML;
+                console.debug("Offline Settings suggest to redirect to webpage:", redirectPage);
+                // TODO show dialog, suggest to redirect to webpage.
+                document.getElementById("chatloader").style.visibility = "hidden";
+                document.getElementById("chatbutton").style.visibility = "visible";
+            } else if (resp.querySelector('emailAddress')) {
+                console.debug("Offline Settings suggest to ask the user to leave a message.");
+                const emailAddress = resp.querySelector('emailAddress').innerHTML;
+                const offlineText = resp.querySelector('offlineText').innerHTML;
+                const subject = resp.querySelector('subject').innerHTML;
+                // TODO show form, invite to leave a message, send data back to server (using iq 'send-email' 'http://jivesoftware.com/protocol/workgroup')
+                document.getElementById("chatloader").style.visibility = "hidden";
+                document.getElementById("chatbutton").style.visibility = "visible";
+            } else if (resp.querySelector('redirectMUC')) {
+                const redirectMUC = resp.querySelector('redirectMUC').innerHTML;
+                console.debug("Offline Settings suggest to redirect to MUC:", redirectMUC);
+                document.getElementById("chatloader").style.visibility = "hidden";
+                // TODO show dialog, suggest to redirect to MUC, open after confirmation.
+                _converse.api.rooms.open(redirectMUC, {nick: getNick()});
+            } else {
+                // TODO show apology
+
+                // Remove the spinner, restore the 'chat' button.
+                document.getElementById("chatloader").style.visibility = "hidden";
+                document.getElementById("chatbutton").style.visibility = "visible";
+            }
+
+        }, function(err) {
+            // Service will return a 404 if there are no offline settings for a workgroup (which is the default).
+            console.info("Offline settings not available.");
+
+            // Remove the spinner, restore the 'chat' button.
+            document.getElementById("chatloader").style.visibility = "hidden";
+            document.getElementById("chatbutton").style.visibility = "visible";
+        });
+    }
+
     function getNick()
     {
         if (!window.localStorage["webmeet.nick"]) return undefined;
diff --git a/src/database/fastpath_db2.sql b/src/database/fastpath_db2.sql
index 0c6f4a9e8..19d40dc80 100644
--- a/src/database/fastpath_db2.sql
+++ b/src/database/fastpath_db2.sql
@@ -180,6 +180,7 @@ CREATE TABLE fpOfflineSetting (
     emailAddress varChar(255),
     subject varChar(255),
     offlineText LONG VARCHAR,
+    redirectMUC varChar(255),
     CONSTRAINT fpOfflineSet_pk PRIMARY KEY(workgroupID)
 );
 
diff --git a/src/database/fastpath_hsqldb.sql b/src/database/fastpath_hsqldb.sql
index 4ec71e45b..db18cb8e7 100644
--- a/src/database/fastpath_hsqldb.sql
+++ b/src/database/fastpath_hsqldb.sql
@@ -175,6 +175,7 @@ CREATE TABLE fpOfflineSetting (
   emailAddress varChar(255),
   subject varChar(255),
   offlineText LONGVARCHAR,
+  redirectMUC varChar(255),
   PRIMARY KEY(workgroupID)
 );
 
diff --git a/src/database/fastpath_mysql.sql b/src/database/fastpath_mysql.sql
index 25076f0d5..09b36ec9c 100644
--- a/src/database/fastpath_mysql.sql
+++ b/src/database/fastpath_mysql.sql
@@ -177,6 +177,7 @@ CREATE TABLE fpOfflineSetting (
   emailAddress varChar(255),
   subject varChar(255),
   offlineText TEXT,
+  redirectMUC varChar(255),
   PRIMARY KEY(workgroupID)
 );
 
diff --git a/src/database/fastpath_oracle.sql b/src/database/fastpath_oracle.sql
index f1627d8ac..b0bf15cec 100644
--- a/src/database/fastpath_oracle.sql
+++ b/src/database/fastpath_oracle.sql
@@ -173,6 +173,7 @@ CREATE TABLE fpOfflineSetting (
   emailAddress  VARCHAR2(255),
   subject       VARCHAR2(255),
   offlineText   LONG,
+  redirectMUC   VARCHAR2(255),
   PRIMARY KEY(workgroupID)
 );
 
diff --git a/src/database/fastpath_postgresql.sql b/src/database/fastpath_postgresql.sql
index 150f46b0c..22c41758d 100644
--- a/src/database/fastpath_postgresql.sql
+++ b/src/database/fastpath_postgresql.sql
@@ -175,6 +175,7 @@ CREATE TABLE fpOfflineSetting (
   emailAddress     varChar(255),
   subject          varChar(255),
   offlineText      TEXT,
+  redirectMUC      varChar(255),
   PRIMARY KEY(workgroupID)
 );
 
diff --git a/src/database/fastpath_sqlserver.sql b/src/database/fastpath_sqlserver.sql
index e02136d7a..a8e08726c 100644
--- a/src/database/fastpath_sqlserver.sql
+++ b/src/database/fastpath_sqlserver.sql
@@ -175,6 +175,7 @@ CREATE TABLE fpOfflineSetting (
   emailAddress      NVARCHAR(255),
   subject           NVARCHAR(255),
   offlineText       TEXT,
+  redirectMUC       NVARCHAR(255),
   CONSTRAINT fpOfflineSetting_pk PRIMARY KEY(workgroupID)
 );
 
diff --git a/src/database/upgrade/1/fastpath_db2.sql b/src/database/upgrade/1/fastpath_db2.sql
index d4ce0e844..593673470 100644
--- a/src/database/upgrade/1/fastpath_db2.sql
+++ b/src/database/upgrade/1/fastpath_db2.sql
@@ -1,4 +1,5 @@
 ALTER TABLE fpQueue ADD COLUMN dispatcherClass LONG VARCHAR NULL;
+ALTER TABLE fpOfflineSetting ADD COLUMN redirectMUC VARCHAR(255) NULL;
 
 -- Update database version
 UPDATE ofVersion SET version = 1 WHERE name = 'fastpath';
diff --git a/src/database/upgrade/1/fastpath_hsqldb.sql b/src/database/upgrade/1/fastpath_hsqldb.sql
index 64a847fbc..37454e246 100644
--- a/src/database/upgrade/1/fastpath_hsqldb.sql
+++ b/src/database/upgrade/1/fastpath_hsqldb.sql
@@ -1,4 +1,5 @@
 ALTER TABLE fpQueue ADD COLUMN dispatcherClass VARCHAR(4000) NULL;
+ALTER TABLE fpOfflineSetting ADD COLUMN redirectMUC VARCHAR(255) NULL;
 
 -- Update database version
 UPDATE ofVersion SET version = 1 WHERE name = 'fastpath';
diff --git a/src/database/upgrade/1/fastpath_mysql.sql b/src/database/upgrade/1/fastpath_mysql.sql
index 579447356..96e29e6f1 100644
--- a/src/database/upgrade/1/fastpath_mysql.sql
+++ b/src/database/upgrade/1/fastpath_mysql.sql
@@ -1,4 +1,5 @@
 ALTER TABLE fpQueue ADD COLUMN dispatcherClass TEXT NULL;
+ALTER TABLE fpOfflineSetting ADD COLUMN redirectMUC VARCHAR(255) NULL;
 
 -- Update database version
 UPDATE ofVersion SET version = 1 WHERE name = 'fastpath';
diff --git a/src/database/upgrade/1/fastpath_oracle.sql b/src/database/upgrade/1/fastpath_oracle.sql
index 2dd6874f2..14261fde7 100644
--- a/src/database/upgrade/1/fastpath_oracle.sql
+++ b/src/database/upgrade/1/fastpath_oracle.sql
@@ -1,4 +1,5 @@
 ALTER TABLE fpQueue ADD dispatcherClass VARCHAR(3900) NULL;
+ALTER TABLE fpOfflineSetting ADD redirectMUC VARCHAR2(255) NULL;
 
 -- Update database version
 UPDATE ofVersion SET version = 1 WHERE name = 'fastpath';
diff --git a/src/database/upgrade/1/fastpath_postgresql.sql b/src/database/upgrade/1/fastpath_postgresql.sql
index 579447356..96e29e6f1 100644
--- a/src/database/upgrade/1/fastpath_postgresql.sql
+++ b/src/database/upgrade/1/fastpath_postgresql.sql
@@ -1,4 +1,5 @@
 ALTER TABLE fpQueue ADD COLUMN dispatcherClass TEXT NULL;
+ALTER TABLE fpOfflineSetting ADD COLUMN redirectMUC VARCHAR(255) NULL;
 
 -- Update database version
 UPDATE ofVersion SET version = 1 WHERE name = 'fastpath';
diff --git a/src/database/upgrade/1/fastpath_sqlserver.sql b/src/database/upgrade/1/fastpath_sqlserver.sql
index 6f810b8d9..e991620e4 100644
--- a/src/database/upgrade/1/fastpath_sqlserver.sql
+++ b/src/database/upgrade/1/fastpath_sqlserver.sql
@@ -1,4 +1,5 @@
 ALTER TABLE fpQueue ADD dispatcherClass NVARCHAR(3900) NULL;
+ALTER TABLE fpOfflineSetting ADD redirectMUC NVARCHAR(255) NULL;
 
 -- Update database version
 UPDATE ofVersion SET version = 1 WHERE name = 'fastpath';
diff --git a/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettings.java b/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettings.java
index bb3e99c9b..e6c373240 100644
--- a/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettings.java
+++ b/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettings.java
@@ -17,6 +17,7 @@
 package org.jivesoftware.openfire.fastpath.settings.offline;
 
 import org.jivesoftware.xmpp.workgroup.utils.ModelUtil;
+import org.xmpp.packet.JID;
 
 public class OfflineSettings {
     private String redirectURL;
@@ -25,6 +26,8 @@ public class OfflineSettings {
     private String emailAddress;
     private String subject;
 
+    private JID redirectMUC;
+
     public String getRedirectURL() {
         if(!ModelUtil.hasLength(redirectURL)){
             return "";
@@ -69,7 +72,19 @@ public void setSubject(String subject) {
         this.subject = subject;
     }
 
-    public boolean redirects(){
+    public JID getRedirectMUC() {
+        return redirectMUC;
+    }
+
+    public void setRedirectMUC(JID redirectMUC) {
+        this.redirectMUC = redirectMUC;
+    }
+
+    public boolean redirectsWeb(){
         return (ModelUtil.hasLength(getRedirectURL()));
     }
+
+    public boolean redirectsMUC(){
+        return redirectMUC != null;
+    }
 }
diff --git a/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsManager.java b/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsManager.java
index 5bd7ac19d..ccdcc1dcb 100644
--- a/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsManager.java
+++ b/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsManager.java
@@ -24,6 +24,7 @@
 import org.jivesoftware.xmpp.workgroup.Workgroup;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
+import org.xmpp.packet.JID;
 
 /**
  * Retrieves and persists offline settings for a workgroup.
@@ -35,22 +36,22 @@ public class OfflineSettingsManager {
     private static final Logger Log = LoggerFactory.getLogger(OfflineSettingsManager.class);
     
     private static final String GET_OFFLINE_SETTTINGS =
-            "SELECT redirectPage, emailAddress, subject, offlineText FROM " +
+            "SELECT redirectPage, emailAddress, subject, offlineText, redirectMUC FROM " +
             "fpOfflineSetting WHERE workgroupID=?";
     private static final String INSERT_OFFLINE_SETTINGS =
             "INSERT INTO fpOfflineSetting(workgroupID, redirectPage, emailAddress, subject, " +
-            "offlineText) VALUES(?,?,?,?,?)";
+            "offlineText, redirectMUC) VALUES(?,?,?,?,?,?)";
     private static final String UPDATE_OFFLINE_SETTINGS =
-            "UPDATE fpOfflineSetting SET redirectPage=?, emailAddress=?, subject=?, offlineText=? " +
+            "UPDATE fpOfflineSetting SET redirectPage=?, emailAddress=?, subject=?, offlineText=?, redirectMUC=?" +
             "WHERE workgroupID=?";
 
     public OfflineSettings saveOfflineSettings(Workgroup workgroup, String webPage, String email,
-            String subject, String offlineText)
+            String subject, String offlineText, JID redirectMUC)
     {
         OfflineSettings offline;
         try {
             offline = getOfflineSettings(workgroup);
-            return updateOfflineSettings(workgroup, webPage, email, subject, offlineText);
+            return updateOfflineSettings(workgroup, webPage, email, subject, offlineText, redirectMUC);
         }
         catch (OfflineSettingsNotFound osnf) {
             offline = new OfflineSettings();
@@ -60,6 +61,7 @@ public OfflineSettings saveOfflineSettings(Workgroup workgroup, String webPage,
         offline.setEmailAddress(email);
         offline.setOfflineText(offlineText);
         offline.setSubject(subject);
+        offline.setRedirectMUC(redirectMUC);
 
         String redirectURL = webPage != null ? webPage : "";
         String emailAddress = email != null ? email : "";
@@ -78,6 +80,7 @@ public OfflineSettings saveOfflineSettings(Workgroup workgroup, String webPage,
             pstmt.setString(4, subject);
 
             DbConnectionManager.setLargeTextField(pstmt, 5, offlineText);
+            pstmt.setString(6, redirectMUC == null ? null : redirectMUC.toString());
             pstmt.executeUpdate();
         }
         catch (Exception ex) {
@@ -91,14 +94,14 @@ public OfflineSettings saveOfflineSettings(Workgroup workgroup, String webPage,
     }
 
     public OfflineSettings updateOfflineSettings(Workgroup workgroup, String webPage,
-            String email, String subject, String offlineText)
+            String email, String subject, String offlineText, JID redirectMUC)
     {
         final OfflineSettings offline = new OfflineSettings();
         offline.setRedirectURL(webPage);
         offline.setEmailAddress(email);
         offline.setOfflineText(offlineText);
         offline.setSubject(subject);
-
+        offline.setRedirectMUC(redirectMUC);
 
         String redirectURL = webPage != null ? webPage : "";
         String emailAddress = email != null ? email : "";
@@ -115,7 +118,8 @@ public OfflineSettings updateOfflineSettings(Workgroup workgroup, String webPage
             pstmt.setString(3, subject);
 
             DbConnectionManager.setLargeTextField(pstmt, 4, offlineText);
-            pstmt.setLong(5, workgroup.getID());
+            pstmt.setString(5, redirectMUC == null ? null : redirectMUC.toString());
+            pstmt.setLong(6, workgroup.getID());
             pstmt.executeUpdate();
         }
         catch (Exception ex) {
@@ -138,7 +142,7 @@ public OfflineSettings updateOfflineSettings(Workgroup workgroup, String webPage
      */
     public OfflineSettings getOfflineSettings(Workgroup workgroup) throws OfflineSettingsNotFound {
         OfflineSettings offlineSettings = null;
-
+        Log.trace("Get offline settings for workgroup {}", workgroup.getDisplayName());
         Connection con = null;
         PreparedStatement pstmt = null;
         ResultSet rs = null;
@@ -152,12 +156,20 @@ public OfflineSettings getOfflineSettings(Workgroup workgroup) throws OfflineSet
                 String emailAddress = rs.getString(2);
                 String subject = rs.getString(3);
                 String offlineText = DbConnectionManager.getLargeTextField(rs, 4);
+                String muc = rs.getString(5);
+                JID redirectMUC;
+                if (muc == null || muc.isEmpty()) {
+                    redirectMUC = null;
+                } else {
+                    redirectMUC = new JID(muc);
+                }
 
                 offlineSettings = new OfflineSettings();
                 offlineSettings.setRedirectURL(redirectPage);
                 offlineSettings.setEmailAddress(emailAddress);
                 offlineSettings.setSubject(subject);
                 offlineSettings.setOfflineText(offlineText);
+                offlineSettings.setRedirectMUC(redirectMUC);
             }
         }
         catch (Exception ex) {
diff --git a/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsProvider.java b/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsProvider.java
index 4b634ba25..1eea59ba1 100644
--- a/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsProvider.java
+++ b/src/java/org/jivesoftware/openfire/fastpath/settings/offline/OfflineSettingsProvider.java
@@ -54,6 +54,9 @@ public void executeGet(IQ packet, Workgroup workgroup) {
         if (ModelUtil.hasLength(settings.getRedirectURL())) {
             offline.addElement("redirectPage").setText(settings.getRedirectURL());
         }
+        else if (settings.getRedirectMUC() != null) {
+            offline.addElement("redirectMUC").setText(settings.getRedirectMUC().toString());
+        }
         else {
             offline.addElement("emailAddress").setText(settings.getEmailAddress());
             offline.addElement("offlineText").setText(settings.getOfflineText());
diff --git a/src/java/org/jivesoftware/xmpp/workgroup/WorkgroupIQHandler.java b/src/java/org/jivesoftware/xmpp/workgroup/WorkgroupIQHandler.java
index f28826dbb..bfb6cec0e 100644
--- a/src/java/org/jivesoftware/xmpp/workgroup/WorkgroupIQHandler.java
+++ b/src/java/org/jivesoftware/xmpp/workgroup/WorkgroupIQHandler.java
@@ -81,6 +81,7 @@ public void setWorkgroup(Workgroup workgroup) {
 
     public void process(IQ packet) {
         try {
+            Log.trace("Processing IQ: {}", packet.toXML());
             IQ.Type type = packet.getType();
             if (type == IQ.Type.set) {
                 handleIQSet(packet);
@@ -508,7 +509,7 @@ else if ("vcard-temp".equals(namespace)) {
             getVCard(packet);
         }
         else {
-
+            Log.trace("providers");
             // Check all Workgroup Providers for handling this GET request. If
             // none are found, send bad request error.
             for (WorkgroupProvider provider : providerManager.getWorkgroupProviders()) {
diff --git a/src/web/workgroup-offline.jsp b/src/web/workgroup-offline.jsp
index 80a8e78f5..bbfa1f3f5 100644
--- a/src/web/workgroup-offline.jsp
+++ b/src/web/workgroup-offline.jsp
@@ -7,8 +7,7 @@
                  org.xmpp.packet.JID,
                  org.jivesoftware.openfire.fastpath.settings.offline.OfflineSettingsManager,
                  org.jivesoftware.openfire.fastpath.settings.offline.OfflineSettings,
-                 org.jivesoftware.openfire.fastpath.settings.offline.OfflineSettingsNotFound,
-                 org.jivesoftware.openfire.fastpath.util.WorkgroupUtils"
+                 org.jivesoftware.openfire.fastpath.settings.offline.OfflineSettingsNotFound"
 %><%@ page import="org.jivesoftware.util.JiveGlobals" %>
  <%
      String wgID = request.getParameter("wgID");
@@ -17,6 +16,13 @@
 
      OfflineSettingsManager offlineManager = new OfflineSettingsManager();
      String redirectValue = request.getParameter("redirectToPage");
+     String muc = request.getParameter("redirectToMUC");
+     JID redirectMUC;
+     if (muc != null && !muc.isEmpty()) {
+         redirectMUC = new JID(muc);
+     } else {
+         redirectMUC = null;
+     }
      String statusMessage = "";
 
      OfflineSettings offlineSettings = null;
@@ -35,14 +41,14 @@
          String emailAddress = request.getParameter("email");
          String subject = request.getParameter("subject");
          String header = request.getParameter("headerField");
-         offlineSettings = offlineManager.saveOfflineSettings(workgroup, redirectValue, emailAddress, subject, header);
+         offlineSettings = offlineManager.saveOfflineSettings(workgroup, redirectValue, emailAddress, subject, header, redirectMUC);
          if (offlineSettings != null) {
              statusMessage = "Offline settings have been saved.";
          }
      }
      else if(delete){
         statusMessage = "Offline settings have been deleted.";
-        offlineSettings = offlineManager.saveOfflineSettings(workgroup, redirectValue, "", "", "");
+        offlineSettings = offlineManager.saveOfflineSettings(workgroup, redirectValue, "", "", "", null);
      }
      else {
          try {
@@ -74,14 +80,29 @@
                  document.offline.email.value = "";
                  document.offline.subject.value = "";
                  document.offline.headerField.value = "";
+                 document.offline.redirectToMUC.value = "";
 
                  document.offline.submit();
              }
-             else if(todo[1].checked){
+             else if(todo[1].checked) {
+                 var muc = document.offline.redirectToMUC.value;
+                 if(!Jtrim(muc)){
+                     alert("Please specify the MUC to forward to.");
+                     document.offline.redirectToMUC.focus();
+                     return;
+                 }
+                 document.offline.email.value = "";
+                 document.offline.subject.value = "";
+                 document.offline.headerField.value = "";
+
+                 document.offline.submit();
+             }
+             else if(todo[2].checked){
                var email = document.offline.email.value;
                var subject = document.offline.subject.value;
                var message = document.offline.headerField.value;
                document.offline.redirectToPage.value = '';
+               document.offline.redirectToMUC.value = '';
 
                if(!Jtrim(email) || !Jtrim(subject) || !Jtrim(message)){
                  alert("All fields are required.");
@@ -130,7 +151,7 @@
         </div>
         <table width="100%" cellpadding="3" cellspacing="0" border="0" class="jive-contentBox">
                 <tr valign="top">
-                <% String checked = offlineSettings.redirects() ? "checked" : ""; %>
+                <% String checked = offlineSettings.redirectsWeb() ? "checked" : ""; %>
                             <td width="1%">
                                 <input type="radio" name="todo" value="redirectToPage" <%= checked %> />
                             </td>
@@ -141,9 +162,21 @@
                                  <span class="jive-description">e.g. http://www.jivesoftware.com/contact.html</span>
                             </td>
                 </tr>
+                <tr valign="top">
+                    <% checked = offlineSettings.redirectsMUC() ? "checked" : ""; %>
+                    <td width="1%">
+                        <input type="radio" name="todo" value="redirectToMUC" <%= checked %> />
+                    </td>
+                    <td nowrap><b>Redirect To Chat Room</b>
+                    </td>
+                    <td class="c2">
+                        <input type="text" name="redirectToMUC" size="40" value="<%= offlineSettings.getRedirectMUC() == null ? "" : offlineSettings.getRedirectMUC() %>" /><br/>
+                        <span class="jive-description">e.g. support@conference.example.org</span>
+                    </td>
+                </tr>
                 <tr>
                     <td nowrap width="1%">
-                         <input type="radio" name="todo" value="showEmailPage" <%=!offlineSettings.redirects() ? "checked" :"" %>/>
+                         <input type="radio" name="todo" value="showEmailPage" <%=!offlineSettings.redirectsWeb() && !offlineSettings.redirectsMUC() ? "checked" :"" %>/>
                          <td><b>Display Email Form</b></td>
                      </td>
                      <td>&nbsp;</td>