Made buildList build lists

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@13208 594d385d-05f5-0310-b6e9-bd551577e9d8

diff --git a/gosa-core/include/utils/class_msgPool.inc b/gosa-core/include/utils/class_msgPool.inc
index b736208f048ba506493aeeaf57593485046277dc..e8d3900dfdeb98e16b4662848586558d38e9f5e5 100644 (file)
--- a/gosa-core/include/utils/class_msgPool.inc
+++ b/gosa-core/include/utils/class_msgPool.inc
@@ -44,15 +44,15 @@ class msgPool
     }
     if (count($name) == 1){
       if($type == ""){
-        return (_("This object will be deleted:")."<br>".msgPool::buildList($name)."<br><br>");
+        return (_("This object will be deleted:")."<br>".msgPool::buildList($name));
       }else{
-        return (sprintf(_("This '%s' object will be deleted:"),$type). "<br>".msgPool::buildList($name)."<br><br>");
+        return (sprintf(_("This '%s' object will be deleted:"),$type). "<br>".msgPool::buildList($name));
       }
     }
     if($type == ""){
-      return (sprintf(_("These objects will be deleted: %s"), "<br>".msgPool::buildList($name)."<br><br>"));
+      return (sprintf(_("These objects will be deleted: %s"), "<br>".msgPool::buildList($name)));
     }else{
-      return (sprintf(_("These '%s' objects will be deleted: %s"),$type, "<br>".msgPool::buildList($name)."<br><br>"));
+      return (sprintf(_("These '%s' objects will be deleted: %s"),$type, "<br>".msgPool::buildList($name)));
     }
   }
 
@@ -359,14 +359,15 @@ class msgPool
 
   public static function buildList($data)
   {
-    $objects= "";
+    $objects= "<ul>";
     foreach ($data as $key => $value){
       if (is_numeric($key)){
-        $objects.= "<br>\n<i>$value</i>";
+        $objects.= "<li>\n<i>$value</i></li>";
       } else {
-        $objects.= "<br>\n$value (<i>$key</i>)";
+        $objects.= "<li>\n$value (<i>$key</i>)</li>";
       }
     }
+    $objects.= "</ul>";
     return($objects);
   }
 
@@ -491,12 +492,12 @@ class msgPool
       return sprintf(_("This '%s' is still in use by this object: %s"), $type, "<br><br>".$objects);
     }
     if (count($objects) == 1){
-      return sprintf(_("This '%s' is still in use by this object: %s"), $type, "<br><br>".msgPool::buildList($objects));
+      return sprintf(_("This '%s' is still in use by this object: %s"), $type, "<br>".msgPool::buildList($objects));
     }
     if (count($objects) == 0){
       return sprintf(_("This '%s' is still in use."), $type);
     }
-    return sprintf(_("This '%s' is still in use by these objects: %s"), $type, "<br><br>".msgPool::buildList($objects));
+    return sprintf(_("This '%s' is still in use by these objects: %s"), $type, "<br>".msgPool::buildList($objects));
   }