From 44539012d5949c8a055c4eed5978e14e1eedd3d6 Mon Sep 17 00:00:00 2001
From: helge <helge@e4a50df8-12e2-0310-a44c-efbce7f8a7e3>
Date: Thu, 31 Aug 2006 00:47:12 +0000
Subject: [PATCH] quickfix

git-svn-id: http://svn.opengroupware.org/SOPE/trunk@1356 e4a50df8-12e2-0310-a44c-efbce7f8a7e3
---
 .../NGObjWeb/DynamicElements/WOCheckBoxList.m | 40 +++++++++----------
 1 file changed, 20 insertions(+), 20 deletions(-)

diff --git a/sope-appserver/NGObjWeb/DynamicElements/WOCheckBoxList.m b/sope-appserver/NGObjWeb/DynamicElements/WOCheckBoxList.m
index 8f96fba2..c23c07cc 100644
--- a/sope-appserver/NGObjWeb/DynamicElements/WOCheckBoxList.m
+++ b/sope-appserver/NGObjWeb/DynamicElements/WOCheckBoxList.m
@@ -164,35 +164,35 @@
   canSetItem  = [self->item  isValueSettable];
   
   for (cnt = 0; cnt < goCount; cnt++) {
-    id object = [array objectAtIndex:cnt];
+    NSString *pLabel, *sLabel;
+    id object;
     
+    object = [array objectAtIndex:cnt];
     if (canSetIndex)
       [self->index setUnsignedIntValue:cnt inComponent:sComponent];
     
     if (canSetItem)
       [self->item setValue:object inComponent:sComponent];
+
+    pLabel = [self->prefix stringValueInComponent:sComponent];
+    sLabel = [self->suffix stringValueInComponent:sComponent];
+    if (![pLabel isNotEmpty]) pLabel = nil; /* remove null/empty strings */
+    if (![sLabel isNotEmpty]) sLabel = nil; /* remove null/empty strings */
+
+    if (pLabel != nil || sLabel != nil)
+      WOResponse_AddCString(_response, "<label>");
+
+    if (pLabel != nil)
+      WOResponse_AddString(_response, pLabel);
     
-    if (self->prefix != nil) {
-      NSString *ps;
-      
-      ps = [self->prefix stringValueInComponent:sComponent];
-      WOResponse_AddString(_response, ps);
-    }
-  
     /* add checkbox */
     {
-      NSString *n, *label;
+      NSString *n;
   
-      label = [self->suffix stringValueInComponent:sComponent];
-      if (![label isNotEmpty]) label = nil; /* remove null/empty strings */
-      
       n = self->name != nil
 	? [self->name stringValueInComponent:sComponent]
 	: OWFormElementName(self, _ctx);
       
-      if (label != nil)
-	WOResponse_AddCString(_response, "<label>");
-      
       WOResponse_AddCString(_response, "<input type=\"checkbox\" name=\"");
       [_response appendContentHTMLAttributeValue:n];
       WOResponse_AddCString(_response, "\" value=\"");
@@ -225,12 +225,12 @@
 	n = [self->value stringValueInComponent:sComponent];
 	WOResponse_AddHtmlString(_response, n);
       }
-
-      if (label != nil) {
-	WOResponse_AddString(_response, label);
-	WOResponse_AddCString(_response, "</label>");
-      }
     }
+
+    if (sLabel != nil)
+      WOResponse_AddString(_response, sLabel);
+    if (pLabel != nil || sLabel != nil)
+      WOResponse_AddCString(_response, "<.label>");
     
     [_ctx incrementLastElementIDComponent];
   }
-- 
2.39.5