6.0.0-beta1
7/4/25

[#10482] "standard" sharing with a group don't work
Summary "standard" sharing with a group don't work
Queue Kronolith
Queue Version 3.0.9
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester xavier.montagutelli (at) unilim (dot) fr
Created 09/02/2011 (5054 days ago)
Due
Updated 09/05/2011 (5051 days ago)
Assigned 09/05/2011 (5051 days ago)
Resolved 09/05/2011 (5051 days ago)
Github Issue Link
Github Pull Request
Milestone 3.0.10
Patch No

History
09/05/2011 03:56:12 PM xavier (dot) montagutelli (at) unilim (dot) fr Comment #11 Reply to this comment
Changes have been made in Git for this ticket:
Great, it works perfectly now, thanks.
09/05/2011 02:50:54 PM Git Commit Comment #10 Reply to this comment
Changes have been made in Git for this ticket:

[jan] Update group permission when changing group down list in basic 
permissions (Bug #10482).

  3 files changed, 10 insertions(+), 0 deletions(-)
http://git.horde.org/horde-git/-/commit/6389f44a8e4a1527ab3a828f09a3702e35f09755
09/05/2011 01:56:52 PM xavier (dot) montagutelli (at) unilim (dot) fr Comment #9 Reply to this comment
I'm not sure what you mean. Do you want the radiobox for group 
shareing to be automatically checked if you change the group drop 
down list?
Not at all.

The bug is not completely corrected with the patch. Whatever group I 
choose in the list, the share is created for the *first* group.

  Adding the attribute
onchange="KronolithCore.permsClickHandler('internal', 'G')"
to the list of groups (<select> tag with 
id="kronolithCinternalPGList") seems to correct the bug.

09/05/2011 01:40:07 PM Jan Schneider Comment #8 Reply to this comment
I'm not sure what you mean. Do you want the radiobox for group 
shareing to be automatically checked if you change the group drop down 
list?
09/05/2011 01:28:45 PM xavier (dot) montagutelli (at) unilim (dot) fr Comment #7 Reply to this comment
Changes have been made in Git for this ticket:

No need to reload the group list here, it's already loaded when 
opening the calendar (Bug #10482).

  1 files changed, 1 insertions(+), 2 deletions(-)
http://git.horde.org/horde-git/-/commit/03fe38f0ba184fac47dec7f7979e781a75cdbb0e
Thank you very much for being so quick at fixing bugs.

I have applied the patch against my kronolith/js/kronolith.js file and 
removed all .js files in my "static/" directory.

Now I don't have any JS error. But sharing only works for the first 
group in the list. I am member of more than one group ; the share is 
created for the first group, even if I select another one in the 
dropdwon list.

Looks like the "permsClickHandler" function is called when I click on 
any radiobutton, but not when I use the dropdown menu to select 
another group. Adding with firebug :

onchange="KronolithCore.permsClickHandler('internal', 'G')"

to the "kronolithCinternalPGList" div resolves completely the bug, but 
I don't know where to put that on the PHP/JS code ...


NB : the rest of my Horde / Kronolith code is still in "stable" branch !!

09/05/2011 11:53:48 AM Jan Schneider Milestone ⇒ 3.0.10
 
09/05/2011 11:53:36 AM Git Commit Comment #6 Reply to this comment
Changes have been made in Git for this ticket:

[jan] Fix race condition when editing group permission in basic 
permission screen (Bug #10482).

  2 files changed, 4 insertions(+), 0 deletions(-)
http://git.horde.org/horde-git/-/commit/4ba01aa8554f01e4e85786bd78da1bc68547f5b4
09/05/2011 11:53:33 AM Git Commit Comment #5 Reply to this comment
Changes have been made in Git for this ticket:

No need to reload the group list here, it's already loaded when 
opening the calendar (Bug #10482).

  1 files changed, 1 insertions(+), 2 deletions(-)
http://git.horde.org/horde-git/-/commit/03fe38f0ba184fac47dec7f7979e781a75cdbb0e
09/05/2011 11:52:06 AM Jan Schneider State ⇒ Resolved
 
09/05/2011 10:35:13 AM Jan Schneider Comment #4
State ⇒ Assigned
Assigned to Jan Schneider
Reply to this comment
It actually doesn't have anything to do with LDAP groups.
09/05/2011 06:52:56 AM xavier (dot) montagutelli (at) unilim (dot) fr Comment #3 Reply to this comment
Any chance you are using LDAP groups?
Yes, sorry I didn't mention it.

The groups are from our LDAP directory :

$conf['group']['params']['basedn'] = 'ou=XXXX,dc=univ,dc=fr';
$conf['group']['params']['scope'] = 'one';
$conf['group']['params']['gid'] = 'cn';
$conf['group']['params']['memberuid'] = 'memberUid';
$conf['group']['params']['attrisdn'] = false;
$conf['group']['params']['newgroup_objectclass'] = array('posixGroup');
$conf['group']['params']['search']['objectclass'] = array('posixGroup');
$conf['group']['params']['search']['filter_type'] = 'objectclass';
$conf['group']['params']['driverconfig'] = 'horde';
$conf['group']['driver'] = 'Ldap';

09/04/2011 05:27:13 PM Jan Schneider Comment #2
State ⇒ Feedback
Reply to this comment
Any chance you are using LDAP groups?
09/02/2011 03:44:39 PM xavier (dot) montagutelli (at) unilim (dot) fr Comment #1
Priority ⇒ 1. Low
State ⇒ Unconfirmed
Patch ⇒ No
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ "standard" sharing with a group don't work
Type ⇒ Bug
Reply to this comment
If I want to share a calendar with a group the new "standard" 
interface does'n work. The "advanced" sharing work.

I don't have any error message, the web interface says it works, but 
the sahres don't change.

Adding messages in Kronolith.php, I see that the  $g_names var in 
function readPermsForm doesn't hold the new value.

A JS console show an error message :

Uncaught TypeError: Cannot call method 'setValue' of null
KronolithCore.permsClickHandler56d8fc83ad2c16eec45ddb43691c39b6.js:862
KronolithCore.clickHandler56d8fc83ad2c16eec45ddb43691c39b6.js:967
(anonymous function)64721a9db0cd1b876919a8be52055911.js:55
_createResponder.responder64721a9db0cd1b876919a8be52055911.js:538

Dev tools IE9 says it is in $('kronolithC'+type+'PGshow_'+group).setValue(1)

Saved Queries