Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#2121 closed Bug/Fehler (fixed)

account_history_info.php - $ORDER_COMMENTS fehlt

Reported by: web-looks.de Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.7.0
Component: Shop Version: 2.0.6.0
Keywords: Cc:
Blocked By: Blocking:

Description

Im Template wird die Variable {$ORDER_COMMENTS} zur Anzeige des Kundenkommentars genutzt.
Diese Variable wird aber in der PHP Datei nicht zugewiesen.

/account_history_info.php
/templates/tpl_modified_responsive/module/account_history_info.html

Analog zur checkout_confirmation.php könnte man folgenden Code nutzen:

if (xtc_not_null($order->info['comments'])) {
	$smarty->assign('ORDER_COMMENTS', nl2br(encode_htmlspecialchars($order->info['comments'])));
}

Attachments (2)

orderHistoryBad.jpg (19.9 KB ) - added by noRiddle 4 years ago.
orderHistoryBad
orderHistoryGood.jpg (20.1 KB ) - added by noRiddle 4 years ago.
orderHistoryGood

Download all attachments as: .zip

Change History (6)

comment:1 by Gerhard Waldemair, 4 years ago

Resolution: fixed
Status: newclosed

In 13794:

fix #2121 - order comments for account history info

comment:2 by noRiddle, 4 years ago

Mich würde interessieren wie das gelöst wurde.
Siehe dazu bitte meinen Kommentar hier:
Kunden-Kommentar in der "account_history_info"

Wenn nicht bereits geschehen sollte man auch diesen Code aus der /account_history_info.php

$history_block .= xtc_date_short($statuses['date_added']). '&nbsp;<strong>' .$statuses['orders_status_name']. '</strong>&nbsp;' . (empty ($statuses['comments']) || empty($statuses['comments_sent']) ? '&nbsp;' : nl2br(encode_htmlspecialchars($statuses['comments']))) .'<br />';

ändern in

$history_block .= xtc_date_short($statuses['date_added']). '&nbsp;<strong>' .$statuses['orders_status_name']. '</strong>' . (empty ($statuses['comments']) || empty($statuses['comments_sent']) ? '&nbsp;' : '<br />'.nl2br(encode_htmlspecialchars($statuses['comments']))) .'<br />';

weil ansonsten die erste Zeile eines evtl. vorhandenen Kommentars sofort hinter dem orders_status_name ercsheint, was bescheiden aussieht, siehe beigefügte Screenshots.

Noch besser wäre das HTML rauszuwerfen und die Komponenten in einzelen Vars zu speichern, damit man das Ganze im Template formatieren und stylen kann.

Gruß,
noRiddle

by noRiddle, 4 years ago

Attachment: orderHistoryBad.jpg added

orderHistoryBad

by noRiddle, 4 years ago

Attachment: orderHistoryGood.jpg added

orderHistoryGood

comment:3 by Torsten Riemer, 4 years ago

Ich denke, dass da sogar noch ein &nbsp; entfernt werden kann, da dann überflüssig:

    $history_block .= xtc_date_short($statuses['date_added']). '&nbsp;<strong>' .$statuses['orders_status_name']. '</strong>' . (empty ($statuses['comments']) || empty($statuses['comments_sent']) ? '' : '<br />'.nl2br(encode_htmlspecialchars($statuses['comments']))) .'<br />';

comment:4 by Gerhard Waldemair, 4 years ago

In 13912:

fix #2121 - format order comments and add smarty array

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.