Problema con objeto en IE

[_cronos2]

Sigo teniendo problemas con IE . Tengo esta función, pero me dice que esperaba un objeto (en FF y Chrome sí funciona) :

Código Javascript:

// [...]
  opacity : function(){
   var isIE = document.all, estilos = isIE ? ['filter', 100] : ['opacity', 1], valor = isIE ? ['alpha(opacity=', ')'] : ['', ''];
   alert(this.tagName); // undefined según IE ¬¬
   if(arguments.length){
    this.style[estilos[0]] = valor[0] + (arguments[0] * estilos[1]) + valor[1];
   }else{
    if(this.style[estilos[0]]){
     return this.style[estilos[0]].match(/\d+(\.\d+)?/)[0]/estilos[1];
    }else{
     if(SS.find(this.tagName, isIE?'filter':'opacity')){
      return isIE?SS.find(this.tagName, 'filter').match(/\d+(\.\d+)?/)[0]/100:SS.find(this.tagName, 'opacity');
     }else{
      if(SS.find('#'+this.id, isIE?'filter':'opacity'))
       return isIE?SS.find('#'+this.id, 'filter').match(/\d+(\.\d+)?/)[0]/100:SS.find('#'+this.id, 'opacity');
      else{
       for(var i=0, partir=this.className.split(' '); clase=partir[i]; i++){
        if(SS.find('.'+clase, isIE?'filter':'opacity'))
         return isIE?SS.find('.'+clase, 'filter').match(/\d+(\.\d+)?/)[0]/100:SS.find('.'+clase, 'opacity');
       }
      }
     }
    }
   }
   return 1;
  }
// [...]
 

¿Alguien me puede ayudar?
Muchas gracias :D

[masterpuppet]

Que tal _cronos2, opacity es un método de que objeto ?

[_cronos2]

Es de una librería que estoy intentando montarme XD El código es muy largo pero dejo lo importante:

Código Javascript:

var JSPlus = (function(){
 var privado = {
  // [...]
  opacity : function(){
   var isIE = document.all, estilos = isIE ? ['filter', 100] : ['opacity', 1], valor = isIE ? ['alpha(opacity=', ')'] : ['', ''];
   if(arguments.length){
    this.style[estilos[0]] = valor[0] + (arguments[0] * estilos[1]) + valor[1];
   }else{
    if(this.style[estilos[0]]){
     return this.style[estilos[0]].match(/\d+(\.\d+)?/)[0]/estilos[1];
    }else{
     if(SS.find(this.tagName, estilos[0])){
      return SS.find(this.tagName, estilos[0]).match(/\d+(\.\d+)?/)[0]/estilos[1];
     }else{
      if(SS.find('#'+this.id, estilos[0]))
       return SS.find('#'+this.id, estilos[0]).match(/\d+(\.\d+)?/)[0]/estilos[1];
      else{
       for(var i=0, partir=this.className.split(' '); clase=partir[i]; i++){
        if(SS.find('.'+clase, estilos[0]))
         return SS.find('.'+clase, estilos[0]).match(/\d+(\.\d+)?/)[0]/estilos[1];
       }
      }
     }
    }
    return 1;
   }
   return P(this);
  },
  // [...]
 };
 return {
  extender : function(elem, obj){
   if(elem.extendido && elem != privado) return elem;
   for(var i in obj){
    elem[i] = obj[i];
   }
   return elem;
  },
  get : function(obj){
   switch(typeof obj){
    case 'object':
     return JSPlus.extender(obj, privado);
    break;
    case 'string': 
     return JSPlus.extender(document.querySelector(obj), privado);
    break;
   }
   return null;
  }
 };
})();
var P = JSPlus.get;
 

Entonces si por ejemplo tengo un código así:

Código Javascript:

P('#img').opacity(0.3);
 

Funciona en FF y Chrome, pero en IE me dice que se esperaba un objeto (en la línea de this.style[estilos[0]] = valor[0] + (arguments[0] * estilos[1]) + valor[1]; )
Saludos (:
PD: Gracias a @Panino por la idea del mixin, jamás se me hubiera ocurrido XD

[masterpuppet]

En que versión de IE ?, en la 8 va bien.

[_cronos2]

Cierto, acabo de probar el código y sí funciona (IE9). Entonces es un problema de scope, porque en realidad el código con el que estaba haciendo pruebas era este:

Código Javascript:

P('#img').hover(function(){ // onmouseover
 P(this).entrada(1000); // fadeIn
});
 

pero como el error me lo marcaba en la función opacity (porque para entrada uso opacity) no se me ocurrió pensar en el scope Pero sigue siendo problema de IE, porque en los demás me funciona. ¿Se te ocurre algo?
Muchas gracias por la ayuda :D

[masterpuppet]

Habría que ver el código de hover y entrada, lo podes postear ?

[_cronos2]

Aquí está:

Código Javascript:

hover : function(toDo){
   addEvent('mouseover', toDo, this);
   return P(this);
  },
 entrada : function(t){
   var tempo = { 'lento' : 800, 'medio' : 500, 'rapido' : 200};
   var tiempo = isFinite(t)?t:tempo[t], thix = P(this), incr = 20/tiempo;
   thix.opacity(0);
   var timer = setInterval(function(){
    if(thix.opacity()>=1) timer = clearInterval(timer);
    else{
     thix.opacity(thix.opacity()+incr);
    }
   }, 20);
   return P(this);
  }
 

Y la función addEvent:

Código Javascript:

function addEvent(tipo, toDo, obj){
 if(obj.attachEvent)
  obj.attachEvent('on'+tipo, toDo, false);
 else{
  if(obj.addEventListener)
   obj.addEventListener(tipo, toDo, false);
  else
   obj['on'+tipo]=toDo;
 }
 return obj;
}
 

Saludos (:

[masterpuppet]

En addEvent esta el culpable, attachEvent no funciona como addEventListener, no te bindea el scope, dentro de la funcion this hace referencia a window, una forma rapida de solucionarlo seria algo asi:

Código Javascript:

...
if(obj.attachEvent){                                    
   obj.attachEvent('on'+tipo, function() { toDo.call(obj, window.event); }, false);                 
}
...
 

Saludos.

[_cronos2]

¡Sííííííííííííííííííííííííííííííííííí! Borré esa parte del código de @Panino porque no entendí bien lo que hacía y pensé que no servía para nada Aquí dejo cómo quedó el código:

Código Javascript:

function addEvent(tipo, toDo, obj){
 if(obj.attachEvent){
  var thix = obj, fn = function(){ toDo.call(thix, window.event); };
  obj.attachEvent('on'+tipo, fn, false);
 }else{
  if(obj.addEventListener)
   obj.addEventListener(tipo, toDo, false);
  else
   obj['on'+tipo]=toDo;
 }
}
 

PD: No creo que me deje darte karma, pero aún así tienes todo mi agradecimiento (:
Edit: No, no me deja darte más karma -.-'

[IsaBelM]

de hecho es una buena practica, yo siempre lo hago. aunque no uso el tercer valor. hay alguna diferencia??

[buzu]

De hecho attachEvent solo acepta dos parametros:
http://msdn.microsoft.com/en-us/libr...8VS.85%29.aspx
Ahora que alguien explique que rayos hace ese false ahí. Mi teoría es que alguien simplemente lo copió de addEventListener, que si acepta un tercer parámetro y este hace gran diferencia.
Ahora, no hay necesidad de re-inventar la rueda, a menos que la vallas a mejorar y este no es el caso. Mejor usa addEvent de Dustin Diaz:
http://www.dustindiaz.com/rock-solid-addevent/

[_cronos2]

Iniciado por buzu

addEventListener, que si acepta un tercer parámetro y este hace gran diferencia.

¿Qué diferencia hay entre usar true y false en addEventListener?

[masterpuppet]

https://developer.mozilla.org/en/DOM...dEventListener

[_cronos2]

La verdad es que ya había estado en esa página, pero mi nivel de inglés no es tan alto ¿La traducción al español sería ...? XD

[buzu]

Para entender la diferencia primero tienes que entender el tema de los burbujeos. El único sitio que conozco que explica eso a detalle está en inglés, pero puedes leer esto para darte una idea:
http://es.wikipedia.org/wiki/Modelo_...s_de_Documento
Siempre es importante tener en cuenta la propagación de los eventos, especialmente cuando se quieren aplicar modelos como la delegación de eventos la cual resulta muy útil en sitios que usan eventos en gran medida.

[masterpuppet]

Te dejo una referencia en español http://www.versiontexto.com/event-bu...ent-capturing/, al final aparecen las referencias a quirksmode, este articulo esta muy bueno lamentablemente al igual que la mayoria de la buena info esta en ingles, Events Order QuirksMode.