 | |  | | | ReviewPost Bug Reports Let us know about any post installation problems you are having with ReviewPost. |
June 18th, 2005, 11:43 AM
|
#1 (permalink)
| | Member Verified Customer
Join Date: Jun 2005
Posts: 179
| New line chars removed?
It seems that on a default install of ReviewPost (using the IP2 forum userbase) the new line character is removed from review submissions in the comment area.
Can someone point me towards the right area of code ot look at to remove this?
Users should be able to enter multiple paragrahs in reviews, however this is disabled in the current system.
|
| |
June 18th, 2005, 03:38 PM
|
#2 (permalink)
| | Photopost Developer Verified Customer
Join Date: Jun 2002 Location: Abingdon,MD
Posts: 71,940
|
HMM You can try removing this from reviews.php
$message = trim($message);
|
| |
June 18th, 2005, 07:40 PM
|
#3 (permalink)
| | Member Verified Customer
Join Date: Jun 2005
Posts: 179
|
Are there any other special characters that that trim command removes? As in, does it only remove newline chars, or will it's removal leave submissions open to XSS (cross site script) style attacks?
|
| |
June 18th, 2005, 07:46 PM
|
#4 (permalink)
| | Photopost Developer Verified Customer
Join Date: Jun 2002 Location: Abingdon,MD
Posts: 71,940
| |
| |
June 18th, 2005, 08:01 PM
|
#5 (permalink)
| | Member Verified Customer
Join Date: Jun 2005
Posts: 179
|
Thanks, now the tabs and extra spaces/return chars are stored, however the characters are simply dumped into HTML as newline and spaces, rather than coded into the appropriate %nbsp and <br/> tags.
Where abouts int he code is the best place to add routines that parse the text correclty for the display routine?
Last edited by Arnie; June 18th, 2005 at 08:14 PM.
|
| |
June 23rd, 2005, 11:58 PM
|
#6 (permalink)
| | Junior Member Verified Customer
Join Date: May 2004
Posts: 28
|
ive been meaning to sort this as well
Arnies right about that - removing that string just puts in a lot of html spacing code
im a little confused as to why this hasnt been looked it before. Surely paragraphs and breaks need to remain in order for the article to be easily read
whats the best solution to this?
|
| |
June 24th, 2005, 01:04 AM
|
#7 (permalink)
| | Registered User
Join Date: Apr 2005
Posts: 148
|
Stick $message = trim($message); back into reviews.php as that strips whitespace from the beginning and end of a string; no need to keep that space around. What you want to do instead is modify the convert_returns function in the pp-inc.php file from: Code: Content visible to verified customers only.
To something like the following: Code: Content visible to verified customers only.
|
| |
June 28th, 2005, 11:09 AM
|
#8 (permalink)
| | Junior Member Verified Customer
Join Date: May 2004
Posts: 28
|
thanks for that frost
though that doesnt seem to change anything - its just the same as the default installation, paragraphs etc still dont work
|
| |
June 30th, 2005, 10:12 AM
|
#9 (permalink)
| | Junior Member Verified Customer
Join Date: May 2004
Posts: 28
|
has anyone resolved this issue as yet
|
| |
June 30th, 2005, 10:35 AM
|
#10 (permalink)
| | Member Verified Customer
Join Date: Jun 2005
Posts: 179
|
Not that I'm aware of, I've not had a great deal that's been of use mentioned to me. Personally my feeling is that a lot of the integration with other forums is sadly somewhat lacking.
|
| |
July 13th, 2005, 02:18 AM
|
#11 (permalink)
| | Member Verified Customer
Join Date: Jun 2005
Posts: 179
|
Bug now fixed: No spaces at all in reviews(FIXED)
I posted the code that was at fault and the code changes needed. There's no word if the fix will be incorporated in the next release, but there you go.
|
| | |
Currently Active Users Viewing This Thread: 1 (0 members and 1 guests) | | | | Thread Tools | | | | Display Modes | Rate This Thread | Linear Mode | |
Posting Rules
| You may not post new threads You may not post replies You may not post attachments You may not edit your posts HTML code is Off | | | All times are GMT -5. The time now is 11:16 PM. | |