Changeset 4873 for trunk/www/common/datafiles/select_files.jsp

Timestamp:

Apr 2, 2009, 10:58:58 AM (13 years ago)

Author:

Martin Svensson

Message:

Fixes #1274 Disable/hide "Validate" and "Extract metadata" options on the "Data files" tab if no file type supports it

File:

• trunk/www/common/datafiles/select_files.jsp (modified) (10 diffs)

trunk/www/common/datafiles/select_files.jsp

@@ -135 +135 @@
   }
   var lastFileInputName;
-  function browseOnClick(inputName, extension)
+  var updateCheckboxes = false;
+  function browseOnClick(inputName, extension, updateCheck)
   {
     var frm = document.forms['datafiles'];
+    updateCheckboxes = updateCheck;
     var url = '../../filemanager/index.jsp?ID=<%=ID%>&cmd=SelectOne&callback=setFileCallback';
     if (extension)
@@ -154 +156 @@
     var frm = document.forms['datafiles'];
     frm[lastFileInputName].value = path;
-    fileOnChange(lastFileInputName);
-  }
-  function fileOnChange(inputName)
+    fileOnChange(lastFileInputName, updateCheckboxes);
+  }
+  function fileOnChange(inputName, updateCheckbox)
   {
     var frm = document.forms['datafiles'];
-    if (frm[inputName].value != '')
-    {
-      frm['datafiles.validate'].checked = true;
-      frm['datafiles.metadata'].checked = true;
-    }
-  }
-  function recentOnChange(inputName)
+    var validateCheckbox = frm['datafiles.validate'];
+    var metadataCheckbox = frm['datafiles.metadata'];
+
+    // Check boxes should be updated if the data file type 
+    //supports validation or metadata-extraction
+    if (updateCheckbox)
+    {
+      if (frm[inputName].value != '')
+      {
+        // Data filetype supports validation
+        if (validateCheckbox)
+        {       
+          if (validateCheckbox.disabled) validateCheckbox.disabled = false;  
+          validateCheckbox.checked = "1";
+        }
+        // Data filetype supports metadata extraction
+        if (metadataCheckbox)
+        {     
+          if (metadataCheckbox.disabled) metadataCheckbox.disabled = false;  
+          metadataCheckbox.checked = "1";
+        }
+      }
+    }   
+  }
+  function recentOnChange(inputName, updateCheck)
   {
     var frm = document.forms['datafiles'];
@@ -176 +196 @@
       {
         frm[inputName].value = path;
-        fileOnChange(inputName);
+        fileOnChange(inputName, updateCheck);
       }
       recentInput.selectedIndex = 0;
@@ -209 +229 @@
       <table class="form" cellspacing="2" border="0" cellpadding="0" width="100%">
       <%
-      boolean hasNonPlatformFiles = false;
+      boolean hasNonPlatformFiles = false;      
+      boolean activateCheckBoxes = false;
+
+      boolean validationSupport = false;
+      boolean metadataSupport = false;
+      
       for (DataFileType dft : fileTypes)
       {
@@ -223 +248 @@
         boolean deniedFile = false;
         String extension = dft.getExtension();
+        
+        boolean hasValidator = dft.getValidatorClass() != null;
+        boolean hasMetadataExtractor = dft.getMetadataReaderClass() != null;
+        boolean affectCheckboxes = hasValidator || hasMetadataExtractor;
+        validationSupport = hasValidator ? true : validationSupport;
+        metadataSupport = hasMetadataExtractor ? true : metadataSupport;
         if (member != null)
         {
           try
           {
-            file = member.getFile();
+            file = member.getFile();            
           }
           catch (PermissionDeniedException ex)
@@ -239 +270 @@
         {
           path = file.getPath().toString();
+          activateCheckBoxes = activateCheckBoxes || hasValidator || hasMetadataExtractor;
         }
         else if (deniedFile || deniedPlatform)
@@ -245 +277 @@
         }
         String inputName= "datafile."+dft.getId();
-        recentFiles = (List<File>)cc.getRecent(dc, Item.FILE, dft.getExternalId());
+        recentFiles = (List<File>)cc.getRecent(dc, Item.FILE, dft.getExternalId());       
+        
         %>
         <tr>
@@ -267 +300 @@
                 size="50" title="<%=HTML.encodeTags(dft.getDescription())%>"
                 <%=deniedFile || deniedPlatform ? "disabled" : "" %>
-                onchange="fileOnChange('<%=inputName%>')">&nbsp;
+                onchange="fileOnChange('<%=inputName%>', <%=affectCheckboxes %>)">&nbsp;
               </td>
               <td><base:button 
                   title="Browse&hellip;"
-                  onclick="<%="browseOnClick('"+inputName+"', '" + HTML.javaScriptEncode(extension) + "')"%>"
+                  onclick="<%="browseOnClick('"+inputName+"', '" + HTML.javaScriptEncode(extension) + "', "+ affectCheckboxes + ")"%>"
                   disabled="<%=deniedFile || deniedPlatform %>"
                   />
@@ -286 +319 @@
           {
             %>
-            <select name="recent.<%=inputName%>" onchange="recentOnChange('<%=inputName%>')">
+            <select name="recent.<%=inputName%>" onchange="recentOnChange('<%=inputName%>', <%=affectCheckboxes%>)">
             <option value="">- recently used -
             <option value="">- clear -
@@ -305 +338 @@
         <%
       }
+      if (validationSupport)
+      {
       %>
       <tr>
         <td class="prompt">Validate</td>
-        <td><input type="checkbox" value="1" name="datafiles.validate"></td>
+        <td><input type="checkbox" value="1" <%=activateCheckBoxes ? "" : "disabled"%> name="datafiles.validate"></td>
       </tr>
+      <%
+      }
+      if (metadataSupport)
+      {
+      %>
       <tr>
         <td class="prompt">Extract metadata</td>
-        <td><input type="checkbox" value="1" name="datafiles.metadata"></td>
+        <td><input type="checkbox" value="1" <%=activateCheckBoxes ? "" : "disabled"%> name="datafiles.metadata"></td>
       </tr>
+      <%
+      }
+      %>
       </table>
       <%