Congegni/sintonia
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fixed default and override URL settings and removed unused validation

Browse source
This commit is contained in:
Robb Ebright 2017-03-16 10:18:43 -04:00
parent f31236fe66
commit 4c8f2ad435
4 changed files with 23 additions and 53 deletions
Download patch file Download diff file Expand all files Collapse all files

1
airtime_mvc/application/controllers/PreferenceController.php
View file

@ -264,7 +264,6 @@ class PreferenceController extends Zend_Controller_Action
// pulling this from the 2.5.x branch
if (!Application_Model_Preference::GetMasterDjConnectionUrlOverride()) {
$master_connection_url = "http://".$_SERVER['SERVER_NAME'].":".$values["master_source_port"].$values["master_source_mount"];
if (empty($values["master_source_port"]) || empty($values["master_source_port"])) {
Application_Model_Preference::SetMasterDJSourceConnectionURL('N/A');
} else {

49
airtime_mvc/application/forms/LiveStreamingPreferences.php
View file

@ -77,6 +77,7 @@ class Application_Form_LiveStreamingPreferences extends Zend_Form_SubForm
->setValue($m_port)
->setValidators(array($betweenValidator))
->addValidator('regex', false, array('pattern'=>'/^[0-9]+$/', 'messages'=>array('regexNotMatch'=>_('Only numbers are allowed.'))));
$this->addElement($masterSourcePort);
$m_mount = Application_Model_StreamSetting::getMasterLiveStreamMountPoint();
@ -153,53 +154,7 @@ class Application_Form_LiveStreamingPreferences extends Zend_Form_SubForm
public function isValid($data)
{
$isValid = parent::isValid($data);
$master_source_port = $data['master_source_port'];
$show_source_port = $data['show_source_port'];
if ($master_source_port == $show_source_port && $master_source_port != "") {
$element = $this->getElement('show_source_port');
$element->addError(_("You cannot use same port as Master DJ port."));
$isValid = false;
}
if ($master_source_port != "") {
if (is_numeric($master_source_port)) {
if ($master_source_port != Application_Model_StreamSetting::getMasterLiveStreamPort()) {
$sock = socket_create(AF_INET, SOCK_STREAM, SOL_TCP);
try {
socket_bind($sock, 0, $master_source_port);
} catch (Exception $e) {
$element = $this->getElement("master_source_port");
$element->addError(sprintf(_("Port %s is not available"), $master_source_port));
$isValid = false;
}
socket_close($sock);
}
} else {
$isValid = false;
}
}
if ($show_source_port != "") {
if (is_numeric($show_source_port)) {
if ($show_source_port != Application_Model_StreamSetting::getDjLiveStreamPort()) {
$sock = socket_create(AF_INET, SOCK_STREAM, SOL_TCP);
try {
socket_bind($sock, 0, $show_source_port);
} catch (Exception $e) {
$element = $this->getElement("show_source_port");
$element->addError(sprintf(_("Port %s is not available"), $show_source_port));
$isValid = false;
}
socket_close($sock);
}
} else {
$isValid = false;
}
}
return $isValid;
return $isValid = parent::isValid($data);
}
}

14
airtime_mvc/application/models/Preference.php
View file

@ -1102,7 +1102,13 @@ class Application_Model_Preference
public static function GetMasterDJSourceConnectionURL()
{
return self::getValue("master_dj_source_connection_url");
$master_connection_url = self::getValue("master_dj_source_connection_url");
if ($master_connection_url == "") {
$master_connection_url = "http://".$_SERVER['SERVER_NAME'].":". Application_Model_StreamSetting::getMasterLiveStreamPort() . Application_Model_StreamSetting::getMasterLiveStreamMountPoint();
}
return $master_connection_url;
}
public static function SetLiveDJSourceConnectionURL($value)
@ -1112,7 +1118,11 @@ class Application_Model_Preference
public static function GetLiveDJSourceConnectionURL()
{
return self::getValue("live_dj_source_connection_url");
$livedj_connection_url = self::getValue("live_dj_source_connection_url");
if ($livedj_connection_url == "") {
$livedj_connection_url = "http://".$_SERVER['SERVER_NAME'].":". Application_Model_StreamSetting::getDjLiveStreamPort() . Application_Model_StreamSetting::getDjLiveStreamMountPoint();
}
return $livedj_connection_url;
}
/* Source Connection URL override status starts */