From ccc9e1c46984c622be59faf851b705d90f077da9 Mon Sep 17 00:00:00 2001
From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Wed, 30 Sep 2009 13:04:32 +0000
Subject: [PATCH] Updated permissions for product license usage

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@14411 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 .../opsiLicenses/class_licenseByProduct.inc   | 33 +++++++++----------
 1 file changed, 15 insertions(+), 18 deletions(-)

diff --git a/gosa-plugins/opsi/admin/opsiLicenses/class_licenseByProduct.inc b/gosa-plugins/opsi/admin/opsiLicenses/class_licenseByProduct.inc
index b9d684d89..2a6ef7bce 100644
--- a/gosa-plugins/opsi/admin/opsiLicenses/class_licenseByProduct.inc
+++ b/gosa-plugins/opsi/admin/opsiLicenses/class_licenseByProduct.inc
@@ -35,15 +35,17 @@ class licenseByProduct extends plugin
     $this->dn = $this->orig_dn = $dn;
     $this->si = new opsiLicenceHandler($this->config);
 
+    // Is this a new object? 
     $this->is_account=TRUE;
     if($this->dn == "new"){
       $this->initially_was_account = FALSE;
     }else{
       $this->initially_was_account = TRUE;
+      
+      // Extract pool name out of the fake dn.
       $this->cn = $this->orig_cn = preg_replace("/^opsi:cn=([^,]*),.*$/","\\1",$dn);
     }
 
-    // Extract pool name out of the fake dn.
     $this->init();
   }
 
@@ -55,6 +57,7 @@ class licenseByProduct extends plugin
       $this->init_successfull = TRUE;
     }else{
 
+      // Get license information for the current product.
       $res = $this->si->getLicensesForProduct($this->cn);
       if($this->si->is_error()){
         $this->init_successfull = FALSE;
@@ -81,25 +84,22 @@ class licenseByProduct extends plugin
     // Create usage list
     $list = new divSelectBox("licenseUsage");
   
-    foreach($this->licenseUses as $license){
-      $f1 = array("string" => $license['HOSTID'][0]['VALUE']);
-      $f2 = array("string" => $license['SOFTWARELICENSEID'][0]['VALUE']);
-      $f3 = array("string" => $license['LICENSEPOOLID'][0]['VALUE'],
-                 "attach" => "style='border-right:0px;'");
-      $list->addEntry(array($f1,$f2,$f3));
+    $ui = get_userinfo();
+    $readable = $ui->get_permissions($this->config->current['BASE'], "opsi/licenseByProduct","");
+    if($readable) {
+      foreach($this->licenseUses as $license){
+        $f1 = array("string" => $license['HOSTID'][0]['VALUE']);
+        $f2 = array("string" => $license['SOFTWARELICENSEID'][0]['VALUE']);
+        $f3 = array("string" => $license['LICENSEPOOLID'][0]['VALUE'],
+            "attach" => "style='border-right:0px;'");
+        $list->addEntry(array($f1,$f2,$f3));
+      } 
     } 
 
     $smarty = get_smarty();
 
     // Assign ACls 
     $plInfo = $this->plInfo();
-    foreach($plInfo['plProvidedAcls'] as $name => $desc){
-      $smarty->assign("{$name}ACL",$this->getacl($name));
-    }
-    foreach($this->attributes as $attr){
-      $smarty->assign($attr,$this->$attr);
-    }
-
     $smarty->assign("licenseUses", $list->DrawList());
     $smarty->assign("init_successfull", $this->init_successfull);
     $smarty->assign("initially_was_account", $this->initially_was_account);
@@ -140,10 +140,7 @@ class licenseByProduct extends plugin
           "plPriority"    => 1,
           "plSection"     => array("administration"),
           "plCategory"    => array("opsi"),
-          "plProvidedAcls"=> array(
-            "cn"                => _("Name"),
-            "description" => _("Description"))
-          ));
+          "plProvidedAcls"=> array()));
   }
 }
 
-- 
2.30.2