Deprecated: str_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated in /var/www/mybb-es.com/html/inc/plugins/avatarep.php on line 1102

Deprecated: str_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated in /var/www/mybb-es.com/html/inc/class_language.php on line 201
[Fix] Evitar Salto de línea en imágenes juntas
waylayer   16 Dec, 2013, 9:23 pm
#1
Pues... hum es algo así como un reporte de bug y el fix... que no es la gran cosa la verdad.

El Bug:

En los "changedfiles" del 1.6.11 dentro de la carpeta inc tenemos el archivo class_parser.php

el cual se encarga de manejar el mycode para convertirlo al html (o algo así, si alguien puede dar una mejor explicación... bienvenido sea...)

Bueno el susodicho está en la línea 1041 (según notepad++)

la cual sería

Código:
return $this->mycode_parse_img($matches[2])."\n";

y como ven, hay un \n al final de que retorna el valor parseado de img...

y un \n es claramente un salto de línea.

[spoiler][Imagen: 001.jpg]
[Imagen: 002.jpg][/spoiler]

El Fix:

Borrar el \n o de plano borrar el ."\n"

Yo solo borré el \n

dejándolo así

Código:
return $this->mycode_parse_img($matches[2])."";

lo cual hizo que ya me funcionara perfectamente.

Aquí pueden ver un ejemplo de lo que se logra al borrar eso:

[Imagen: 001.jpg][Imagen: 002.jpg]
(si lo ven separado entonces achiquen un poco la web.... en chrome ctrl+scroll hacia abajo)

ejemplo en mi web:

Efimerizados - Todas las Ediciones

por cierto aquí una imagen adjunta de la comparación entre el class_parser.php modificado y el original. Del changedfiles 1.6.11:

[Imagen: jibd0r1.png]


pd: con monitores de 768px de ancho o menos se verá con salto ese ejemplo dentro de este foro por la forma en que está el postbit

pd2: Cuando aun no estaba aplicado el fix entonces siempre se agraba un
Código:
<br/>
al final de cada cierre de etiqueta
Código:
<img src="link"/>
Última modificación: 16 Dec, 2013, 9:25 pm por ivnic.
Eloy   17 Dec, 2013, 12:48 am
#2
Excelente Recomendación para darle Fix a ese problema

Aprendiendo cada vez mas
Dark Neo   17 Dec, 2013, 4:46 pm
#3
Yo creo que igual si se comenta esa línea o se deja con una función normal debe funcionar sin problemas:

Código:
return $this->mycode_parse_img($matches[2]);

Paraque no haga nada o comentandola igual debe ir bien, porque no se estaría ejecutando al encontrar mas de un resultado.

Código:
//return $this->mycode_parse_img($matches[2])."\n";

Simplemente es interpretar lo que hace el código, asi que debe funcionar sin problemas en ambos casos, pero al final a muchos les viene mejor porque cuando ingresas imágenes seguidas se pone una después de la otra y se ve mal el contenido, tal ves por eso lo hicieron, pero la mayroría ponemos saltos en el código.
Omar G.   17 Dec, 2013, 9:42 pm
#4
No estoy seguro comentar esa línea sea lo mejor Dark, lo mejor sería crear un MyCode desde el ACP para sustituir la original y evitar ediciones de archivos en general.
Dark Neo   17 Dec, 2013, 10:11 pm
#5
(17 Dec, 2013, 9:42 pm)Omar G. escribió: No estoy seguro comentar esa línea sea lo mejor Dark, lo mejor sería crear un MyCode desde el ACP para sustituir la original y evitar ediciones de archivos en general.

Si tienes razón xD, hace varios llamados con funciones en imágenes para hacer algunos arreglos, lo mejor es que únicamente se elimine el salto de línea con la primera función para que simplemente haga el llamado y el retorno de la variable y no mande errores.

Lo mejor es nunca editar los archivos del núcleo de MyBB, si comentas esa línea lo mas seguro que mande un error al no encontrar datos en la función y si que sería algo malo en su foro, acabo de ver la función y revisa la función de imágenes y luego de acuerdo al valor contenido de imágenes hace un recorrido para asignarlas al mismo salto de carro en ciertas ocasiones, repitiendo la consulta, con un MyCode igualmente pueda mejor funcionar diciendo que coja únicamente como en el antigüo MyCode.
Jaizu   18 Dec, 2013, 5:46 am
#6
Woa Zero me sosprendiste xDD
Algo tan simple que marca unos detalles en un buen Foro C:
waylayer   16 Jan, 2014, 2:37 pm
#7
x3 gracias jaizhu :3

no te había leído gomen x3 a los demás tampoco xD

pero bueno, el problema si era algo como un problema porque en esta nueva versión de mybb lo han corregido

pd: estas capturas las hice el mismo día que tuve los changed files.... pero recien recordé colocarlas aquí x3

[Imagen: 1722BsO.png]


la comparación es esa:

a la izquierda está la nueva clase parser y a la derecha la que había modificado yo para usarla en mi foro.

además también modificaron dos lineas más que también tenian ese salto de línea aquí se aprecia.

[Imagen: TMpwfm2.png]

eso es todo :3 ya se solucionó tal vez puedes dar por cerrado este tema :3 ya que no creo que vuelvan a cometer algo así x3

saludos Big Grin
  
Powered By MyBB, © 2002-2024 MyBB Group.
Made with by Curves UI.