PhotoPost Photo Gallery Sales PhotoPost Sales Toll Free Phone Number
Mon-Fri 9am-4pm EST
  PhotoPost Photo Sharing Photo Gallery    Visualize community tm
| | | | | | | | |

Go Back   PhotoPost Community > PhotoPost Support > ReviewPost Pro Support Forums > ReviewPost Bug Reports

ReviewPost Bug Reports Let us know about any post installation problems you are having with ReviewPost.

Reply
 
LinkBack Thread Tools Rate Thread Display Modes
Old 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.
Arnie is offline   Reply With Quote
Old June 18th, 2005, 03:38 PM   #2 (permalink)
Photopost Developer
Verified Customer
 
Chuck S's Avatar
 
Join Date: Jun 2002
Location: Abingdon,MD
Posts: 71,940
HMM You can try removing this from reviews.php

$message = trim($message);
__________________
Photopost Developer and Support Engineer

Please do not PM me for support or sales questions. Thank you for your understanding.
Chuck S is offline   Reply With Quote
Old 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?
Arnie is offline   Reply With Quote
Old June 18th, 2005, 07:46 PM   #4 (permalink)
Photopost Developer
Verified Customer
 
Chuck S's Avatar
 
Join Date: Jun 2002
Location: Abingdon,MD
Posts: 71,940
http://us2.php.net/manual/en/function.trim.php

removes dead space and a few other things
__________________
Photopost Developer and Support Engineer

Please do not PM me for support or sales questions. Thank you for your understanding.
Chuck S is offline   Reply With Quote
Old 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.
Arnie is offline   Reply With Quote
Old 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?
fonzerelli_79 is offline   Reply With Quote
Old 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.
Frost is offline   Reply With Quote
Old 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
fonzerelli_79 is offline   Reply With Quote
Old 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
fonzerelli_79 is offline   Reply With Quote
Old 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.
Arnie is offline   Reply With Quote
Old 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.
Arnie is offline   Reply With Quote
Reply


Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
 
Thread Tools
Display Modes Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is Off
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are On


Similar Threads
Thread Thread Starter Forum Replies Last Post
Not enough chars Snowy Installs and Upgrade - vBulletin 3.5.X 2 November 10th, 2005 04:30 PM
Tags and chars in product desc Arnie ReviewPost Bug Reports 1 July 19th, 2005 09:27 PM
Delete Photo - Notifications not removed(not a bug) DJT Photopost Pro Bug Reports 5 June 2nd, 2005 04:33 AM
Curious why Version Number is removed iloco General Discussion 10 April 2nd, 2005 04:08 PM
Items that are SOLD are not being removed ludachris Classifieds Bug Reports 3 July 2nd, 2004 10:10 PM


All times are GMT -5. The time now is 11:16 PM.

Powered by vBulletin® Version 3.8.1
Copyright ©2000 - 2013, Jelsoft Enterprises Ltd.
Search Engine Friendly URLs by vBSEO 3.2.0