diff --git a/xCAT-UI/js/configure/configure.js b/xCAT-UI/js/configure/configure.js
index b67272c5a..de0bc445b 100644
--- a/xCAT-UI/js/configure/configure.js
+++ b/xCAT-UI/js/configure/configure.js
@@ -114,7 +114,7 @@ function loadTableNames(data) {
$('#' + tabId).append(tablesDIV);
// Create info bar
- var infoBar = createInfoBar('Select a table to edit');
+ var infoBar = createInfoBar('Select a table to view or edit');
tablesDIV.append(infoBar);
// Create a list for the tables
diff --git a/xCAT-UI/js/configure/update.js b/xCAT-UI/js/configure/update.js
index 39be01c08..a1740606e 100644
--- a/xCAT-UI/js/configure/update.js
+++ b/xCAT-UI/js/configure/update.js
@@ -67,45 +67,48 @@ function showRepository(data) {
StableRepository = "http://xcat.sourceforge.net/yum/xcat-core/";
}
+ var repoList = $('
');
+
//display the Devel Repository, remember user's last selection
- Show = Show + "";
- Show = Show + DevelRepository + "(Devel) ";
+ Show = Show + DevelRepository + "(Devel)";
- $('#repository fieldset').append(Show);
+ repoList.append(Show);
//display the Stable Repository, remember user's last selection
- Show = "";
- Show = Show + StableRepository + "(Stable) ";
+ Show = Show + StableRepository + "(Stable)";
- $('#repository fieldset').append(Show);
+ repoList.append(Show);
//display the Input Repository, remember user's last selection
if (($.cookie('xcatrepository'))
&& (1 != $.cookie('xcatrepository'))
&& (2 != $.cookie('xcatrepository')))
{
- Show = "Other:";
- Show += "";
+ Show = "
Other: ";
+ Show += "";
}
else
{
- Show = "Other:";
- Show += "";
+ Show = "
Other: ";
+ Show += "";
}
- $('#repository fieldset').append(Show);
+ repoList.append(Show);
+ $('#repository fieldset').append(repoList);
}
function showRpmInfo(data)
@@ -133,10 +136,10 @@ function showRpmInfo(data)
$('#rpm fieldset').append("");
Show = "
";
- Show += "
";
- Show += "
";
- Show += "
Package Name
Version
";
- Show += "
";
+ Show += "
";
+ Show += "
";
+ Show += "
Package Name
Version
";
+ Show += "
";
for (temp = 0; temp < Rpms.length; temp++)
{
@@ -178,6 +181,9 @@ function updateSelectAll()
function updateRpm()
{
+ // Remove any warning messages
+ $('#updateTab').find('.ui-state-error').remove();
+
var rpmPath = $('input[type=radio]:checked').val();
var rpmPathType = "0";
var rpms = "";
@@ -226,28 +232,31 @@ function updateRpm()
{
rpms = rpms.slice(0, -1);
}
-
- $('#update').show();
- if ("" == rpms)
- {
- $('#update').empty();
- $('#update').append("Please select the rpm!");
- return;
+
+ // Check RPM and repository
+ var errMsg = '';
+ if (!rpms) {
+ errMsg = "Please select the rpm! ";
}
- if ("" == rpmPath)
- {
- $('#update').empty();
- $('#update').append("Please select or input the repository!");
- return;
+ if (!rpmPath) {
+ errMsg += "Please select or input the repository!";
+ }
+
+ if (!rpms || !rpmPath) {
+ // Show warning message
+ var warn = createWarnBar(errMsg);
+ warn.prependTo($('#updateTab'));
+ return;
}
//remember users' choice and input
$.cookie('xcatrepository', rpmPathType, { path: '/xcat', expires: 10 });
+ $('#update').show();
$('#update').empty();
$('#update').append("
Updating " + rpms + " from " + rpmPath + "
");
- $('#update').append("");
+ $('#update').append("");
$('#rpm button').attr('disabled', 'true');
// send the update command to server
@@ -272,26 +281,34 @@ function ShowUpdateResult(data)
var resArray = data.rsp[0].split(/\n/);
if (0 < resArray.length){
+ // Show last lines
if (('' == resArray[resArray.length - 1]) && (resArray.length > 1)){
$('#update').append(resArray[resArray.length - 2]);
}
else{
$('#update').append(resArray[resArray.length - 1]);
}
- }
-
- $('#update').append(' Response Detail:');
- $('#update a').bind('click', function(){
- $('#resDetail').show();
- });
-
- var resDetail = $('');
- resDetail.hide();
- $('#update').append(resDetail);
-
- for (temp = 0; temp < resArray.length; temp++)
- {
- resDetail.append(resArray[temp] + " ");
+
+ // Create link to show details
+ $('#update').append(' Show details');
+ $('#update a').bind('click', function(){
+ // Toggle details and change text
+ $('#resDetail').toggle();
+ if ($('#update a').text() == 'Show details') {
+ $('#update a').text('Hide details');
+ } else {
+ $('#update a').text('Show details');
+ }
+ });
+
+ var resDetail = $('');
+ resDetail.hide();
+ $('#update').append(resDetail);
+
+ for (temp = 0; temp < resArray.length; temp++)
+ {
+ resDetail.append(resArray[temp] + " ");
+ }
}
//update the rpm info