[Bedework-commit] carddav r184 - trunk/clients/javascript/bwAddrbookClient/resources

[svnadmin at bedework.org]

Author: johnsa
Date: 2010-10-27 20:52:15 -0400 (Wed, 27 Oct 2010)
New Revision: 184

Modified:
   trunk/clients/javascript/bwAddrbookClient/resources/addressbook.css
   trunk/clients/javascript/bwAddrbookClient/resources/addressbook.js
Log:
javascript client: fix group additions

Modified: trunk/clients/javascript/bwAddrbookClient/resources/addressbook.css
===================================================================
--- trunk/clients/javascript/bwAddrbookClient/resources/addressbook.css	2010-10-27 22:03:51 UTC (rev 183)
+++ trunk/clients/javascript/bwAddrbookClient/resources/addressbook.css	2010-10-28 00:52:15 UTC (rev 184)
@@ -377,6 +377,9 @@
   clear: both;
 }
 /* dialog boxes */
+.ui-widget-content {
+	line-height: 1.2em;
+}
 .ui-widget-content h3 {
   margin-top: 0.5em;
 }

Modified: trunk/clients/javascript/bwAddrbookClient/resources/addressbook.js
===================================================================
--- trunk/clients/javascript/bwAddrbookClient/resources/addressbook.js	2010-10-27 22:03:51 UTC (rev 183)
+++ trunk/clients/javascript/bwAddrbookClient/resources/addressbook.js	2010-10-28 00:52:15 UTC (rev 184)
@@ -467,11 +467,12 @@
     vcData += "CLASS:PRIVATE\n";
     vcData += "REV:" + getRevDate() + "\n";
     vcData += "NOTE:" + $("#GROUP-NOTE").val() + "\n";
-    if (curCard.MEMBER != undefined) {
-      $(curGroup.MEMBER).each(function() {
-        vcData += "MEMBER:mailto:" + $(this).value + "\n";
-      }); 
-    }
+    if (curGroup.MEMBER != undefined) {
+      for (var i=0; i<curGroup.MEMBER.length; i++) {
+        // no need for mailto: here - it's in the value
+        vcData += "MEMBER:" + curGroup.MEMBER[i].value + "\n";
+      }; 
+    };
     vcData += "END:VCARD";
     
     this.updateEntry(vcData,curCard.href,curCard.etag,"#groupForm");
@@ -524,10 +525,11 @@
       vcData += "REV:" + getRevDate() + "\n";
       vcData += "NOTE:" + note + "\n";
       if (curGroup.MEMBER != undefined) {
-        $(curGroup.MEMBER).each(function() {
-          vcData += "MEMBER:mailto:" + $(this).value + "\n";
-        }); 
-      }
+        for (var i=0; i<curGroup.MEMBER.length; i++) {
+          // no need for mailto: here - it's in the value
+          vcData += "MEMBER:" + curGroup.MEMBER[i].value + "\n";
+        }; 
+      };
       // now tag on the new member:
       vcData += "MEMBER:mailto:" + curMember.EMAIL[0].value + "\n";
       vcData += "END:VCARD";