From: Jan Schneider <jan@horde.org>
Date: Fri, 16 Apr 2010 12:38:43 +0000 (+0200)
Subject: Add basic permissions for users.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=25abb56de144ee2d0ffdc83a54aad3ee9a0694e4;p=horde.git

Add basic permissions for users.
---

diff --git a/kronolith/js/kronolith.js b/kronolith/js/kronolith.js
index 1ed244e70..83d5d2850 100644
--- a/kronolith/js/kronolith.js
+++ b/kronolith/js/kronolith.js
@@ -2412,9 +2412,12 @@ KronolithCore = {
                                       ['kronolithC' + type + 'PGNew']]);
             $('kronolithC' + type + 'PBasic').show();
             $('kronolithC' + type + 'PAdvanced').hide();
+            $('kronolithC' + type + 'PNone').setValue(1);
             $('kronolithC' + type + 'PAllShow').disable();
             $('kronolithC' + type + 'PGList').disable();
             $('kronolithC' + type + 'PGPerms').disable();
+            $('kronolithC' + type + 'PUList').disable();
+            $('kronolithC' + type + 'PUPerms').disable();
             $('kronolithC' + type + 'PAdvanced').select('tr').findAll(function(tr) {
                 return tr.retrieve('remove');
             }).invoke('remove');
@@ -2567,11 +2570,15 @@ KronolithCore = {
             $('kronolithC' + type + 'PAllShow').disable();
             $('kronolithC' + type + 'PGList').disable();
             $('kronolithC' + type + 'PGPerms').disable();
+            $('kronolithC' + type + 'PUList').disable();
+            $('kronolithC' + type + 'PUPerms').disable();
             break;
         case 'All':
             $('kronolithC' + type + 'PAllShow').enable();
             $('kronolithC' + type + 'PGList').disable();
             $('kronolithC' + type + 'PGPerms').disable();
+            $('kronolithC' + type + 'PUList').disable();
+            $('kronolithC' + type + 'PUPerms').disable();
             var perms = {
                 'default': Kronolith.conf.perms.read,
                 'guest': Kronolith.conf.perms.read
@@ -2586,6 +2593,8 @@ KronolithCore = {
             $('kronolithC' + type + 'PAllShow').disable();
             $('kronolithC' + type + 'PGList').enable();
             $('kronolithC' + type + 'PGPerms').enable();
+            $('kronolithC' + type + 'PUList').disable();
+            $('kronolithC' + type + 'PUPerms').disable();
             var group = $F('kronolithC' + type + 'PGSingle')
                 ? $F('kronolithC' + type + 'PGSingle')
                 : $F('kronolithC' + type + 'PGList');
@@ -2602,6 +2611,29 @@ KronolithCore = {
                 $('kronolithC' + type + 'PGdelegate_' + group).setValue(0);
             }
             break;
+        case 'U':
+            $('kronolithC' + type + 'PAllShow').disable();
+            $('kronolithC' + type + 'PGList').disable();
+            $('kronolithC' + type + 'PGPerms').disable();
+            $('kronolithC' + type + 'PUList').enable();
+            $('kronolithC' + type + 'PUPerms').enable();
+            var users = $F('kronolithC' + type + 'PUList').strip();
+            users = users ? users.split(/,\s*/) : [];
+            users.each(function(user) {
+                this.insertGroupOrUser(type, 'user', user, true);
+                $('kronolithC' + type + 'PUshow_' + user).setValue(1);
+                $('kronolithC' + type + 'PUread_' + user).setValue(1);
+                if ($F('kronolithC' + type + 'PUPerms') == 'edit') {
+                    $('kronolithC' + type + 'PUedit_' + user).setValue(1);
+                } else {
+                    $('kronolithC' + type + 'PUedit_' + user).setValue(0);
+                }
+                $('kronolithC' + type + 'PUdelete_' + user).setValue(0);
+                if ($('kronolithC' + type + 'PUdelegate_' + user)) {
+                    $('kronolithC' + type + 'PUdelegate_' + user).setValue(0);
+                }
+            }, this);
+            break;
         }
     },
 
@@ -2619,8 +2651,8 @@ KronolithCore = {
         }
 
         var allperms = $H(Kronolith.conf.perms),
-            advanced = false,
-            basic, same, groupPerms, groupId;
+            advanced = false, users = [],
+            basic, same, groupPerms, groupId, userPerms;
         $H(perms).each(function(perm) {
             switch (perm.key) {
             case 'default':
@@ -2673,7 +2705,25 @@ KronolithCore = {
                                 delete perm.value[user.key];
                                 return;
                             }
-                            advanced = true;
+                            // Check if we already have other basic permissions.
+                            if (Object.isUndefined(userPerms) &&
+                                !Object.isUndefined(basic)) {
+                                advanced = true;
+                            }
+                            // Check if all users have the same permissions.
+                            if (!Object.isUndefined(userPerms) &&
+                                userPerms != user.value) {
+                                advanced = true;
+                            }
+                            userPerms = user.value;
+                            if (!advanced &&
+                                (userPerms == (Kronolith.conf.perms.show | Kronolith.conf.perms.read) ||
+                                 userPerms == (Kronolith.conf.perms.show | Kronolith.conf.perms.read | Kronolith.conf.perms.edit))) {
+                                basic = userPerms == (Kronolith.conf.perms.show | Kronolith.conf.perms.read) ? 'user_read' : 'user_edit';
+                                users.push(user.key);
+                            } else {
+                                advanced = true;
+                            }
                         }
                     }, this);
                 }
@@ -2750,6 +2800,15 @@ KronolithCore = {
                 $('kronolithC' + type + 'PBasic').show();
                 $('kronolithC' + type + 'PGPerms').enable();
                 break;
+            case 'user_read':
+            case 'user_edit':
+                $('kronolithC' + type + 'PUList').enable().setValue(users.join(', '));
+                $('kronolithC' + type + 'PU').setValue(1);
+                $('kronolithC' + type + 'PUPerms').setValue(basic.substring(5));
+                $('kronolithC' + type + 'PAdvanced').hide();
+                $('kronolithC' + type + 'PBasic').show();
+                $('kronolithC' + type + 'PUPerms').enable();
+                break;
             }
         }
    },
@@ -3520,9 +3579,11 @@ KronolithCore = {
             case 'kronolithCinternalPNone':
             case 'kronolithCinternalPAll':
             case 'kronolithCinternalPG':
+            case 'kronolithCinternalPU':
             case 'kronolithCtasklistsPNone':
             case 'kronolithCtasklistsPAll':
             case 'kronolithCtasklistsPG':
+            case 'kronolithCtasklistsPU':
                 var info = id.match(/kronolithC(.*)P(.*)/);
                 this.permsClickHandler(info[1], info[2]);
                 break;
diff --git a/kronolith/templates/chunks/permissions.inc b/kronolith/templates/chunks/permissions.inc
index 95ad720f3..79c7eb120 100644
--- a/kronolith/templates/chunks/permissions.inc
+++ b/kronolith/templates/chunks/permissions.inc
@@ -39,6 +39,17 @@ if (count($groups) > 1) {
         ?>
         <br />
       </span>
+      <?php printf(_("or share with %s %sthe users%s %s and %sallow them to%s %s"),
+                   '<input type="radio" id="kronolithC' . $type . 'PU" name="basic_perms" />',
+                   '<label for="kronolithC' . $type . 'PU">', '</label>',
+                   '<input type="text" id="kronolithC' . $type . 'PUList" onchange="KronolithCore.permsClickHandler(\'' . $type . '\', \'U\')" />',
+                   '<label for="kronolithC' . $type . 'PUPerms">', '</label>',
+                   '<select id="kronolithC' . $type . 'PUPerms" onchange="KronolithCore.permsClickHandler(\'' . $type . '\', \'U\')">
+                      <option value="read">' . ($type == 'internal' ? _("read the events") : _("read the tasks")) . '</option>
+                      <option value="edit">' . ($type == 'internal' ? _("read and edit the events") : _("read and edit the tasks")) . '</option>
+                   </select>')
+      ?>
+      <br />
     </div>
   </div>
   <div id="kronolithC<?php echo $type ?>PAdvanced" class="kronolithCPAdvanced" style="display:none">