Re: [sdx-developers] Quelques bogues corriges

[Pierrick Brihaye]

Re,

Je reviens là-dessus...

Pierrick Brihaye a écrit:

> Dans <xsl:template match="sdx:deleteDocument "> (l'espace ne fait pas 
> très propre) :

Il y a une autre espace dans <xsl:template match="sdx:userIsAdmin "> 
(sdx-actions.xsl).

Dans sdx-queries.xsl, on a ça dans <xsl:template name="sdx:listQuery" 
match="sdx:listQuery"> :

if (!"".equals(sdx_values[sdx_i]) &amp;&amp; !" ".equals(sdx_values[sdx_i]))

Quel est le justificatif de ce test sur une espace ? Pourqoi pas 2 ? 3 ?

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 ?

Dans sdx.xsl, encore une espace inutile :
if (sdx_show==null) sdx_show="user get ";

... car les autres parties du code ajoutent la leur.

Toujours dans cette taglib, je reviens sur :

sdx_user.setHost(request.getRemoteHost()); /* ?? performances ?? */

(noter le commentaire judicieux ;-).

Je pense savoir par quel bout prendre le problème : il faudrait définir 
une variable (sticky !) booléenne enableLookups et, si elle est vraie, 
chercher à résoudre l'hôte.

Pour la positionner :

http://jakarta.apache.org/tomcat/tomcat-4.0-doc/catalina/docs/api/org/apache/catalina/Connector.html 
(méthode getEnableLookups).

On a normalement un accès au connecteur depuis la Request (getConnector).

A+

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