From 432de1c04996a19bea301f0070454dddcd83cefa Mon Sep 17 00:00:00 2001 From: Michael Jerris Date: Fri, 2 Apr 2010 00:16:34 -0500 Subject: [PATCH 1/7] Release freeswitch-1.0.5.14226d2 --- configure.in | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.in b/configure.in index 16ba3e6999..4a5c0934ed 100644 --- a/configure.in +++ b/configure.in @@ -3,11 +3,11 @@ # Must change all of the below together # For a release, set revision for that tagged release as well and uncomment -AC_INIT([freeswitch], [1.0.head], BUG-REPORT-ADDRESS) +AC_INIT([freeswitch], [1.0.5], BUG-REPORT-ADDRESS) AC_SUBST(SWITCH_VERSION_MAJOR, [1]) AC_SUBST(SWITCH_VERSION_MINOR, [0]) -AC_SUBST(SWITCH_VERSION_MICRO, [head]) -#AC_SUBST(SWITCH_VERSION_REVISION, []) +AC_SUBST(SWITCH_VERSION_MICRO, [5]) +AC_SUBST(SWITCH_VERSION_REVISION, [14226d2]) AC_CONFIG_FILES([src/include/switch_version.h.in:src/include/switch_version.h.template]) AC_CONFIG_FILES([.version:.version.in]) From a8c20e294f01fe2809fcebc926d335d297af3679 Mon Sep 17 00:00:00 2001 From: Michael Jerris Date: Tue, 6 Apr 2010 13:05:27 -0500 Subject: [PATCH 2/7] Release freeswitch-1.0.6 --- configure.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.in b/configure.in index 16ba3e6999..33bd8d9cb8 100644 --- a/configure.in +++ b/configure.in @@ -3,10 +3,10 @@ # Must change all of the below together # For a release, set revision for that tagged release as well and uncomment -AC_INIT([freeswitch], [1.0.head], BUG-REPORT-ADDRESS) +AC_INIT([freeswitch], [1.0.6], BUG-REPORT-ADDRESS) AC_SUBST(SWITCH_VERSION_MAJOR, [1]) AC_SUBST(SWITCH_VERSION_MINOR, [0]) -AC_SUBST(SWITCH_VERSION_MICRO, [head]) +AC_SUBST(SWITCH_VERSION_MICRO, [6]) #AC_SUBST(SWITCH_VERSION_REVISION, []) AC_CONFIG_FILES([src/include/switch_version.h.in:src/include/switch_version.h.template]) From d7e9c2673fcf63e5023145301fd066253706ce97 Mon Sep 17 00:00:00 2001 From: davidy Date: Wed, 7 Apr 2010 11:22:25 -0400 Subject: [PATCH 3/7] Added support for passing CLI commands to boost sigmod module --- .../ftmod_sangoma_boost/ftmod_sangoma_boost.c | 99 ++++++++++++++++++- .../sangoma_boost_interface.h | 12 ++- 2 files changed, 109 insertions(+), 2 deletions(-) diff --git a/libs/freetdm/src/ftmod/ftmod_sangoma_boost/ftmod_sangoma_boost.c b/libs/freetdm/src/ftmod/ftmod_sangoma_boost/ftmod_sangoma_boost.c index baaa301818..a258184d01 100644 --- a/libs/freetdm/src/ftmod/ftmod_sangoma_boost/ftmod_sangoma_boost.c +++ b/libs/freetdm/src/ftmod/ftmod_sangoma_boost/ftmod_sangoma_boost.c @@ -60,6 +60,9 @@ static time_t congestion_timeouts[MAX_TRUNK_GROUPS]; static ftdm_sangoma_boost_trunkgroup_t *g_trunkgroups[MAX_TRUNK_GROUPS]; +static ftdm_io_interface_t ftdm_sangoma_boost_interface; +static ftdm_status_t ftdm_sangoma_boost_list_sigmods(ftdm_stream_handle_t *stream); + #define BOOST_QUEUE_SIZE 500 /* get freetdm span and chan depending on the span mode */ @@ -1762,6 +1765,81 @@ end: return NULL; } + +#define FTDM_BOOST_SYNTAX "list sigmods | " +/** + * \brief API function to kill or debug a sangoma_boost span + * \param stream API stream handler + * \param data String containing argurments + * \return Flags + */ +static FIO_API_FUNCTION(ftdm_sangoma_boost_api) +{ + char *mycmd = NULL, *argv[10] = { 0 }; + int argc = 0; + + if (data) { + mycmd = ftdm_strdup(data); + argc = ftdm_separate_string(mycmd, ' ', argv, (sizeof(argv) / sizeof(argv[0]))); + } + + if (argc > 1) { + if (!strcasecmp(argv[0], "list")) { + if (!strcasecmp(argv[1], "sigmods")) { + if (ftdm_sangoma_boost_list_sigmods(stream) != FTDM_SUCCESS) { + stream->write_function(stream, "%s: -ERR failed to execute cmd\n", __FILE__); + goto done; + } + goto done; + } + } else { + boost_sigmod_interface_t *sigmod_iface = NULL; + sigmod_iface = hashtable_search(g_boost_modules_hash, argv[0]); + if (sigmod_iface) { + char *p = strchr(data, ' '); + if (++p) { + char* mydup = strdup(p); + if(sigmod_iface->exec_api == NULL) { + stream->write_function(stream, "%s does not support api functions\n", sigmod_iface->name); + goto done; + } + //stream->write_function(stream, "sigmod:%s command:%s\n", sigmod_iface->name, mydup); + if (sigmod_iface->exec_api(stream, mydup) != FTDM_SUCCESS) { + stream->write_function(stream, "-ERR:failed to execute command:%s\n", mydup); + } + free(mydup); + } + + goto done; + } else { + stream->write_function(stream, "-ERR: Could not find sigmod %s\n", argv[0]); + } + } + } + stream->write_function(stream, "-ERR: Usage: %s\n", FTDM_BOOST_SYNTAX); +done: + ftdm_safe_free(mycmd); + return FTDM_SUCCESS; +} + +/** + * \brief Loads sangoma_boost IO module + * \param fio FreeTDM IO interface + * \return Success + */ +static FIO_IO_LOAD_FUNCTION(ftdm_sangoma_boost_io_init) +{ + assert(fio != NULL); + memset(&ftdm_sangoma_boost_interface, 0, sizeof(ftdm_sangoma_boost_interface)); + + ftdm_sangoma_boost_interface.name = "boost"; + ftdm_sangoma_boost_interface.api = ftdm_sangoma_boost_api; + + *fio = &ftdm_sangoma_boost_interface; + + return FTDM_SUCCESS; +} + /** * \brief Loads sangoma boost signaling module * \param fio FreeTDM IO interface @@ -2250,12 +2328,31 @@ static FIO_CONFIGURE_SPAN_SIGNALING_FUNCTION(ftdm_sangoma_boost_configure_span) return FTDM_SUCCESS; } +static ftdm_status_t ftdm_sangoma_boost_list_sigmods(ftdm_stream_handle_t *stream) +{ + ftdm_hash_iterator_t *i = NULL; + boost_sigmod_interface_t *sigmod_iface = NULL; + const void *key = NULL; + void *val = NULL; + + stream->write_function(stream, "List of loaded sigmod modules:\n"); + for (i = hashtable_first(g_boost_modules_hash); i; i = hashtable_next(i)) { + hashtable_this(i, &key, NULL, &val); + if (key && val) { + sigmod_iface = val; + stream->write_function(stream, " %s\n", sigmod_iface->name); + } + } + stream->write_function(stream, "\n"); + return FTDM_SUCCESS; +} + /** * \brief FreeTDM sangoma boost signaling module definition */ EX_DECLARE_DATA ftdm_module_t ftdm_module = { /*.name =*/ "sangoma_boost", - /*.io_load =*/ NULL, + /*.io_load =*/ ftdm_sangoma_boost_io_init, /*.io_unload =*/ NULL, /*.sig_load = */ ftdm_sangoma_boost_init, /*.sig_configure =*/ NULL, diff --git a/libs/freetdm/src/ftmod/ftmod_sangoma_boost/sangoma_boost_interface.h b/libs/freetdm/src/ftmod/ftmod_sangoma_boost/sangoma_boost_interface.h index b555e24338..d7ad6de84f 100644 --- a/libs/freetdm/src/ftmod/ftmod_sangoma_boost/sangoma_boost_interface.h +++ b/libs/freetdm/src/ftmod/ftmod_sangoma_boost/sangoma_boost_interface.h @@ -37,7 +37,6 @@ #include "freetdm.h" - #ifdef __cplusplus extern "C" { #endif @@ -180,6 +179,15 @@ typedef ftdm_status_t (*boost_on_load_func_t) BOOST_ON_LOAD_ARGS; typedef ftdm_status_t (*boost_on_unload_func_t) BOOST_ON_UNLOAD_ARGS; #define BOOST_ON_UNLOAD_FUNCTION(name) ftdm_status_t name BOOST_ON_UNLOAD_ARGS +/*! + \brief Called when user wants to execute sigmod api function + \return FTDM_SUCCESS or FTDM_FAIL + */ +#define BOOST_API_ARGS (ftdm_stream_handle_t *stream, char *cmd) +typedef ftdm_status_t (*boost_api_func_t) BOOST_API_ARGS; +#define BOOST_API_FUNCTION(name) ftdm_status_t name BOOST_API_ARGS + + /*! \brief The boost signaling module interface */ @@ -212,6 +220,8 @@ typedef struct boost_sigmod_interface_s { boost_on_load_func_t on_load; /*! \brief the module is about to be unloaded */ boost_on_unload_func_t on_unload; + /*! \brief module api function */ + boost_api_func_t exec_api; /*! \brief private pointer for the interface user */ void *pvt; } boost_sigmod_interface_t; From ada27c2f70a1fbf1d609c3403469dee51403db40 Mon Sep 17 00:00:00 2001 From: Mathieu Rene Date: Wed, 7 Apr 2010 14:55:55 -0400 Subject: [PATCH 4/7] add switch_channel_export_variable_printf --- src/include/switch_channel.h | 2 ++ src/switch_channel.c | 24 ++++++++++++++++++++++++ 2 files changed, 26 insertions(+) diff --git a/src/include/switch_channel.h b/src/include/switch_channel.h index 5529d9af66..84b0f7fdaf 100644 --- a/src/include/switch_channel.h +++ b/src/include/switch_channel.h @@ -249,6 +249,8 @@ SWITCH_DECLARE(const char *) switch_channel_get_variable_partner(switch_channel_ SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_var_check(switch_channel_t *channel, const char *varname, const char *value, switch_bool_t var_check, switch_bool_t nolocal); #define switch_channel_export_variable(_channel, _varname, _value, _nolocal) switch_channel_export_variable_var_check(_channel, _varname, _value, SWITCH_TRUE, _nolocal) +SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_printf(switch_channel_t *channel, const char *varname, switch_bool_t nolocal, const char *fmt, ...); + /*! \brief Retrieve a variable from a given channel diff --git a/src/switch_channel.c b/src/switch_channel.c index ba9429721a..6019836c15 100644 --- a/src/switch_channel.c +++ b/src/switch_channel.c @@ -768,6 +768,30 @@ done: return status; } +SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_printf(switch_channel_t *channel, const char *varname, switch_bool_t nolocal, const char *fmt, ...) +{ + switch_status_t status = SWITCH_STATUS_FALSE; + char *data = NULL; + va_list ap; + int ret; + + switch_assert(channel != NULL); + + va_start(ap, fmt); + ret = switch_vasprintf(&data, fmt, ap); + va_end(ap); + + if (ret == -1) { + return SWITCH_STATUS_FALSE; + } + + status = switch_channel_export_variable(channel, varname, data, nolocal); + + free(data); + + return status; +} + SWITCH_DECLARE(switch_status_t) switch_channel_set_variable_var_check(switch_channel_t *channel, const char *varname, const char *value, switch_bool_t var_check) { From 2c830f84aa8e5a5b05c55c5f02ae0025c4986bd2 Mon Sep 17 00:00:00 2001 From: Mathieu Rene Date: Wed, 7 Apr 2010 15:02:00 -0400 Subject: [PATCH 5/7] remove switch_channel_export_variable's nolocal argument, more confusing than anything else --- src/include/switch_channel.h | 6 +++--- src/switch_channel.c | 29 ++++++++++------------------- 2 files changed, 13 insertions(+), 22 deletions(-) diff --git a/src/include/switch_channel.h b/src/include/switch_channel.h index 84b0f7fdaf..f21d36d63a 100644 --- a/src/include/switch_channel.h +++ b/src/include/switch_channel.h @@ -246,10 +246,10 @@ SWITCH_DECLARE(const char *) switch_channel_get_variable_partner(switch_channel_ #define switch_channel_set_variable_partner(_channel, _var, _val) switch_channel_set_variable_partner_var_check(_channel, _var, _val, SWITCH_TRUE) -SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_var_check(switch_channel_t *channel, const char *varname, const char *value, switch_bool_t var_check, switch_bool_t nolocal); +SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_var_check(switch_channel_t *channel, const char *varname, const char *value, switch_bool_t var_check); -#define switch_channel_export_variable(_channel, _varname, _value, _nolocal) switch_channel_export_variable_var_check(_channel, _varname, _value, SWITCH_TRUE, _nolocal) -SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_printf(switch_channel_t *channel, const char *varname, switch_bool_t nolocal, const char *fmt, ...); +#define switch_channel_export_variable(_channel, _varname, _value) switch_channel_export_variable_var_check(_channel, _varname, _value, SWITCH_TRUE) +SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_printf(switch_channel_t *channel, const char *varname, const char *fmt, ...); /*! diff --git a/src/switch_channel.c b/src/switch_channel.c index 6019836c15..345d4e791a 100644 --- a/src/switch_channel.c +++ b/src/switch_channel.c @@ -737,38 +737,29 @@ SWITCH_DECLARE(switch_status_t) switch_channel_set_profile_var(switch_channel_t return status; } -SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_var_check(switch_channel_t *channel, const char *varname, const char *value, switch_bool_t var_check, switch_bool_t nolocal) +SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_var_check(switch_channel_t *channel, const char *varname, const char *value, switch_bool_t var_check) { - const char *exports, *exports_varname = varname; - switch_status_t status; + const char *exports; + switch_status_t status = SWITCH_STATUS_FALSE; exports = switch_channel_get_variable(channel, SWITCH_EXPORT_VARS_VARIABLE); - - if (nolocal) { - exports_varname = switch_mprintf("nolocal:%s", varname); - } - - if ((status = switch_channel_set_variable_var_check(channel, exports_varname, value, var_check)) != SWITCH_STATUS_SUCCESS) { - goto done; + + if ((status = switch_channel_set_variable_var_check(channel, varname, value, var_check)) != SWITCH_STATUS_SUCCESS) { + return status; } if (varname && value) { if (exports) { - switch_channel_set_variable_printf(channel, SWITCH_EXPORT_VARS_VARIABLE, "%s,%s", exports, exports_varname); + switch_channel_set_variable_printf(channel, SWITCH_EXPORT_VARS_VARIABLE, "%s,%s", exports, varname); } else { - switch_channel_set_variable(channel, SWITCH_EXPORT_VARS_VARIABLE, exports_varname); + switch_channel_set_variable(channel, SWITCH_EXPORT_VARS_VARIABLE, varname); } } -done: - if (exports_varname != varname) { - free((char*)exports_varname); - } - return status; } -SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_printf(switch_channel_t *channel, const char *varname, switch_bool_t nolocal, const char *fmt, ...) +SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_printf(switch_channel_t *channel, const char *varname, const char *fmt, ...) { switch_status_t status = SWITCH_STATUS_FALSE; char *data = NULL; @@ -785,7 +776,7 @@ SWITCH_DECLARE(switch_status_t) switch_channel_export_variable_printf(switch_cha return SWITCH_STATUS_FALSE; } - status = switch_channel_export_variable(channel, varname, data, nolocal); + status = switch_channel_export_variable(channel, varname, data); free(data); From d3bb72dbb665076a334e6045921d4de7a396a962 Mon Sep 17 00:00:00 2001 From: Joao Mesquita Date: Thu, 8 Apr 2010 00:22:41 -0300 Subject: [PATCH 6/7] Add hold feature. --- fscomm/call.cpp | 24 +- fscomm/call.h | 9 +- fscomm/channel.cpp | 1 + fscomm/channel.h | 3 + fscomm/debugtools/statedebugdialog.cpp | 1 + fscomm/fshost.cpp | 29 +- fscomm/mainwindow.cpp | 53 ++- fscomm/mainwindow.h | 1 + fscomm/mainwindow.ui | 442 +++++++++++++------------ 9 files changed, 332 insertions(+), 231 deletions(-) diff --git a/fscomm/call.cpp b/fscomm/call.cpp index a13f01712e..80dc316fb3 100644 --- a/fscomm/call.cpp +++ b/fscomm/call.cpp @@ -36,6 +36,27 @@ Call::Call() _answeredEpoch = 0; } +switch_status_t Call::toggleHold(bool holdPressed) +{ + if (_state != FSCOMM_CALL_STATE_ANSWERED) return SWITCH_STATUS_FALSE; + switch_stream_handle_t stream = { 0 }; + SWITCH_STANDARD_STREAM(stream); + QString holdStr; + if (holdPressed) + { + holdStr = _channel.data()->getUuid(); + } + else + { + holdStr = "off " + _channel.data()->getUuid(); + } + + switch_status_t st = switch_api_execute("uuid_hold", holdStr.toAscii().data(), NULL, &stream); + switch_safe_free(stream.data); + return st; + +} + switch_status_t Call::toggleRecord(bool startRecord) { QDir conf_dir = QDir::home(); @@ -82,8 +103,7 @@ QTime Call::getCurrentStateTime() { if (_direction == FSCOMM_CALL_DIRECTION_INBOUND) { - /* TODO: DOESNT WORK - How do I get what time it started to ring? */ - _channel.data()->getProgressEpoch() == 0 ? time = _channel.data()->getProgressMediaEpoch() : time = _channel.data()->getProgressEpoch(); + time = _channel.data()->getCreatedEpoch(); } else _otherLegChannel.data()->getProgressEpoch() == 0 ? time = _otherLegChannel.data()->getProgressMediaEpoch() : time = _otherLegChannel.data()->getProgressEpoch(); diff --git a/fscomm/call.h b/fscomm/call.h index 891bb17f54..81f712a679 100644 --- a/fscomm/call.h +++ b/fscomm/call.h @@ -50,8 +50,8 @@ class Call { public: Call(); /* Needs rework */ - QString getCidName(void) { return _channel.data()->getCidName(); } - QString getCidNumber(void) { return _channel.data()->getCidNumber(); } + QString getCidName(void) { return (_direction == FSCOMM_CALL_DIRECTION_INBOUND) ? _otherLegChannel.data()->getCidName() : _channel.data()->getCidName(); } + QString getCidNumber(void) { return (_direction == FSCOMM_CALL_DIRECTION_INBOUND) ? _otherLegChannel.data()->getCidNumber() : _channel.data()->getCidNumber(); } QString getDestinationNumber(void) { return _otherLegChannel.data()->getDestinationNumber(); } void setChannel(QSharedPointer channel) { _channel = channel; } @@ -66,11 +66,12 @@ public: fscomm_call_direction_t getDirection() { return _direction; } fscomm_call_state_t getState() { return _state; } void setState(fscomm_call_state_t state) { _state = state; } - void setCause(QString cause) { _cause = cause; } - QString getCause() { return _cause; } + void setCause(QString cause) { _cause = cause; qDebug()<listDetails->clear(); int r = ui->listEvents->currentRow(); + if (r == -1) return; QString uuid = ui->listUUID->currentItem()->text(); QList > tmpListEvents = _events.value(uuid); QSharedPointer e = tmpListEvents.at(r); diff --git a/fscomm/fshost.cpp b/fscomm/fshost.cpp index fda22b59d6..b37452d01d 100644 --- a/fscomm/fshost.cpp +++ b/fscomm/fshost.cpp @@ -394,13 +394,13 @@ void FSHost::eventChannelOutgoing(QSharedPointerevent, QString u else { Call *callPtr = new Call(); - callPtr->setCallDirection(FSCOMM_CALL_DIRECTION_INBOUND); - callPtr->setChannel(_channels.value(switch_event_get_header_nil(event.data(), "Other-Leg-Unique-ID"))); - callPtr->setOtherLegChannel(_channels.value(uuid)); + callPtr->setChannel(_channels.value(uuid)); + callPtr->setOtherLegChannel(_channels.value(switch_event_get_header_nil(event.data(), "Other-Leg-Unique-ID"))); QSharedPointer call(callPtr); - _active_calls.insert(switch_event_get_header_nil(event.data(), "Other-Leg-Unique-ID"), call); + _active_calls.insert(uuid, call); call.data()->setState(FSCOMM_CALL_STATE_RINGING); + _channels.value(uuid).data()->setCreatedEpoch(QString(switch_event_get_header_nil(event.data(), "Caller-Channel-Created-Time")).toULongLong()); emit ringing(call); } } @@ -426,16 +426,23 @@ void FSHost::eventChannelBridge(QSharedPointerevent, QString uui if (time.toULongLong() > 0) _channels.value(uuid).data()->setProgressMediaEpoch(time.toULongLong()); } void FSHost::eventChannelHangup(QSharedPointerevent, QString uuid) -{ - if (_active_calls.contains(uuid)) - { - emit hungup(_active_calls.take(uuid)); - } -} +{} void FSHost::eventChannelUnbridge(QSharedPointerevent, QString uuid) {} void FSHost::eventChannelHangupComplete(QSharedPointerevent, QString uuid) -{} +{ + if (_active_calls.contains(uuid)) + { + if (_active_calls.value(uuid).data()->getState() != FSCOMM_CALL_STATE_ANSWERED) + { + _active_calls.value(uuid).data()->setState(FSCOMM_CALL_STATE_FAILED); + _active_calls.value(uuid).data()->setCause(switch_event_get_header_nil(event.data(), "variable_originate_disposition")); + emit callFailed(_active_calls.value(uuid)); + return; + } + emit hungup(_active_calls.take(uuid)); + } +} void FSHost::eventChannelDestroy(QSharedPointerevent, QString uuid) { _channels.take(uuid); diff --git a/fscomm/mainwindow.cpp b/fscomm/mainwindow.cpp index 6a99a4bb50..b4720f5034 100644 --- a/fscomm/mainwindow.cpp +++ b/fscomm/mainwindow.cpp @@ -100,6 +100,7 @@ MainWindow::MainWindow(QWidget *parent) : connect(ui->answerBtn, SIGNAL(clicked()), this, SLOT(paAnswer())); connect(ui->hangupBtn, SIGNAL(clicked()), this, SLOT(paHangup())); connect(ui->recoredCallBtn, SIGNAL(toggled(bool)), SLOT(recordCall(bool))); + connect(ui->btnHold, SIGNAL(toggled(bool)), this, SLOT(holdCall(bool))); connect(ui->tableCalls, SIGNAL(itemDoubleClicked(QTableWidgetItem*)), this, SLOT(callTableDoubleClick(QTableWidgetItem*))); connect(ui->action_Preferences, SIGNAL(triggered()), this, SLOT(prefTriggered())); connect(ui->action_Exit, SIGNAL(triggered()), this, SLOT(close())); @@ -363,16 +364,35 @@ void MainWindow::paHangup() ui->hangupBtn->setEnabled(false); } +void MainWindow::holdCall(bool pressed) +{ + + QSharedPointer call = g_FSHost.getCurrentActiveCall(); + + if (call.isNull()) + { + switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Could not hold call because there is not current active call!.\n"); + return; + } + + if (call.data()->toggleHold(pressed) != SWITCH_STATUS_SUCCESS) + { + QMessageBox::warning(this,tr("Hold call"), + tr("

Could not get active call to hold/unhold." + "

Please report this bug."), + QMessageBox::Ok); + switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Could not hold/unhold call [%s].\n", call.data()->getUuid().toAscii().data()); + return; + } + +} + void MainWindow::recordCall(bool pressed) { QSharedPointer call = g_FSHost.getCurrentActiveCall(); if (call.isNull()) { - QMessageBox::warning(this,tr("Record call"), - tr("

FSComm reports that there are no active calls to be recorded." - "

Please report this bug."), - QMessageBox::Ok); switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Could not record call because there is not current active call!.\n"); return; } @@ -472,6 +492,9 @@ void MainWindow::answered(QSharedPointer call) } ui->recoredCallBtn->setEnabled(true); ui->recoredCallBtn->setChecked(false); + ui->btnHold->setEnabled(true); + ui->btnHold->setChecked(false); + ui->btnTransfer->setEnabled(true); ui->dtmf0Btn->setEnabled(true); ui->dtmf1Btn->setEnabled(true); ui->dtmf2Btn->setEnabled(true); @@ -504,13 +527,26 @@ void MainWindow::callFailed(QSharedPointer call) break; } } - ui->textEdit->setText(tr("Call with %1 (%2) failed with reason %3.").arg(call.data()->getCidName(), - call.data()->getCidNumber(), - call.data()->getCause())); + if (call.data()->getDirection() == FSCOMM_CALL_DIRECTION_INBOUND) + { + ui->textEdit->setText(tr("Call from %1 (%2) failed with reason %3.").arg(call.data()->getCidName(), + call.data()->getCidNumber(), + call.data()->getCause())); + } + else + { + ui->textEdit->setText(tr("Call to %1 failed with reason %3.").arg(call.data()->getCidName(), + call.data()->getCidNumber(), + call.data()->getCause())); + } + call.data()->setActive(false); /* TODO: Will cause problems if 2 calls are received at the same time */ ui->recoredCallBtn->setEnabled(false); ui->recoredCallBtn->setChecked(false); + ui->btnHold->setEnabled(false); + ui->btnHold->setChecked(false); + ui->btnTransfer->setEnabled(false); ui->answerBtn->setEnabled(false); ui->hangupBtn->setEnabled(false); ui->dtmf0Btn->setEnabled(false); @@ -558,6 +594,9 @@ void MainWindow::hungup(QSharedPointer call) /* TODO: Will cause problems if 2 calls are received at the same time */ ui->recoredCallBtn->setEnabled(false); ui->recoredCallBtn->setChecked(false); + ui->btnHold->setEnabled(false); + ui->btnHold->setChecked(false); + ui->btnTransfer->setEnabled(false); ui->answerBtn->setEnabled(false); ui->hangupBtn->setEnabled(false); ui->dtmf0Btn->setEnabled(false); diff --git a/fscomm/mainwindow.h b/fscomm/mainwindow.h index 46031b3723..e7348c13d6 100644 --- a/fscomm/mainwindow.h +++ b/fscomm/mainwindow.h @@ -76,6 +76,7 @@ private slots: void hungup(QSharedPointer); void callFailed(QSharedPointer); void recordCall(bool); + void holdCall(bool); void setDefaultAccount(); void accountAdd(QSharedPointer); void accountDel(QSharedPointer); diff --git a/fscomm/mainwindow.ui b/fscomm/mainwindow.ui index 9950b7d088..3987094abf 100644 --- a/fscomm/mainwindow.ui +++ b/fscomm/mainwindow.ui @@ -14,28 +14,250 @@ FSComm - A FreeSWITCH Communicator - + - + - - - false - - - true - - + + + + + false + + + true + + + + + + + + + false + + + New Call + + + + + + + false + + + Hold + + + true + + + false + + + + + + + + + + + false + + + Answer + + + + + + + false + + + Hangup + + + + + + + + + + + false + + + 1 + + + + + + + false + + + 2 + + + + + + + false + + + 3 + + + + + + + false + + + 4 + + + + + + + false + + + 5 + + + + + + + false + + + 6 + + + + + + + false + + + 7 + + + + + + + false + + + 8 + + + + + + + false + + + 9 + + + + + + + false + + + * + + + + + + + false + + + 0 + + + + + + + false + + + # + + + + + + + false + + + A + + + + + + + false + + + B + + + + + + + false + + + D + + + + + + + false + + + C + + + + + + - + - + false - New Call + Transfer @@ -47,200 +269,6 @@ Record - - true - - - false - - - - - - - - - - - false - - - Answer - - - - - - - false - - - Hangup - - - - - - - - - - - false - - - 1 - - - - - - - false - - - 2 - - - - - - - false - - - 3 - - - - - - - false - - - 4 - - - - - - - false - - - 5 - - - - - - - false - - - 6 - - - - - - - false - - - 7 - - - - - - - false - - - 8 - - - - - - - false - - - 9 - - - - - - - false - - - * - - - - - - - false - - - 0 - - - - - - - false - - - # - - - - - - - false - - - A - - - - - - - false - - - B - - - - - - - false - - - D - - - - - - - false - - - C - From 089e7f65da3585ab6e716bfe471d3cb9dbd0699f Mon Sep 17 00:00:00 2001 From: Moises Silva Date: Thu, 8 Apr 2010 10:59:41 -0400 Subject: [PATCH 7/7] fix loop --- libs/openzap/src/zap_threadmutex.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libs/openzap/src/zap_threadmutex.c b/libs/openzap/src/zap_threadmutex.c index 4c2f8ac312..e7a2d6e740 100644 --- a/libs/openzap/src/zap_threadmutex.c +++ b/libs/openzap/src/zap_threadmutex.c @@ -425,7 +425,7 @@ OZ_DECLARE(zap_status_t) zap_interrupt_multiple_wait(zap_interrupt_t *interrupts return ZAP_TIMEOUT; } - for (i = size; i < zap_array_len(ints); i++) { + for (i = 0; i < size; i++) { if (ints[i].revents & POLLIN) { res = read(ints[0].fd, pipebuf, sizeof(pipebuf)); if (res == -1) {