|
|
|
|
Joined: Oct 2000
Posts: 60
Power User
|
|
Power User
Joined: Oct 2000
Posts: 60 |
The following block appears in both the PrintBody and Body sections of addpost:
// --------------------------------
// Get rid of HTML in the signature
$user['U_Signature'] = str_replace("<","<",$user['U_Signature']);
$user['U_Signature'] = str_replace(">",">",$user['U_Signature']);
$user['U_Signature'] = str_replace("<br>","<br>",$user['U_Signature']);
// -----------------------------
// Always markup the signature
$sig = $user['U_Signature'];
$sig = $html -> do_markup($printsig);
Is that supposed to be in there twice? I don't believe either was inside a loop....
|
|
|
|
|
Joined: May 1999
Posts: 3,039
Guru
|
|
Guru
Joined: May 1999
Posts: 3,039 |
You are right. It looks like this is a bit of redundant code that can be cleaned up.
UBB.threads Developer
|
|
|
|
|
Joined: Apr 2001
Posts: 50
Journeyman
|
|
Journeyman
Joined: Apr 2001
Posts: 50 |
'Official PHP bug fixes' is feeling neglected. How about tossing the cleaned-up version in that direction?
+++ Quis custodiet ipsos custodies? +++
|
|
|
|
|
Joined: May 1999
Posts: 3,039
Guru
|
|
Guru
Joined: May 1999
Posts: 3,039 |
Actually, upon further investigation this cannot be removed. It causes some problems with the signatures. It does need to be cleaned up however just need to figure out where the cleaning needs to take place.[]/testimages/icons/wink.gif[/]
UBB.threads Developer
|
|
|
Donate to UBBDev today to help aid in Operational, Server and Script Maintenance, and Development costs.
Please also see our parent organization VNC Web Services if you're in the need of a new UBB.threads Install or Upgrade, Site/Server Migrations, or Security and Coding Services.
|
|
|
Posts: 808
Joined: July 2001
|
|
|
Forums63
Topics37,575
Posts293,932
Members13,824
| |
Most Online6,139
Sep 21st, 2024
|
|
Currently Online
Topics Created
Posts Made
Users Online
Birthdays
|
|
|
|
|
|
