diff --git a/web/ajax/modal.php b/web/ajax/modal.php index 64200f559..4166e72e8 100644 --- a/web/ajax/modal.php +++ b/web/ajax/modal.php @@ -9,10 +9,6 @@ $modal = validJsStr($_REQUEST['modal']); $data = array(); switch ( $modal ) { - case 'storage' : - if ( !isset($_REQUEST['id']) ) ajaxError('Storage Id Not Provided'); - $data['html'] = getStorageModalHTML($_REQUEST['id']); - break; case 'server' : if ( !isset($_REQUEST['id']) ) ajaxError('Storage Id Not Provided'); $data['html'] = getServerModalHTML($_REQUEST['id']); diff --git a/web/ajax/modals/storage.php b/web/ajax/modals/storage.php new file mode 100644 index 000000000..638fcfd82 --- /dev/null +++ b/web/ajax/modals/storage.php @@ -0,0 +1,110 @@ + Storage + if ( !isset($_REQUEST['id']) ) { + ajaxError('Storage Id Not Provided'); + return; + } + + $null = ''; + $checked = 'checked="checked"'; + $sid = $_REQUEST['id']; + + if ( !canEdit('System') ) return; + + require_once('includes/Server.php'); + require_once('includes/Storage.php'); + + if ( $_REQUEST['id'] ) { + if ( !($newStorage = ZM\Storage::find_one(array('Id'=>$sid)) ) ) { + // Perhaps do something different here, rather than return nothing + return; + } + } else { + $newStorage = new ZM\Storage(); + $newStorage->Name(translate('NewStorage')); + } + + $type_options = array( 'local' => translate('Local'), 's3fs' => translate('s3fs') ); + $scheme_options = array( + 'Deep' => translate('Deep'), + 'Medium' => translate('Medium'), + 'Shallow' => translate('Shallow'), + ); + + $servers = ZM\Server::find( null, array('order'=>'lower(Name)') ); + $ServersById = array(); + foreach ( $servers as $S ) { + $ServersById[$S->Id()] = $S; + } + +?> + + diff --git a/web/skins/classic/includes/functions.php b/web/skins/classic/includes/functions.php index fdabfce04..b663178ab 100644 --- a/web/skins/classic/includes/functions.php +++ b/web/skins/classic/includes/functions.php @@ -815,110 +815,6 @@ function getStatsTableHTML($eid, $fid, $row='') { return $result; } -function getStorageModalHTML($sid) { - $result = ''; - $null = ''; - $checked = 'checked="checked"'; - - if ( !canEdit('System') ) return; - - require_once('includes/Server.php'); - require_once('includes/Storage.php'); - - if ( $_REQUEST['id'] ) { - if ( !($newStorage = ZM\Storage::find_one(array('Id'=>$sid)) ) ) { - // Perhaps do something different here, rather than return nothing - return; - } - } else { - $newStorage = new ZM\Storage(); - $newStorage->Name(translate('NewStorage')); - } - - $type_options = array( 'local' => translate('Local'), 's3fs' => translate('s3fs') ); - $scheme_options = array( - 'Deep' => translate('Deep'), - 'Medium' => translate('Medium'), - 'Shallow' => translate('Shallow'), - ); - - $servers = ZM\Server::find( null, array('order'=>'lower(Name)') ); - $ServersById = array(); - foreach ( $servers as $S ) { - $ServersById[$S->Id()] = $S; - } - - $result .= ''.PHP_EOL; - - return $result; -} - function getEventDetailHTML($eid='', $eids='') { $result = ''; $inputs = '';