Fix unselect button for IE when js is off

htdocs/artefact/file/form/elements/filebrowser.php

@@ -353,16 +353,18 @@ function pieform_element_filebrowser_get_value(Pieform $form, $element) {
 
         // If we got to this point, the doupdate function couldn't select or unselect a file,
         // so we need to let it go through to the form's submit function to deal with.
-        if ($result['select']) {
+        if (!empty($result['select'])) {
             if ($element['config']['selectone']) {
                 $selected = array($result['select']);
             }
             else {
                 $selected = is_array($selected) ? $selected : array();
+                if (!in_array($result['select'], $selected)) {
                     $selected[] = $result['select'];
                 }
             }
-        else if ($result['unselect']) {
+        }
+        else if (!empty($result['unselect'])) {
             $selected = is_array($selected) ? array_diff($selected, array($result['unselect'])) : array();
         }
     }

htdocs/artefact/file/theme/raw/form/selectedlist.tpl

@@ -20,8 +20,7 @@
     </td>
     <td>{$file->description|escape}</td>
     <td>
-       <noscript><button type="submit" class="button small unselect" name="{$prefix}_unselect[{$file->id}]" value="{$file->id}">{str tag=remove}</button></noscript>
-       <button type="button" class="button small unselect hidden" name="{$prefix}_unselect[{$file->id}]" value="{$file->id}">{str tag=remove}</button>
+       <button type="submit" class="button small unselect" name="{$prefix}_unselect[{$file->id}]" value="{$file->id}">{str tag=remove}</button>
        <input type="hidden" name="{$prefix}_selected[{$file->id}]" value="{$file->id}">
     </td>
   </tr>