Parece que se acerca la solución al problema del remitente erróneo

[diegort]

En la lista del seguimiento de cyano de este error han comentado que ya se ha localizado este error en el código, y van a portar la función que lo contiente desde la 1.6 que si funciona bien, así que dentro de poco parece que podremos recibir sms correctamente

[jrsoft]

Por fin.
Esa es una buena noticia.

Sent from my HTC Magic using Tapatalk

[raziel-77]

Esperemos que sea verdad que lleva tiempo cyano detras del problema por lo que tengo entendido

[diegort]

Os pego el correo:

In extractEmailAddressFromMessageBody() at
http://github.com/cyanogen/android_frameworks_base/blob/eclair/telephony/java/com/android/internal/telephony/SmsMessageBase.java there is no checking if its a real email
address.
It just returns the first word separated by a " /" or space as sender if it came from
a 4 digit or less number.
I think if this function is reverted to the donut version this will be fixed.
http://github.com/cyanogen/android_frameworks_base/blob/donut/telephony/java/com/android/internal/telephony/SmsMessageBase.java

Y aquí ambas funciones

Android 2.1
protected void extractEmailAddressFromMessageBody() {

/* Some carriers may use " /" delimiter as below
*
* 1. [x@y][ ]/[subject][ ]/[body]
* -or-
* 2. [x@y][ ]/[body]
*/
String[] parts = messageBody.split("( /)|( )", 2);
if (parts.length < 2) return;
emailFrom = parts[0];
emailBody = parts[1];
isEmail = true;
}

Android 1.6
protected void extractEmailAddressFromMessageBody() {

/*
* a little guesswork here. I haven't found doc for this.
* the format could be either
*
* 1. [x@y][ ]/[subject][ ]/[body]
* -or-
* 2. [x@y][ ]/[body]
*/
int slash = 0, slash2 = 0, atSymbol = 0;

try {
slash = messageBody.indexOf(" /");
if (slash == -1) {
return;
}

atSymbol = messageBody.indexOf('@');
if (atSymbol == -1 || atSymbol > slash) {
return;
}

emailFrom = messageBody.substring(0, slash);

slash2 = messageBody.indexOf(" /", slash + 2);

if (slash2 == -1) {
pseudoSubject = null;
emailBody = messageBody.substring(slash + 2);
} else {
pseudoSubject = messageBody.substring(slash + 2, slash2);
emailBody = messageBody.substring(slash2 + 2);
}

isEmail = true;
} catch (Exception ex) {
Log.w(LOG_TAG,
"extractEmailAddressFromMessageBody: exception slash="
+ slash + ", atSymbol=" + atSymbol + ", slash2="
+ slash2, ex);
}
}
Como veis, al función del 2.1 es bastante simple, mientras que en la 1.6 sí que se hacen más comprobaciones para cerciorarse de que es correcto.

[raziel-77]

lo de simple es poco ¿en que estarian pensando en google?

[avileitor]

lo de simple es poco ¿en que estarian pensando en google?

en simplificar el código. Pero se han pasao :risitas:

[diegort]

La verdad es que si, vaya un mier** de función que se cuajaron...
Durante mis años estudiando ingeniería informática llegué a una conclusión: si una cosa funciona, aunque no sepas como, no la toques!!!

[raziel-77]

en simplificar el código. Pero se han pasao :risitas:

jejejeje Si lo simplifican un pelin mas los mensajes los enviamos por señales de humo:risitas::risitas::risitas::risitas:

[diegort]

jejejeje Si lo simplifican un pelin mas los mensajes los enviamos por señales de humo:risitas::risitas::risitas::risitas:

jajaja!!! ni que lo digas!! xD

[seke]

Simplificaciones como esa me imagino que serán las que hacen que Eclair sea más fluida...

De todos modos, es cierto que un usuario ha localizado el error, pero nadie del equipo de cyano ha dicho que vayan a portar la función ni nada del estilo, al menos no en el tracker.

Yo confío en que lo arreglen porque es una chorrada, pero a ver.

[diegort]

en la 5.0.8-test2 sigue el mismo error...

[seke]

¿Y el HTCmania team no se ve con fuerzas para arreglarlo? X-D

[diegort]

¿Y el HTCmania team no se ve con fuerzas para arreglarlo? X-D

seguro que sí, pero no se si ellos compilan el kernel o "solamente" hacen modificaciones sobre las versiones de cyano.

Cuando tenga un ratito quiero probar a compilar por mi mismo una ROM e intentar solventar fallos como ese (ahora que está localizado)

[raziel-77]

seguro que sí, pero no se si ellos compilan el kernel o "solamente" hacen modificaciones sobre las versiones de cyano.

Cuando tenga un ratito quiero probar a compilar por mi mismo una ROM e intentar solventar fallos como ese (ahora que está localizado)

Pues creo que cada uno del team comento cuando iban por libre que no compilaban, ahora que se an unido tal vez nos den algun dia la sorpresa

[diegort]

Pues van viendo buenas noticias: alguien ha probado la solución y dice que por el momento no está teniendo problemas

The fix from comment #24 works with no negative side-effects for me (so far). If it was a simple apk I'd post it but it's in the framework, which is a much bigger change. Hopefully it will be fixed for the next release.

Intento traducir:
La solución del comentario #24 funciona sin efectos secundarios para mi. Si fuese un simple apk lo podría publicar, pero es un cambio en el framework, que ya es un cambio más grande. Espero que esté solucionado en la siguiente publicación.

[dukula]

Cual es el foro en donde miras la noticia?

[diegort]

http://code.google.com/p/cyanogenmod/issues/detail?id=1419

Te suscribes ahí y te llegan al correo los comentarios que hagan sobre esta incidencia.

[pauetmv]

Por fin!!! Espero que Cyano lo ponga pronto... Es mi barrera hacia la 2.1 :(

[seke]

Ahora SÍ parece que en la siguiente versión de cyano estará solucionado este problema. O al menos tan bien como funcionaba en 1.6.

[diegort]

efectivamente!!ahora mismo acabo de ver el comentario en mi correo

[Zet4]

Por fin!!! Espero que Cyano lo ponga pronto... Es mi barrera hacia la 2.1 :(

Pues anda que no usar una 2.1 por este "problema" si se le puede llamar.. manda cojones...xd

[maxpower76]

Pues anda que no usar una 2.1 por este "problema" si se le puede llamar.. manda cojones...xd

jeje, ya ves tú... si todos los problemas fueran eso.

[pauetmv]

Pues anda que no usar una 2.1 por este "problema" si se le puede llamar.. manda cojones...xd

Bueno, depende... si cuentas todos los mensajes que me llegan mal de la familia por Qtal (num 41XX) y de la novia por A2 (num 101) empieza a no ser tan chorra la cosa... Molesta bastante ver cada mensaje que te llega en una conversación nueva.

A parte de todo que la última 1.6 de Super va como un mísil! :silbando:

[seke]

No es que sea una conversación nueva, es que es imposible saber quién te lo manda. Los 11 números más usados y no poder distinguirlos, y no lo véis un problema... pues nada oye xD.

[jrsoft]

Chicos, ya está solucionado el problema de los mensajes de numeración corta.
La solución vendrá en la próxima test5 de cyano, y supongo que a partir de ahí en todas.

[seke]

Chicos, ya está solucionado el problema de los mensajes de numeración corta.
La solución vendrá en la próxima test5 de cyano, y supongo que a partir de ahí en todas.

Ya lo hemos comentado varios mensajes atrás xD

En principio la siguiente no será test5, será 5.0.8-Stable (al menos para D/S).

5.0.8-test4 is up now! This going to be the last release (I hope) before 5.0.8-final. I'm going to be away until Sunday, so help find the bugs and make this the best CM release ever!