Petals CDK

getExtensions(String) does not trim values

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Resolution: Won't Fix
  • Affects Version/s: 5.0.3
  • Fix Version/s: None
  • Component/s: Utils
  • Security Level: Public
  • Description:
    Hide

    When retrivieving parameter values defines the jbi.xml of a SU (and maybe from a component), the value is not trimed.
    As an example, in the EJB BC, depending on the XML shape, there can be an error while loading the EJB class, because of formatting spaces.

    <ejb:class-name>com.toto.MyEJB</ejb:class-name>

    is not interpreted like

    <ejb:class-name>
                    com.toto.MyEJB
    </ejb:class-name>

    The value is retrieved in the component by using getExtension( "class-name" ).
    If the value is not null, then it should be trimed. Notice that some components bypass it in the SU manager to use this method only once and by triming the result.

    getComponentExtension(String) should also be checked.

    Show
    When retrivieving parameter values defines the jbi.xml of a SU (and maybe from a component), the value is not trimed. As an example, in the EJB BC, depending on the XML shape, there can be an error while loading the EJB class, because of formatting spaces.
    <ejb:class-name>com.toto.MyEJB</ejb:class-name>
    is not interpreted like
    <ejb:class-name>
                    com.toto.MyEJB
    </ejb:class-name>
    The value is retrieved in the component by using getExtension( "class-name" ). If the value is not null, then it should be trimed. Notice that some components bypass it in the SU manager to use this method only once and by triming the result. getComponentExtension(String) should also be checked.
  • Environment:
    Windows XP

Activity

Mathieu Carrolle made changes - Thu, 4 Nov 2010 - 17:39:22 +0100
Field Original Value New Value
Resolution Won't Fix [ 2 ]
Status New [ 10000 ] Resolved [ 10004 ]

People

Dates

  • Created:
    Thu, 4 Nov 2010 - 15:27:18 +0100
    Updated:
    Thu, 4 Nov 2010 - 17:39:22 +0100
    Resolved:
    Thu, 4 Nov 2010 - 17:39:22 +0100