RE : [sdx-developers] Quelques bogues corriges

[Martin Sevigny]

Bonjour,

> 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 ?)

Oui, dans utils...? Rasik?

> 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...

Oui. Rasik?

> <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...

Oui, on devrait le faire avec un ignoreCase. Rasik?

A bientôt,

Martin Sévigny