From cb6a1c94c48eca66306d1bc407fda8d661fb1edf Mon Sep 17 00:00:00 2001
From: Graham Knop <graham@plainblack.com>
Date: Wed, 9 Apr 2008 22:27:30 +0000
Subject: [PATCH] fixed: Clipboard select all doesn't work with one item in IE

---
 docs/changelog/7.x.x.txt     |  1 +
 lib/WebGUI/AssetClipboard.pm | 15 ++++++++++-----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/docs/changelog/7.x.x.txt b/docs/changelog/7.x.x.txt
index bcddbe96e..59259732c 100644
--- a/docs/changelog/7.x.x.txt
+++ b/docs/changelog/7.x.x.txt
@@ -17,6 +17,7 @@
  - fixed: After saving Photo, Add an Image points to wrong link
  - fixed: "Back to Album" link doesn't work
  - fixed: email validation links can be broken by mail servers
+ - fixed: Clipboard select all doesn't work with one item in IE
 
 7.5.9
  - fixed: Collaboration System attachments follow site's max size instead of CS's
diff --git a/lib/WebGUI/AssetClipboard.pm b/lib/WebGUI/AssetClipboard.pm
index 3b4d650ea..6ec29e1ef 100644
--- a/lib/WebGUI/AssetClipboard.pm
+++ b/lib/WebGUI/AssetClipboard.pm
@@ -425,11 +425,16 @@ $self->session->style->setLink($self->session->url->extras('assetManager/assetMa
 		assetManager.AddButton("'.$i18n->get("restore").'","restoreList","manageClipboard");
                 assetManager.Write();        
                 var assetListSelectAllToggle = false;
-                function toggleAssetListSelectAll(form){
-                        assetListSelectAllToggle = assetListSelectAllToggle ? false : true;
-                        for(var i = 0; i < form.assetId.length; i++)
-                        form.assetId[i].checked = assetListSelectAllToggle;
-                 }
+                function toggleAssetListSelectAll(form) {
+                    assetListSelectAllToggle = assetListSelectAllToggle ? false : true;
+                    if (typeof form.assetId.length == "undefined") {
+                        form.assetId.checked = assetListSelectAllToggle;
+                    }
+                    else {
+                        for (var i = 0; i < form.assetId.length; i++)
+                            form.assetId[i].checked = assetListSelectAllToggle;
+                    }
+                }
 		 //]]>
                 </script> <div class="adminConsoleSpacer"> &nbsp;</div>';
 	return $ac->render($output, $header);