Bug en Subsonic 3.x al comparar instancias de objetos
He utilizado Subsonic de Rob Conery por varios años, desde sus primera version.
Es un ORM decente, hecho por un buen desarrollador. Subsonic ya no se desarrolla activamente (ahora se enfoca en Massive, su micro ORM), pero la ultima version, 3.0.0.4, sigue siendo utilizada por varios proyectos.
Hace algunas semanas desarrolle una funcionalidad nueva en un proyecto existente, que utilizaba Subsonic, y al estar probando la funcionalidad descubri un bug en Subsonic y la manera en que compara instancias de objetos de la misma clase, haciendo que siempre regrese false en una comparacion. El codigo problema esta en el metodo Equals, generado por el template T4 de ActiveRecord de Subsonic:
public override bool Equals(object obj)
{
if(obj.GetType()==typeof(Entity))
{
Entity compare=(Entity)obj;
return compare.KeyValue()==this.KeyValue();
}
else
{
return base.Equals(obj);
}
}
Y el metodo KeyValue hace esto:
public object KeyValue()
{
return this.EntityId;
}
Al estar comparando dos objetos, este metodo es llamado, y si los objetos que se estan comparando son de la misma clase, ejecuta la linea
return compare.KeyValue()==this.KeyValue();
El metodo KeyValue() simplemente regresa el valor del campo que este asignado como Id de la clase, ya sea un entero, un GUID, long, etc. El problema es que regresa el valor casteandolo a tipo object. Es decir, la siguiente comparacion regresara false, incorrectamente:
bool testOne = (object)5 == (object)5; //this will return false
No importa que estamos comparando un valor constante, porque lo estamos casteando a ser de tipo object. La comparacion correcta seria comparar los objetos directamente (sin castearlos), o utilizando el metodo Equals de la misma clase.
bool testTwo = 5.Equals(5); //this will return true, as we expect.
Entonces, para corregir el bug en Subsonic, se puede actualizar el template T4 de la siguiente manera:
public override bool Equals(object obj)
{
if(obj.GetType()==typeof(<#=tbl.ClassName#>))
{
<#=tbl.ClassName#> compare=(<#=tbl.ClassName#>)obj;
return compare.KeyValue().Equals(this.KeyValue());
}
else
{
return base.Equals(obj);
}
}
Es decir, reemplazando la comparacion anterior (que usaba el operador ==) con una llamada explicita al metodo Equals de la instancia del objeto que estamos comparando.
Espero les sirva.
