Modify

Opened 5 years ago

Closed 5 years ago

#1850 closed Bug/Fehler (fixed)

strlen in xtc_show_category.inc.php

Reported by: noRiddle Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.6.0
Component: Template Version: 2.0.5.1
Keywords: Cc:
Blocked By: Blocking:

Description

In der /templates/THE_TEMPLATE/source/inc/xtc_show_category.inc.php gibt es diese Zeile Code:

$categories_string = substr($categories_string, 0, strlen($categories_string) -5);  //letztes  </li>  entfernen 

Ich denke, daß entweder mb_strlen() (inkl. Angabe des Encodings) benutzt werden sollte oder, besser, schlicht dies:

$categories_string = substr($categories_string, 0, -5);  //letztes  </li>  entfernen 

Da strlen() oder auch mb_strlen() ohnehin unnötig ist würde das auch einen kleinen Performance-Vorteil bringen.

Analog kann die Sache auch in der xtc_show_content.inc.php korrigiert werden.

Gruß,
noRiddle

Attachments (0)

Change History (2)

comment:1 by Torsten Riemer, 5 years ago

Component: ShopTemplate

comment:2 by Gerhard Waldemair, 5 years ago

Resolution: fixed
Status: newclosed

In 12822:

fix #1850 - removed unnecessary use of function strlen()

Modify Ticket

Action
as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.