Re: [Glpi-dev] Patch for ajax updateItemOnEventJsCode

2011-08-13 Thread MoYo 

Le 13/08/2011 15:24, David DURIEUX a écrit :

With patch, it's better ;)


patch applied

Thanks

Julien



Le Sat, 13 Aug 2011 15:01:21 +0200
David DURIEUX  a écrit:


Hello,

I have made fix for function Ajax::updateItemOnEventJsCode
When use $minsize or $forceloadfor parameter and if $toobserve is an
array, it don't works because use id the array instead each value of
the array.

++

David

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] Patch for ajax updateItemOnEventJsCode

2011-08-13 Thread David DURIEUX 
With patch, it's better ;)

Le Sat, 13 Aug 2011 15:01:21 +0200
David DURIEUX  a écrit:

>Hello, 
>
>I have made fix for function Ajax::updateItemOnEventJsCode
>When use $minsize or $forceloadfor parameter and if $toobserve is an
>array, it don't works because use id the array instead each value of
>the array.
>
>++
>
>David
>
>___
>Glpi-dev mailing list
>Glpi-dev@gna.org
>https://mail.gna.org/listinfo/glpi-dev
index inc/ajax.class.php
--- inc/ajax.class.php
+++ inc/ajax.class.php
@@ -173,14 +173,14 @@ class Ajax {
 function() {";
   $condition = '';
   if ($minsize >= 0) {
- $condition = " Ext.get('$toobserve').getValue().length >= $minsize ";
+ $condition = " Ext.get('$zone').getValue().length >= $minsize ";
   }
   if (count($forceloadfor)) {
  foreach ($forceloadfor as $value) {
 if (!empty($condition)) {
$condition .= " || ";
 }
-$condition .= "Ext.get('$toobserve').getValue() == '$value'";
+$condition .= "Ext.get('$zone').getValue() == '$value'";
  }
   }
   if (!empty($condition)) {
___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

[Glpi-dev] Patch for ajax updateItemOnEventJsCode

2011-08-13 Thread David DURIEUX 
Hello, 

I have made fix for function Ajax::updateItemOnEventJsCode
When use $minsize or $forceloadfor parameter and if $toobserve is an
array, it don't works because use id the array instead each value of
the array.

++

David

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] CommonDBTM::checkSpecificValues / type de paramètre

2011-08-13 Thread MoYo 

Salut,

Effectivement c'est bizarre. Ca ne doit pas être beaucoup utilisé...
La fonction elle même ne peut pas reinséré l'élément dans $this->input.
Je force la copie de l'élément après le checkSpecificValues.

++

Julien


Le 13/08/2011 09:55, Damien Touraine a écrit :

Bonjour,

J'utilises CommonDBTM::checkSpecificValues pour m'assurer que 
certaines valeurs saisies sont cohérentes et pouvoir, le cas échéant, 
les rectifier, si nécessaire (changer la casse, ...).
Le second attribue est passé, par référence. Donc, on s'attend à 
pouvoir le modifier dans checkSpecificValues. Mais, au sein de 
CommonDBTM, la valeur ainsi modifiée n'est pas réinjectée dans le 
$this->input.

Aurais-je mal compris quelque chose ?

Cordialement
Damien




___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

[Glpi-dev] CommonDBTM::checkSpecificValues / type de paramètre

2011-08-13 Thread Damien Touraine 

Bonjour,

J'utilises CommonDBTM::checkSpecificValues pour m'assurer que certaines 
valeurs saisies sont cohérentes et pouvoir, le cas échéant, les 
rectifier, si nécessaire (changer la casse, ...).
Le second attribue est passé, par référence. Donc, on s'attend à pouvoir 
le modifier dans checkSpecificValues. Mais, au sein de CommonDBTM, la 
valeur ainsi modifiée n'est pas réinjectée dans le $this->input.

Aurais-je mal compris quelque chose ?

Cordialement
Damien

--

Damien TOURAINE - Ingénieur de Recherche CNRS, LIMSI-CNRS
Groupe de RV&A "VENISE", (http://www.limsi.fr/venise/)
Bat. 508, Universite Paris-Sud 91403 Orsay cedex - +33 1 69 85 81 64



___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev