• 0
  • 3 RepliesReplies
  • 4952 ReadTimes Read
  • 0 PointsPoints

Topic: puedo cambiar esto por if

Usuario: Snoopix
Rango: Nuevo Usuario
Mensajes: 91
Points: 5
Mi web: http://www.ciudadsatan.com
Perfil: View Profile
Pais:
af
puedo cambiar esto por if
October 21, 2013 - 03:21 15:21
Que tal amigos de smfsimple, no sabía si poner este tema en soporte o aquí, y tampoco qué título ponerle, pero resulta que estaba dandole un toque a mi PersonalMessage.Template.php y me encontré con el siguiente código:

Code: [Select]
<div class="smalltext reportlinks">
', (!empty($modSettings['enableReportPM']) && $context['folder'] != 'sent' ? '<div class="righttext"><a href="' . $scripturl . '?action=pm;sa=report;l=' . $context['current_label_id'] . ';pmsg=' . $message['id'] . '">' . $txt['pm_report_to_admin'] . '</a></div>' : '');

echo '
</div>';

Quitandole el div "righttext" que lo encuentro innecesario para mis propósitos, está bien si escribo esto así?

Code: [Select]
<div class="smalltext reportlinks">';

if (!empty($modSettings['enableReportPM']) && $context['folder'])

echo ' <a href="' . $scripturl . '?action=pm;sa=report;l=' . $context['current_label_id'] . ';pmsg=' . $message['id'] . '">' . $txt['pm_report_to_admin'] . '</a>';

echo '
</div>';

Hice el cambio y me funciona, pero no sé si estoy pasando por alto alguna cosa, así que espero la respuesta de los maestros  ^.^
You do not have permission to give points
point 0 Points

This topic did not receive points.

Usuario: JTX3
Rango: Nuevo Usuario
Mensajes: 173
Points: 18
Mi web: http://themixure.com/index.php
Perfil: View Profile
Pais:
se
Re:puedo cambiar esto por if
Reply #1 October 21, 2013 - 08:53 20:53
Porque complicarse la vida?, lo mejor..

Code: [Select]
<div class="smalltext reportlinks">
', (!empty($modSettings['enableReportPM']) && $context['folder'] != 'sent' ? '<a href="' . $scripturl . '?action=pm;sa=report;l=' . $context['current_label_id'] . ';pmsg=' . $message['id'] . '">' . $txt['pm_report_to_admin'] . '</a>' : '');

echo '
</div>';

Y listo colega..

Usuario: Snoopix
Rango: Nuevo Usuario
Mensajes: 91
Points: 5
Mi web: http://www.ciudadsatan.com
Perfil: View Profile
Pais:
af
Re:puedo cambiar esto por if
Reply #2 October 21, 2013 - 09:42 21:42
 :D

 lo que quería era saber si era similar, porque estuve leyendo unas guías aquí y en otro lado sobre php y no termino de entender !

Usuario: manix
Rango: User Extremo
Mensajes: 437
Points: 214
Perfil: View Profile
Pais:
cr
Re:puedo cambiar esto por if
Reply #3 October 22, 2013 - 01:04 01:04
Que tal amigos de smfsimple, no sabía si poner este tema en soporte o aquí, y tampoco qué título ponerle, pero resulta que estaba dandole un toque a mi PersonalMessage.Template.php y me encontré con el siguiente código:

Code: [Select]
<div class="smalltext reportlinks">
', (!empty($modSettings['enableReportPM']) && $context['folder'] != 'sent' ? '<div class="righttext"><a href="' . $scripturl . '?action=pm;sa=report;l=' . $context['current_label_id'] . ';pmsg=' . $message['id'] . '">' . $txt['pm_report_to_admin'] . '</a></div>' : '');

echo '
</div>';

Quitandole el div "righttext" que lo encuentro innecesario para mis propósitos, está bien si escribo esto así?

Code: [Select]
<div class="smalltext reportlinks">';

if (!empty($modSettings['enableReportPM']) && $context['folder'])

echo ' <a href="' . $scripturl . '?action=pm;sa=report;l=' . $context['current_label_id'] . ';pmsg=' . $message['id'] . '">' . $txt['pm_report_to_admin'] . '</a>';

echo '
</div>';

Hice el cambio y me funciona, pero no sé si estoy pasando por alto alguna cosa, así que espero la respuesta de los maestros  ^.^


Perfecto, muy bien editado. Incluso, le estás dando más orden a tu código y eso es importante porque es más fácil de darle mantenimiento. Excelente,

 

TAGS



top

Smfsimple.com es un sitio destinado a brindar un servicio de soporte y herramientas para foros realizados con el sistema de foros gratuito simplemachines.org. Contamos con un selecto equipo de desarrolladores y conocedores del sistema, sus foros estaran en buenas manos.
This site is not affiliated with or endorsed by Simple Machines