Re: [sdx-developers] Quelques bogues corriges

[Pierrick Brihaye]

...et encore un peu pour la route :-)

Pierrick Brihaye a écrit:

> Sur ce même point, on a des tests sur "and", "or" (connecteurs de 
> queries) et sur "true", "false", "vrai", "faux" (représentation texte de 
> booléens). Ne devrait-on pas prévoir un equalsIgnoreCase sur les tests ?

Ca concerne :

dans sdx-actions.xsl :

<xsl:template match="sdx:userIsMember">
...
if ("and".equals(sdx_parameter)) sdx_bool=true;

Dans sdx-queries.xsl :
<xsl:template match="sdx:show" name="sdx:show">
...
if (sdx_parameter != null &amp;&amp; !"false".equals(sdx_parameter))

(à signaler qu'on n'a de test que sur "false", pas "faux", "0"... 
pourrait-on concevoir des méthodes evaluateBoolean, evaluateConnector ?)

<xsl:template name="sdx:complexQuery" match="sdx:complexQuery">
...
if ( "or".equals(sdx_parameter))
...
else if ( "and".equals(sdx_parameter))

Dans sdx.xsl :

méthode : public boolean sdx_boolean(String string)

if (string == null || "".equals(string) || "0".equals(string) || 
"false".equals(string) || "faux".equals(string)) return false;

Sur un autre point, dans sdx-queries.xsl :
<xsl:template name="sdx:sorts">
else if ("descending".equals(sdx_parameters[sdx_i]) || 
"desc".equals(sdx_parameters[sdx_i]) || "1".equals(sdx_parameters[sdx_i]))

<xsl:template name="sdx:sort" match="sdx:sort">
else if ("descendant".equals(sdx_parameter) || 
"desc".equals(sdx_parameter) || "1".equals(sdx_parameter) )
            sdx_j=SortSpecification.SORT_ORDER_DESCENDANT;

Hormis le problème de sensibilité à la casse, on a un problème de 
cohérence sur descending vs descendant. Ici encore, une méthode 
evaluateSort pourrait peut-être aider...

Et pour terminer (pour l'instant), dans sdx.xsl :

<xsl:template name="sdx:infos">
...
if
    (
            ("GET".equals(sdx_type) &amp;&amp; sdx_show.indexOf("get") 
!= -1)
        ||  ("POST".equals(sdx_type) &amp;&amp; 
sdx_show.indexOf("post") != -1)
        ||  ("PUT".equals(sdx_type) &amp;&amp; sdx_show.indexOf("put") 
!= -1)
    )

Ici, c'est plus cosmétique : un habitué de HTTP cherchera peut-être à 
reprendre le haut de casse pour afficher ses paramètres...

A+

--
Pierrick Brihaye, informaticien
Service régional de l'Inventaire
DRAC Bretagne
mailto:address@hidden