Re: [phpldapadmin-users] Upgrading multiselect in template to 1.2

2010-02-24 Thread Daniel Grimwood
Hi Deon, thanks that looks very good now. It looks the same as in 1.1 :). Regards, Daniel. On 24/02/2010 8:41 PM, Deon George wrote: > OK, a bit more work was required. > > Take a look at patch a1c714 submitted to git. > > http://phpldapadmin.git.sf.net > > ...deon > ---

Re: [phpldapadmin-users] Upgrading multiselect in template to 1.2

2010-02-24 Thread Deon George
OK, a bit more work was required. Take a look at patch a1c714 submitted to git. http://phpldapadmin.git.sf.net ...deon Daniel Grimwood wrote: > thanks yes that has mostly fixed it, it displays the checkboxes in the > template correctly. > > There are still a couple of extra minor issues around

Re: [phpldapadmin-users] Upgrading multiselect in template to 1.2

2010-02-23 Thread Daniel Grimwood
Hi Dion, thanks yes that has mostly fixed it, it displays the checkboxes in the template correctly. There are still a couple of extra minor issues around it... 1) If I don't put in a in the template that corresponds to a value for the entry in LDAP I'm viewing, I get a ticked checkbox wit

Re: [phpldapadmin-users] Upgrading multiselect in template to 1.2

2010-02-23 Thread Deon George
Daniel, This looks like a bug. Edit lib/AttributeFactory.php, and around line 88 add a case statement for "multiselect". It should look like this: case 'password': return $this->newPasswordAttribute($name,$values,$server_id,$source); case 'multiselect': //<- ADD THIS LINE case 'select': ret