// $Id$
//
- // Require login
+// Require login
require_once 'plc_login.php';
// Get session and API handles
require_once 'table.php';
require_once 'nifty.php';
-// --------------------
+// --------------------
// recognized URL arguments
-$peerscope=$_GET['peerscope'];
-$pattern=$_GET['pattern'];
+$peerscope = get_array($_GET, 'peerscope');
+$pattern = get_array($_GET, 'pattern');
// --- decoration
$title="Sites";
$tabs []= tab_sites_local();
$tabs []= tab_mysite();
-// --------------------
+// --------------------
$site_filter=array();
function site_status ($site) {
$class=($site['peer_id']) ? 'plc-foreign' : 'plc-warning';
$messages=array();
-
- if (empty ($site['node_ids']))
+
+ if (empty ($site['node_ids']))
$messages [] = "No node";
// do all this stuff on local sites only
if ( ! $site['peer_id'] ) {
-
+
// check that site is enabled
- if ( ! $site['enabled'])
+ if ( ! $site['enabled'])
$messages [] = "Not enabled";
global $PENDING_CONSORTIUM_ID;
if ( $site['ext_consortium_id'] === $PENDING_CONSORTIUM_ID )
$messages [] = "Pending registration";
-
+
// check that site has at least a PI and a tech
global $api;
$persons=$api->GetPersons(array("person_id"=>$site['person_ids']),array("role_ids"));
$nb_pis=0;
- $nb_tech=0;
- if ( $persons) foreach ($persons as $person) {
- if (in_array( '20', $person['role_ids'])) $nb_pis += 1;
- if (in_array( '40', $person['role_ids'])) $nb_techs += 1;
+ $nb_techs=0;
+ if ( $persons)
+ foreach ($persons as $person) {
+ if (in_array( '20', $person['role_ids'])) $nb_pis += 1;
+ if (in_array( '40', $person['role_ids'])) $nb_techs += 1;
}
if ($nb_pis == 0) $messages [] = "No PI";
if ($nb_techs == 0) $messages [] = "No Tech";
-
+
if (! $site['is_public']) $messages []= "Not public";
// check number of slices
$site_columns = array("site_id", "name", "abbreviated_name", "login_base" , "peer_id" );
$site_filter = array_merge ($site_filter, array ("enabled" => TRUE));
} else {
- $site_columns = array("site_id", "name", "abbreviated_name", "login_base" , "peer_id" ,
+ $site_columns = array("site_id", "name", "abbreviated_name", "login_base" , "peer_id" ,
"is_public", "enabled", "ext_consortium_id",
"person_ids", "max_slices", "slice_ids", "node_ids");
}
-if (plc_is_admin())
+if (plc_is_admin())
$tabs['Joining Sites'] = array ('url'=>l_sites_pending(),
'bubble'=>'Review pending join requests');
$table->cell(l_site_t($site_id,$site_id));
}
$peers->cell($table,$peer_id);
- $table->cell (l_site_t($site_id,htmlentities(utf8_decode($site['name']))));
+ // Thomas Dreibholz suggested that utf8_decode could be non-needed
+ // and maybe even harmful on modern systems like f23
+ // $table->cell (l_site_t($site_id,htmlentities(utf8_decode($site['name']))));
+ $table->cell (l_site_t($site_id,htmlentities($site['name'])));
$table->cell (l_site_t($site_id,$login_base));
$table->cell (htmlentities($site['abbreviated_name']));
if (plc_is_admin()) {