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 <value> in the template that corresponds to a value for the entry in LDAP I'm viewing, I get a ticked checkbox with onBlur="blur_attributename(this);". This goes at the bottom of the list of correctly displayed checkboxes/values.

<tr><td><input  id="attributename"name="new_values[20][]"value="attributename" 
forgottenvalue=""checked="checked"type="checkbox"></td><td><span  style="white-space: 
nowrap;">&nbsp;onBlur="blur_attributename(this);"</span></td></tr>

where attributename and forgottenvalue are my edits. The 20 is correct for this entry. If I put forgottenvalue into the template it displays correctly.


2) This particular attribute I'm playing with is a MUST in the schema, so it gets the asterix on the right of the first line of checkboxes. After the </tr> from above, the html goes:

</tbody></table></td><td  align="right"valign="top">*</td></tr></tbody></table>

followed by extra rows of

<table  border="0"cellpadding="0"cellspacing="0"width="100%"><tbody><tr><td  class="icon"width="25"></td><td  
valign="top"></td><td  align="right"valign="top">*</td></tr></tbody></table>

and then

<table  border="0"cellpadding="0"cellspacing="0"width="100%"><tbody><tr><td  class="icon"width="25"></td><td  valign="top"><div  
id="ajADDVALUEattributename"></div></td><td  align="right"valign="top">*</td></tr></tbody></table>
<table  class="entry"border="0"><tbody><tr><td  
width="25">&nbsp;</td><td></td></tr></tbody></table></td>

By looking at different entries, the number of extra rows of asterixes is one less than the number of ticked checkboxes. (If you count the first asterix, there are as many asterixes as ticked checkboxes.)


Thanks,
Daniel.


On 23/02/2010 5:23 PM, Deon George wrote:
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':
  return $this->newSelectionAttribute($name,$values,$server_id,$source);

Then you should be good to go.

Let me know that it works, and I'll commit this, so that it is included
for the next release.

...deon

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
phpldapadmin-users mailing list
phpldapadmin-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/phpldapadmin-users

Reply via email to