[phpldapadmin] [ phpldapadmin-Bugs-3003782 ] Error on sambaGroupMapping.xml template!!!

Bugs item #3003782, was opened at 2010-05-19 12:06
Message generated for change (Comment added) made by wurley
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498546&aid=3003782&group_id=61828

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: 1.2.x
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Olaf Reitmaier Veracierta (olafrv)
Assigned to: Nobody/Anonymous (nobody)
Summary: Error on sambaGroupMapping.xml template!!!

Initial Comment:
Good afternoon,

In phpldapadmin-1.2.0.5 the template sambaGroupMapping.xml have an error on the specification of the atribute sambaSID, especifically a missing ")" on value propertie.

Here is the WRONG code:

<attribute id="sambaSID">
        <display>Samba SID</display>
        <helper>
                <id>sidsuffix</id>
        <value><![CDATA[=php.GetNextNumber(/;gidNumber;false;(&(objectClass=sambaDomain)(sambaDomainName=mysambadomain));*2,+1000)]]></value>
        </helper>
        <order>4</order>
        <post>=php.Join(-;%sambaSID%,%sidsuffix%)</post>
        <page>1</page>
        <value><![CDATA[=php.PickList(/;(&(objectClass=sambaDomain));sambaSID;%sambaSID% (%sambaDomainName%));;;;sambaDomainName]]></value>
</attribute>

As you can note there is a missing (closing) parentesis ")" in the PickList function call:
        <value><![CDATA[=php.PickList(/;(&(objectClass=sambaDomain));sambaSID;%sambaSID% (%sambaDomainName%));;;;sambaDomainName]]></value>

The correct line must be:

       <value><![CDATA[=php.PickList(/;(&(objectClass=sambaDomain));sambaSID;%sambaSID% (%sambaDomainName%);;;;sambaDomainName)]]></value>

Putting the parentesis in:

... sambaDomainName)]]></value>

Regards.-

----------------------------------------------------------------------

>Comment By: Deon George (wurley)
Date: 2010-11-16 20:12

Message:
Thanks, fixed by commit #34e5bbb

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498546&aid=3003782&group_id=61828

------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
Spend less time writing and  rewriting code and more time creating great
experiences on the web. Be a part of the beta today
http://p.sf.net/sfu/msIE9-sfdev2dev
______________________________________
phpLDAPadmin development mailing list.
To unsbuscribe: https://lists.sourceforge.net/lists/listinfo/phpldapadmin-devel
http://phpldapadmin.sourceforge.net/